cel
/
mastodon-async
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Optimize: Remove clone() call 

This patch makes the `Headers` helper type that is only used to be able
to implement Serialize on it only contain a reference to the actual
headers. This way we don't need to clone() the object returned by
`reqwest::Repsonse::headers()`.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
This commit is contained in:
Matthias Beyer 2022-12-27 16:06:55 +01:00 committed by Scott Boggs
parent 520247fade
commit 963a3323ea
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/helpers/log.rs
View File

@ -48,9 +48,9 @@ impl From<&reqwest::Response> for Status {
/// Helper for logging request headers /// Helper for logging request headers
#[derive(Debug)] #[derive(Debug)]
pub struct Headers(pub reqwest::header::HeaderMap); pub struct Headers<'h>(pub &'h reqwest::header::HeaderMap);
impl Serialize for Headers { impl<'h> Serialize for Headers<'h> {
fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error> fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
where where
S: serde::Serializer, S: serde::Serializer,
@ -63,8 +63,8 @@ impl Serialize for Headers {
} }
} }
impl From<&reqwest::Response> for Headers { impl<'h> From<&'h reqwest::Response> for Headers<'h> {
fn from(value: &reqwest::Response) -> Self { fn from(value: &'h reqwest::Response) -> Self {
Headers(value.headers().clone()) Headers(value.headers())
} }
} }