style(lib): enforce clippy::redundant_closure_for_method_calls

Cargo.toml

@@ -125,7 +125,6 @@ multiple_inherent_impl = "allow"
 multiple_unsafe_ops_per_block = "allow"
 needless_pass_by_value = "allow"
 pattern_type_mismatch = "allow"
-redundant_closure_for_method_calls = "allow"
 redundant_else = "allow"
 struct_excessive_bools = "allow" # TODO: bogus lint?
 trivially_copy_pass_by_ref = "allow"

src/common/buf.rs

@@ -29,7 +29,7 @@ impl<T: Buf> BufList<T> {
 impl<T: Buf> Buf for BufList<T> {
     #[inline]
     fn remaining(&self) -> usize {
-        self.bufs.iter().map(|buf| buf.remaining()).sum()
+        self.bufs.iter().map(Buf::remaining).sum()
     }
 
     #[inline]

src/error.rs

@@ -354,7 +354,7 @@ impl Error {
         // Find an h2::Reason somewhere in the cause stack, if it exists,
         // otherwise assume an INTERNAL_ERROR.
         self.find_source::<h2::Error>()
-            .and_then(|h2_err| h2_err.reason())
+            .and_then(h2::Error::reason)
             .unwrap_or(h2::Reason::INTERNAL_ERROR)
     }
 

src/server/conn/http1.rs

@@ -555,7 +555,7 @@ where
     /// Return the inner IO object, and additional information provided the connection
     /// has not yet been upgraded.
     pub fn into_parts(self) -> Option<Parts<I, S>> {
-        self.inner.map(|conn| conn.into_parts())
+        self.inner.map(Connection::into_parts)
     }
 }