berkus
/
actix-web
mirror of https://github.com/fafhrd91/actix-web
1
0
Fork 0
Code Issues Releases Wiki Activity

split bench

This commit is contained in:
Ali MJ Al-Nasrawy 2022-03-22 20:09:40 +03:00
parent 8d7afb8cfe
commit 7607dab54a
4 changed files with 56 additions and 47 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
actix-router/Cargo.toml
View File

@ -37,3 +37,7 @@ percent-encoding = "2.1"
[[bench]]
name = "router"
harness = false
[[bench]]
name = "quoter"
harness = false

50
actix-router/benches/quoter.rs Normal file
View File

@ -0,0 +1,50 @@
use criterion::{black_box, criterion_group, criterion_main, Criterion};
use std::borrow::Cow;
fn compare_quoters(c: &mut Criterion) {
let mut group = c.benchmark_group("Compare Quoters");
let quoter = actix_router::Quoter::new(b"", b"");
let path_quoted = (0..=0x7f).map(|c| format!("%{:02X}", c)).collect::<String>();
let path_unquoted = ('\u{00}'..='\u{7f}').collect::<String>();
group.bench_function("quoter_unquoted", |b| {
b.iter(|| {
for _ in 0..10 {
black_box(quoter.requote(path_unquoted.as_bytes()));
}
});
});
group.bench_function("percent_encode_unquoted", |b| {
b.iter(|| {
for _ in 0..10 {
let decode = percent_encoding::percent_decode(path_unquoted.as_bytes());
black_box(Into::<Cow<'_, [u8]>>::into(decode));
}
});
});
group.bench_function("quoter_quoted", |b| {
b.iter(|| {
for _ in 0..10 {
black_box(quoter.requote(path_quoted.as_bytes()));
}
});
});
group.bench_function("percent_encode_quoted", |b| {
b.iter(|| {
for _ in 0..10 {
let decode = percent_encoding::percent_decode(path_quoted.as_bytes());
black_box(Into::<Cow<'_, [u8]>>::into(decode));
}
});
});
group.finish();
}
criterion_group!(benches, compare_quoters);
criterion_main!(benches);

47
actix-router/benches/router.rs
View File

@ -2,8 +2,6 @@
use criterion::{black_box, criterion_group, criterion_main, Criterion};
use std::borrow::Cow;
macro_rules! register {
(colon) => {{
register!(finish => ":p1", ":p2", ":p3", ":p4")
@ -164,49 +162,6 @@ fn call() -> impl Iterator<Item = &'static str> {
IntoIterator::into_iter(arr)
}
fn compare_quoters(c: &mut Criterion) {
let mut group = c.benchmark_group("Compare Quoters");
let quoter = actix_router::Quoter::new(b"", b"");
let path = (0..=127).map(|c| format!("%{:02X}", c)).collect::<String>();
group.bench_function("quoter_x", |b| {
b.iter(|| {
for route in call() {
black_box(quoter.requote(route.as_bytes()));
}
});
});
group.bench_function("percent_encode", |b| {
b.iter(|| {
for route in call() {
let decode = percent_encoding::percent_decode(route.as_bytes());
black_box(Into::<Cow<'_, [u8]>>::into(decode));
}
});
});
group.bench_function("quoter_x_stat", |b| {
b.iter(|| {
for _ in 0..10 {
black_box(quoter.requote(path.as_bytes()));
}
});
});
group.bench_function("percent_encode_stat", |b| {
b.iter(|| {
for _ in 0..10 {
let decode = percent_encoding::percent_decode(path.as_bytes());
black_box(Into::<Cow<'_, [u8]>>::into(decode));
}
});
});
group.finish();
}
fn compare_routers(c: &mut Criterion) {
let mut group = c.benchmark_group("Compare Routers");
@ -236,5 +191,5 @@ fn compare_routers(c: &mut Criterion) {
group.finish();
}
criterion_group!(benches, compare_quoters);
criterion_group!(benches, compare_routers);
criterion_main!(benches);

2
actix-router/src/quoter.rs
View File

@ -46,7 +46,7 @@ impl Quoter {
for i in 0..val.len() {
if let (prev, [b'%', p1, p2, rem @ ..]) = val.split_at(i) {
if let Some(ch) = hex_pair_to_char(*p1, *p2)
// ingore protected ascii bytes
// ignore protected ascii bytes
.filter(|&ch| !(ch < 128 && bit_at(&self.protected_table, ch)))
{
*val = rem;