Skip to content

Commit 880d83c

Browse files
committed
Rust fmt after rebase.
1 parent bc8d2e3 commit 880d83c

File tree

4 files changed

+709
-554
lines changed

4 files changed

+709
-554
lines changed

benches/bench_matching.rs

Lines changed: 18 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,7 @@ fn bench_matching_only(
8282

8383
type ParsedRequest = (String, String, String, String, bool);
8484

85-
fn bench_rule_matching_browserlike(
86-
blocker: &Engine,
87-
requests: &Vec<ParsedRequest>,
88-
) -> (u32, u32) {
85+
fn bench_rule_matching_browserlike(blocker: &Engine, requests: &Vec<ParsedRequest>) -> (u32, u32) {
8986
let mut matches = 0;
9087
let mut passes = 0;
9188
requests.iter().for_each(
@@ -350,10 +347,10 @@ fn rule_match_browserlike_comparable(c: &mut Criterion) {
350347
b.iter(|| bench_rule_matching_browserlike(&engine, &requests))
351348
});
352349
group.bench_function("brave-list", |b| {
353-
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
354-
let engine = Engine::from_rules_parametrised(rules, Default::default(), false, true);
355-
b.iter(|| bench_rule_matching_browserlike(&engine, &requests))
356-
});
350+
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
351+
let engine = Engine::from_rules_parametrised(rules, Default::default(), false, true);
352+
b.iter(|| bench_rule_matching_browserlike(&engine, &requests))
353+
});
357354

358355
group.finish();
359356
}
@@ -372,21 +369,20 @@ fn rule_match_first_request(c: &mut Criterion) {
372369
)];
373370

374371
group.bench_function("brave-list", |b| {
375-
b.iter_custom(
376-
|iters| {
377-
let mut total_time = std::time::Duration::ZERO;
378-
for _ in 0..iters {
379-
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
380-
let engine = Engine::from_rules_parametrised(rules, Default::default(), false, true);
381-
382-
// Measure only the matching time, skip setup and destruction
383-
let start_time = std::time::Instant::now();
384-
bench_rule_matching_browserlike(&engine, &requests);
385-
total_time += start_time.elapsed();
386-
}
387-
total_time
372+
b.iter_custom(|iters| {
373+
let mut total_time = std::time::Duration::ZERO;
374+
for _ in 0..iters {
375+
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
376+
let engine =
377+
Engine::from_rules_parametrised(rules, Default::default(), false, true);
378+
379+
// Measure only the matching time, skip setup and destruction
380+
let start_time = std::time::Instant::now();
381+
bench_rule_matching_browserlike(&engine, &requests);
382+
total_time += start_time.elapsed();
388383
}
389-
)
384+
total_time
385+
})
390386
});
391387

392388
group.finish();

benches/bench_memory.rs

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,12 @@
44
* You can obtain one at https://mozilla.org/MPL/2.0/. */
55

66
use criterion::*;
7+
use serde::{Deserialize, Serialize};
78
use std::alloc::{GlobalAlloc, Layout, System};
89
use std::sync::atomic::{AtomicUsize, Ordering};
9-
use serde::{Deserialize, Serialize};
1010

11-
use adblock::Engine;
1211
use adblock::request::Request;
12+
use adblock::Engine;
1313

1414
#[path = "../tests/test_utils.rs"]
1515
mod test_utils;
@@ -110,15 +110,15 @@ fn bench_memory_usage(c: &mut Criterion) {
110110
let mut result = 0;
111111
b.iter_custom(|iters| {
112112
for _ in 0..iters {
113-
ALLOCATOR.reset();
114-
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
115-
let engine = Engine::from_rules(rules, Default::default());
113+
ALLOCATOR.reset();
114+
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
115+
let engine = Engine::from_rules(rules, Default::default());
116116

117-
noise += 1; // add some noise to make criterion happy
118-
result += ALLOCATOR.current_usage() + noise;
117+
noise += 1; // add some noise to make criterion happy
118+
result += ALLOCATOR.current_usage() + noise;
119119

120-
// Prevent engine from being optimized
121-
criterion::black_box(&engine);
120+
// Prevent engine from being optimized
121+
criterion::black_box(&engine);
122122
}
123123

124124
// Return the memory usage as a Duration
@@ -134,15 +134,15 @@ fn bench_memory_usage(c: &mut Criterion) {
134134
let rules = rules_from_lists(&["data/brave/brave-main-list.txt"]);
135135
let engine = Engine::from_rules(rules, Default::default());
136136

137-
for request in first_1000_requests.clone() {
138-
criterion::black_box(engine.check_network_request(&request.into()));
139-
}
137+
for request in first_1000_requests.clone() {
138+
criterion::black_box(engine.check_network_request(&request.into()));
139+
}
140140

141-
noise += 1; // add some noise to make criterion happy
142-
result += ALLOCATOR.current_usage() + noise;
141+
noise += 1; // add some noise to make criterion happy
142+
result += ALLOCATOR.current_usage() + noise;
143143

144-
// Prevent engine from being optimized
145-
criterion::black_box(&engine);
144+
// Prevent engine from being optimized
145+
criterion::black_box(&engine);
146146
}
147147

148148
// Return the memory usage as a Duration

benches/bench_rules.rs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -99,13 +99,12 @@ fn blocker_new(c: &mut Criterion) {
9999
"data/easylist.to/easylist/easyprivacy.txt",
100100
])
101101
.collect();
102-
let brave_list_rules: Vec<_> = rules_from_lists(&[
103-
"data/brave/brave-main-list.txt",
104-
])
105-
.collect();
102+
let brave_list_rules: Vec<_> = rules_from_lists(&["data/brave/brave-main-list.txt"]).collect();
106103

107104
group.bench_function("el+ep", move |b| b.iter(|| get_blocker(&easylist_rules)));
108-
group.bench_function("brave-list", move |b| b.iter(|| get_blocker(&brave_list_rules)));
105+
group.bench_function("brave-list", move |b| {
106+
b.iter(|| get_blocker(&brave_list_rules))
107+
});
109108

110109
group.finish();
111110
}

0 commit comments

Comments
 (0)