OpenSource/axum
OpenSource/axum
1
0
Fork 0
mirror of https://github.com/tokio-rs/axum.git synced 2024-11-26 17:17:22 +01:00
Code Issues Projects Releases Packages Wiki Activity

Small routing cleanup

Browse source 
Storing the paths like this should be slightly more efficient.
This commit is contained in:
David Pedersen 2021-10-25 21:30:35 +02:00
parent 3626facbda
commit 85f5511313
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/routing/mod.rs
View file

@ -20,6 +20,7 @@ use std::{
fmt,
future::ready,
marker::PhantomData,
sync::Arc,
task::{Context, Poll},
};
use tower::{util::ServiceExt, ServiceBuilder};
@ -1055,10 +1056,11 @@ impl<B> Service<Request<B>> for Route<B> {
}
}
/// Wrapper around `matchit::Node` that supports merging two `Node`s.
#[derive(Clone, Default)]
struct Node {
inner: matchit::Node<RouteId>,
paths: Vec<(String, RouteId)>,
paths: Vec<(Arc<str>, RouteId)>,
}
impl Node {
@ -1069,13 +1071,13 @@ impl Node {
) -> Result<(), matchit::InsertError> {
let path = path.into();
self.inner.insert(&path, val)?;
self.paths.push((path, val));
self.paths.push((path.into(), val));
Ok(())
}
fn merge(&mut self, other: Node) -> Result<(), matchit::InsertError> {
for (path, id) in other.paths {
self.insert(path, id)?;
self.insert(&*path, id)?;
}
Ok(())
}