Skip to content

Commit adc016f

Browse files
committed
In light of the previous commit, revert atom length for UAs and OS
Keep devices at 2 as it benefits *massively*, but UAs and OS seem to either not benefit or be slightly hampered by smaller atoms. Would be a good idea to investigate why eventually, and maybe make the bench scripts more flexible so they can be more easily run across the three categories and with varied atom lengths.
1 parent b62080b commit adc016f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ua-parser/src/lib.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ pub mod user_agent {
176176
/// Initialise an empty builder.
177177
pub fn new() -> Self {
178178
Self {
179-
builder: regex_filtered::Builder::new_atom_len(2),
179+
builder: regex_filtered::Builder::new_atom_len(3),
180180
repl: Vec::new(),
181181
}
182182
}
@@ -343,7 +343,7 @@ pub mod os {
343343
///
344344
pub fn new() -> Self {
345345
Self {
346-
builder: regex_filtered::Builder::new_atom_len(2),
346+
builder: regex_filtered::Builder::new_atom_len(3),
347347
repl: Vec::new(),
348348
}
349349
}

0 commit comments

Comments
 (0)