diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2022-07-11 02:19:59 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-07-11 02:19:59 +0000 |
commit | b1b43725dff4a9cc71836f6b45c8b5f66c15abb5 (patch) | |
tree | 6ff46fce9125ab3b1d967f18665b511f6eb3aea8 /test/sys | |
parent | d09c2972b76fd3fa4abcb93b2820bb11c022cd34 (diff) | |
parent | dc1a34b864abc12bfef3de06b267d299298e01cb (diff) | |
download | nix-b1b43725dff4a9cc71836f6b45c8b5f66c15abb5.zip |
Merge #1757
1757: Clippy cleanup in the tests. r=rtzoeller a=asomers
* Remove a redundant closure.
Co-authored-by: Alan Somers <asomers@gmail.com>
Diffstat (limited to 'test/sys')
-rw-r--r-- | test/sys/test_pthread.rs | 2 | ||||
-rw-r--r-- | test/sys/test_socket.rs | 14 |
2 files changed, 6 insertions, 10 deletions
diff --git a/test/sys/test_pthread.rs b/test/sys/test_pthread.rs index fa9b510e..42a4aefa 100644 --- a/test/sys/test_pthread.rs +++ b/test/sys/test_pthread.rs @@ -4,7 +4,7 @@ use nix::sys::pthread::*; #[test] fn test_pthread_self() { let tid = pthread_self(); - assert!(tid != ::std::ptr::null_mut()); + assert!(!tid.is_null()); } #[cfg(not(any(target_env = "musl", target_os = "redox")))] diff --git a/test/sys/test_socket.rs b/test/sys/test_socket.rs index 067717bb..3a553d3a 100644 --- a/test/sys/test_socket.rs +++ b/test/sys/test_socket.rs @@ -345,7 +345,7 @@ mod recvfrom { ) .unwrap(); // Ignore from for stream sockets - let _ = sendrecv(fd1, fd2, |s, m, flags| send(s, m, flags), |_, _| {}); + let _ = sendrecv(fd1, fd2, send, |_, _| {}); } #[test] @@ -1472,7 +1472,7 @@ fn loopback_address( use std::io; use std::io::Write; - let addrs = match getifaddrs() { + let mut addrs = match getifaddrs() { Ok(iter) => iter, Err(e) => { let stdioerr = io::stderr(); @@ -1482,15 +1482,11 @@ fn loopback_address( } }; // return first address matching family - for ifaddr in addrs { - if ifaddr.flags.contains(InterfaceFlags::IFF_LOOPBACK) + addrs.find(|ifaddr| { + ifaddr.flags.contains(InterfaceFlags::IFF_LOOPBACK) && ifaddr.address.as_ref().and_then(SockaddrLike::family) == Some(family) - { - return Some(ifaddr); - } - } - None + }) } #[cfg(any( |