Clippy fixes

src/krate_name.rs

@@ -1,6 +1,7 @@
 use crate::error::{Error, InvalidKrateName};
 
 #[cfg(test)]
+/// Create a `KrateName` from a string literal.
 #[macro_export]
 macro_rules! kn {
     ($kn:literal) => {
@@ -200,7 +201,7 @@ impl<'name> TryFrom<&'name str> for KrateName<'name> {
     }
 }
 
-impl<'name> KrateName<'name> {
+impl KrateName<'_> {
     /// Writes the crate's prefix to the specified string
     ///
     /// Cargo uses a simple prefix in the registry index so that crate's can be
@@ -259,14 +260,14 @@ impl<'name> KrateName<'name> {
 
 use std::fmt;
 
-impl<'k> fmt::Display for KrateName<'k> {
+impl fmt::Display for KrateName<'_> {
     #[inline]
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         f.write_str(self.0)
     }
 }
 
-impl<'k> fmt::Debug for KrateName<'k> {
+impl fmt::Debug for KrateName<'_> {
     #[inline]
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         f.write_str(self.0)

src/utils/flock/unix.rs

@@ -146,10 +146,10 @@ pub(super) fn is_unsupported(err: &std::io::Error) -> bool {
 
 #[inline]
 pub(super) fn is_contended(err: &Error) -> bool {
-    err.raw_os_error().map_or(false, |x| x == libc::EWOULDBLOCK)
+    err.raw_os_error() == Some(libc::EWOULDBLOCK)
 }
 
 #[inline]
 pub(super) fn is_timed_out(err: &Error) -> bool {
-    err.raw_os_error().map_or(false, |x| x == libc::EINTR)
+    err.raw_os_error() == Some(libc::EINTR)
 }

src/utils/git.rs

@@ -93,13 +93,10 @@ pub fn write_fetch_head(
                 }
 
                 rspec.local().map_or(false, |l| {
-                    l.to_str()
-                        .ok()
-                        .and_then(|l| {
-                            l.strip_prefix("refs/remotes/")
-                                .and_then(|l| l.strip_suffix("/HEAD"))
-                        })
-                        .map_or(false, |remote| remote == remote_name)
+                    l.to_str().ok().and_then(|l| {
+                        l.strip_prefix("refs/remotes/")
+                            .and_then(|l| l.strip_suffix("/HEAD"))
+                    }) == Some(remote_name)
                 })
             })
         {

tests/flock.rs

@@ -1,3 +1,5 @@
+#![allow(missing_docs)]
+
 use std::time::Duration;
 use tame_index::utils::flock::LockOptions;
 

tests/git.rs

@@ -1,3 +1,4 @@
+#![allow(missing_docs)]
 #![cfg(feature = "__git")]
 
 mod utils;

tests/local.rs

@@ -1,3 +1,4 @@
+#![allow(missing_docs)]
 #![cfg(all(feature = "local-builder", feature = "sparse"))]
 
 mod utils;

tests/sparse.rs

@@ -1,3 +1,5 @@
+#![allow(missing_docs)]
+
 mod utils;
 
 use http::header;

tests/utils.rs

@@ -1,4 +1,5 @@
 #![allow(dead_code)]
+#![allow(missing_docs)]
 
 pub use tame_index::{IndexKrate, Path, PathBuf};