Miscellaneous clippy fixes. (#868)

Author: sunfishcode

src/backend/libc/net/sockopt.rs

@@ -424,13 +424,8 @@ pub(crate) fn get_socket_reuseport_lb(fd: BorrowedFd<'_>) -> io::Result<bool> {
 ))]
 #[inline]
 pub(crate) fn get_socket_protocol(fd: BorrowedFd<'_>) -> io::Result<Option<Protocol>> {
-    getsockopt(fd, c::SOL_SOCKET, c::SO_PROTOCOL).map(|raw| {
-        if let Some(raw) = RawProtocol::new(raw) {
-            Some(Protocol::from_raw(raw))
-        } else {
-            None
-        }
-    })
+    getsockopt(fd, c::SOL_SOCKET, c::SO_PROTOCOL)
+        .map(|raw| RawProtocol::new(raw).map(Protocol::from_raw))
 }
 
 #[cfg(target_os = "linux")]
@@ -951,7 +946,7 @@ pub(crate) fn get_tcp_cork(fd: BorrowedFd<'_>) -> io::Result<bool> {
 #[cfg(linux_kernel)]
 #[inline]
 pub(crate) fn get_socket_peercred(fd: BorrowedFd<'_>) -> io::Result<UCred> {
-    getsockopt(fd, c::SOL_SOCKET as _, c::SO_PEERCRED)
+    getsockopt(fd, c::SOL_SOCKET, c::SO_PEERCRED)
 }
 
 #[inline]

src/backend/libc/termios/syscalls.rs

@@ -375,7 +375,7 @@ pub(crate) fn cfmakeraw(termios: &mut Termios) {
 pub(crate) fn isatty(fd: BorrowedFd<'_>) -> bool {
     // Use the return value of `isatty` alone. We don't check `errno` because
     // we return `bool` rather than `io::Result<bool>`, because we assume
-    // `BorrrowedFd` protects us from `EBADF`, and any other reasonably
+    // `BorrowedFd` protects us from `EBADF`, and any other reasonably
     // anticipated `errno` value would end up interpreted as “assume it's not a
     // terminal” anyway.
     unsafe { c::isatty(borrowed_fd(fd)) != 0 }

src/backend/linux_raw/net/sockopt.rs

@@ -12,10 +12,9 @@ use crate::fd::BorrowedFd;
 use crate::ffi::CStr;
 use crate::io;
 use crate::net::sockopt::Timeout;
-use crate::net::UCred;
 use crate::net::{
     AddressFamily, Ipv4Addr, Ipv6Addr, Protocol, RawProtocol, SocketAddrAny, SocketAddrStorage,
-    SocketAddrV4, SocketAddrV6, SocketType,
+    SocketAddrV4, SocketAddrV6, SocketType, UCred,
 };
 #[cfg(feature = "alloc")]
 use alloc::borrow::ToOwned;
@@ -797,7 +796,7 @@ pub(crate) fn get_tcp_cork(fd: BorrowedFd<'_>) -> io::Result<bool> {
 
 #[inline]
 pub(crate) fn get_socket_peercred(fd: BorrowedFd<'_>) -> io::Result<UCred> {
-    getsockopt(fd, c::SOL_SOCKET as _, linux_raw_sys::net::SO_PEERCRED)
+    getsockopt(fd, c::SOL_SOCKET, linux_raw_sys::net::SO_PEERCRED)
 }
 
 #[inline]

src/bitcast.rs

@@ -1,3 +1,5 @@
+//! The `bitcast` and `bitflags_bits` macros.
+
 #![allow(unused_macros)]
 
 // Ensure that the source and destination types are both primitive integer

src/runtime.rs

@@ -489,9 +489,9 @@ pub unsafe fn sigtimedwait(set: &Sigset, timeout: Option<Timespec>) -> io::Resul
 /// `getauxval(AT_SECURE)`—Returns the Linux "secure execution" mode.
 ///
 /// Return a boolean value indicating whether "secure execution" mode was
-/// requested, due the the process having elevated privileges. This includes
-/// whether the `AT_SECURE` AUX value is set, and whether the initial real
-/// UID and GID differ from the initial effective UID and GID.
+/// requested, due to the process having elevated privileges. This includes
+/// whether the `AT_SECURE` AUX value is set, and whether the initial real UID
+/// and GID differ from the initial effective UID and GID.
 ///
 /// The meaning of "secure execution" mode is beyond the scope of this comment.
 ///

src/system.rs

@@ -157,7 +157,7 @@ pub fn sethostname(name: &[u8]) -> io::Result<()> {
     backend::system::syscalls::sethostname(name)
 }
 
-/// Reboot command for use use with [`reboot`].
+/// Reboot command for use with [`reboot`].
 #[cfg(target_os = "linux")]
 #[derive(Copy, Clone, Debug, Eq, PartialEq)]
 #[repr(i32)]