From 48ca2b21ad8d3ea759cea1e142969406e30c9340 Mon Sep 17 00:00:00 2001 From: fpagliughi Date: Tue, 1 Mar 2022 21:23:48 -0500 Subject: [PATCH] Fixed some clippy lints --- src/dump.rs | 6 +++--- src/err.rs | 2 +- src/lib.rs | 8 ++++---- src/util.rs | 2 +- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/dump.rs b/src/dump.rs index 4f448d98..d26fc44e 100644 --- a/src/dump.rs +++ b/src/dump.rs @@ -159,9 +159,9 @@ impl Reader { false)?; Ok(Some(CanDumpRecord { - t_us: t_us, - device: device, - frame: frame, + t_us, + device, + frame, })) } } diff --git a/src/err.rs b/src/err.rs index ccb9bbef..11d638e9 100644 --- a/src/err.rs +++ b/src/err.rs @@ -16,7 +16,7 @@ use std::{ fn get_data(frame: &CanFrame, idx: u8) -> Result { Ok(*(frame.data() .get(idx as usize) - .ok_or_else(|| CanErrorDecodingFailure::NotEnoughData(idx)))?) + .ok_or(CanErrorDecodingFailure::NotEnoughData(idx)))?) } diff --git a/src/lib.rs b/src/lib.rs index ead9746d..217b8fd4 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -59,7 +59,7 @@ //! // clippy: do not warn about things like "SocketCAN" inside the docs -#![cfg_attr(feature = "cargo-clippy", allow(doc_markdown))] +#![allow(clippy::doc_markdown)] mod err; pub use err::{CanError, CanErrorDecodingFailure, CanSocketOpenError, ConstructionError}; @@ -184,7 +184,7 @@ pub const ERR_MASK_NONE: u32 = 0; fn c_timeval_new(t: time::Duration) -> timeval { timeval { tv_sec: t.as_secs() as time_t, - tv_usec: (t.subsec_nanos() / 1000) as suseconds_t, + tv_usec: t.subsec_micros() as suseconds_t, } } @@ -445,7 +445,7 @@ impl AsRawFd for CanSocket { impl FromRawFd for CanSocket { unsafe fn from_raw_fd(fd: RawFd) -> CanSocket { - CanSocket { fd: fd } + CanSocket { fd, } } } @@ -521,7 +521,7 @@ impl CanFrame { } Ok(CanFrame { - _id: _id, + _id, _data_len: data.len() as u8, _pad: 0, _res0: 0, diff --git a/src/util.rs b/src/util.rs index cdeedaf4..cc79b8a0 100644 --- a/src/util.rs +++ b/src/util.rs @@ -47,7 +47,7 @@ pub fn set_socket_option_mult(fd: c_int, values: &[T]) -> io::Result<()> { - let rv = if values.len() < 1 { + let rv = if values.is_empty() { // can't pass in a pointer to the first element if a 0-length slice, // pass a nullpointer instead unsafe { setsockopt(fd, level, name, ptr::null(), 0) }