From 3005247c9bbe3da70554f12a388adeca13e1c593 Mon Sep 17 00:00:00 2001
From: Yuri Astrakhan <YuriAstrakhan@gmail.com>
Date: Fri, 1 Sep 2023 00:03:36 -0400
Subject: [PATCH] A few clippy lint fixes

* added `[must_use]`
* added a few semicolons
---
 embedded-can/src/id.rs          | 7 +++++++
 embedded-hal-async/src/delay.rs | 4 ++--
 embedded-hal/src/delay.rs       | 4 ++--
 embedded-io-async/src/lib.rs    | 8 ++++----
 embedded-io/src/lib.rs          | 8 ++++----
 5 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/embedded-can/src/id.rs b/embedded-can/src/id.rs
index 6fb377190..2774b6c3d 100644
--- a/embedded-can/src/id.rs
+++ b/embedded-can/src/id.rs
@@ -15,6 +15,7 @@ impl StandardId {
     ///
     /// This will return `None` if `raw` is out of range of an 11-bit integer (`> 0x7FF`).
     #[inline]
+    #[must_use]
     pub const fn new(raw: u16) -> Option<Self> {
         if raw <= 0x7FF {
             Some(Self(raw))
@@ -28,12 +29,14 @@ impl StandardId {
     /// # Safety
     /// Using this method can create an invalid ID and is thus marked as unsafe.
     #[inline]
+    #[must_use]
     pub const unsafe fn new_unchecked(raw: u16) -> Self {
         Self(raw)
     }
 
     /// Returns this CAN Identifier as a raw 16-bit integer.
     #[inline]
+    #[must_use]
     pub const fn as_raw(&self) -> u16 {
         self.0
     }
@@ -54,6 +57,7 @@ impl ExtendedId {
     ///
     /// This will return `None` if `raw` is out of range of an 29-bit integer (`> 0x1FFF_FFFF`).
     #[inline]
+    #[must_use]
     pub const fn new(raw: u32) -> Option<Self> {
         if raw <= 0x1FFF_FFFF {
             Some(Self(raw))
@@ -67,17 +71,20 @@ impl ExtendedId {
     /// # Safety
     /// Using this method can create an invalid ID and is thus marked as unsafe.
     #[inline]
+    #[must_use]
     pub const unsafe fn new_unchecked(raw: u32) -> Self {
         Self(raw)
     }
 
     /// Returns this CAN Identifier as a raw 32-bit integer.
     #[inline]
+    #[must_use]
     pub const fn as_raw(&self) -> u32 {
         self.0
     }
 
     /// Returns the Base ID part of this extended identifier.
+    #[must_use]
     pub fn standard_id(&self) -> StandardId {
         // ID-28 to ID-18
         StandardId((self.0 >> 18) as u16)
diff --git a/embedded-hal-async/src/delay.rs b/embedded-hal-async/src/delay.rs
index c1d57c8e6..62bd742f2 100644
--- a/embedded-hal-async/src/delay.rs
+++ b/embedded-hal-async/src/delay.rs
@@ -17,11 +17,11 @@ where
 {
     #[inline]
     async fn delay_us(&mut self, us: u32) {
-        T::delay_us(self, us).await
+        T::delay_us(self, us).await;
     }
 
     #[inline]
     async fn delay_ms(&mut self, ms: u32) {
-        T::delay_ms(self, ms).await
+        T::delay_ms(self, ms).await;
     }
 }
diff --git a/embedded-hal/src/delay.rs b/embedded-hal/src/delay.rs
index 32f085729..77b5633d7 100644
--- a/embedded-hal/src/delay.rs
+++ b/embedded-hal/src/delay.rs
@@ -22,11 +22,11 @@ where
 {
     #[inline]
     fn delay_us(&mut self, us: u32) {
-        T::delay_us(self, us)
+        T::delay_us(self, us);
     }
 
     #[inline]
     fn delay_ms(&mut self, ms: u32) {
-        T::delay_ms(self, ms)
+        T::delay_ms(self, ms);
     }
 }
diff --git a/embedded-io-async/src/lib.rs b/embedded-io-async/src/lib.rs
index c94e349a4..98a656be8 100644
--- a/embedded-io-async/src/lib.rs
+++ b/embedded-io-async/src/lib.rs
@@ -58,10 +58,10 @@ pub trait Read: ErrorType {
                 Err(e) => return Err(ReadExactError::Other(e)),
             }
         }
-        if !buf.is_empty() {
-            Err(ReadExactError::UnexpectedEof)
-        } else {
+        if buf.is_empty() {
             Ok(())
+        } else {
+            Err(ReadExactError::UnexpectedEof)
         }
     }
 }
@@ -170,7 +170,7 @@ impl<T: ?Sized + BufRead> BufRead for &mut T {
     }
 
     fn consume(&mut self, amt: usize) {
-        T::consume(self, amt)
+        T::consume(self, amt);
     }
 }
 
diff --git a/embedded-io/src/lib.rs b/embedded-io/src/lib.rs
index f798b5b6e..28a496743 100644
--- a/embedded-io/src/lib.rs
+++ b/embedded-io/src/lib.rs
@@ -362,10 +362,10 @@ pub trait Read: ErrorType {
                 Err(e) => return Err(ReadExactError::Other(e)),
             }
         }
-        if !buf.is_empty() {
-            Err(ReadExactError::UnexpectedEof)
-        } else {
+        if buf.is_empty() {
             Ok(())
+        } else {
+            Err(ReadExactError::UnexpectedEof)
         }
     }
 }
@@ -535,7 +535,7 @@ impl<T: ?Sized + BufRead> BufRead for &mut T {
     }
 
     fn consume(&mut self, amt: usize) {
-        T::consume(self, amt)
+        T::consume(self, amt);
     }
 }