diff options
author | eri <eri@inventati.org> | 2024-03-08 16:28:19 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-08 15:28:19 +0000 |
commit | 43f44965cda8751e04195bf4c4f298147907843f (patch) | |
tree | 970527d416716fcd82caf89a5821c050b62a6145 /components/shared/net/response.rs | |
parent | 3a5ca785d3ecc1fd6cb5a519cf1a91ac61e15c8c (diff) | |
download | servo-43f44965cda8751e04195bf4c4f298147907843f.tar.gz servo-43f44965cda8751e04195bf4c4f298147907843f.zip |
clippy: fix warnings in components/shared (#31565)
* clippy: fix some warnings in components/shared
* fix: unit tests
* fix: review comments
Diffstat (limited to 'components/shared/net/response.rs')
-rw-r--r-- | components/shared/net/response.rs | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/components/shared/net/response.rs b/components/shared/net/response.rs index 15b8e53aa8b..cbdbd2342c5 100644 --- a/components/shared/net/response.rs +++ b/components/shared/net/response.rs @@ -189,10 +189,7 @@ impl Response { } pub fn is_network_error(&self) -> bool { - match self.response_type { - ResponseType::Error(..) => true, - _ => false, - } + matches!(self.response_type, ResponseType::Error(..)) } pub fn get_network_error(&self) -> Option<&NetworkError> { @@ -204,7 +201,7 @@ impl Response { pub fn actual_response(&self) -> &Response { if self.return_internal && self.internal_response.is_some() { - &**self.internal_response.as_ref().unwrap() + self.internal_response.as_ref().unwrap() } else { self } @@ -212,7 +209,7 @@ impl Response { pub fn actual_response_mut(&mut self) -> &mut Response { if self.return_internal && self.internal_response.is_some() { - &mut **self.internal_response.as_mut().unwrap() + self.internal_response.as_mut().unwrap() } else { self } @@ -258,10 +255,7 @@ impl Response { ResponseType::Basic => { let headers = old_headers.iter().filter(|(name, _)| { - match &*name.as_str().to_ascii_lowercase() { - "set-cookie" | "set-cookie2" => false, - _ => true - } + !matches!(&*name.as_str().to_ascii_lowercase(), "set-cookie" | "set-cookie2") }).map(|(n, v)| (n.clone(), v.clone())).collect(); response.headers = headers; }, @@ -315,7 +309,7 @@ impl Response { metadata.status = response.raw_status.clone(); metadata.https_state = response.https_state; metadata.referrer = response.referrer.clone(); - metadata.referrer_policy = response.referrer_policy.clone(); + metadata.referrer_policy = response.referrer_policy; metadata.redirected = response.actual_response().url_list.len() > 1; metadata } |