Fix clippy lints up to Rust 1.82 (#32)

src/directional_position.rs

@@ -22,23 +22,19 @@ impl DirectionalPosition {
     /// Checks whether the value can be a horizontal position.
     #[inline]
     pub fn is_horizontal(&self) -> bool {
-        match self {
-            DirectionalPosition::Center
-            | DirectionalPosition::Left
-            | DirectionalPosition::Right => true,
-            _ => false,
-        }
+        matches!(
+            self,
+            DirectionalPosition::Center | DirectionalPosition::Left | DirectionalPosition::Right
+        )
     }
 
     /// Checks whether the value can be a vertical position.
     #[inline]
     pub fn is_vertical(&self) -> bool {
-        match self {
-            DirectionalPosition::Center
-            | DirectionalPosition::Top
-            | DirectionalPosition::Bottom => true,
-            _ => false,
-        }
+        matches!(
+            self,
+            DirectionalPosition::Center | DirectionalPosition::Top | DirectionalPosition::Bottom
+        )
     }
 }
 
@@ -79,21 +75,21 @@ impl Stream<'_> {
 
         if self.starts_with(b"left") {
             self.advance(4);
-            return Ok(DirectionalPosition::Left);
+            Ok(DirectionalPosition::Left)
         } else if self.starts_with(b"right") {
             self.advance(5);
-            return Ok(DirectionalPosition::Right);
+            Ok(DirectionalPosition::Right)
         } else if self.starts_with(b"top") {
             self.advance(3);
-            return Ok(DirectionalPosition::Top);
+            Ok(DirectionalPosition::Top)
         } else if self.starts_with(b"bottom") {
             self.advance(6);
-            return Ok(DirectionalPosition::Bottom);
+            Ok(DirectionalPosition::Bottom)
         } else if self.starts_with(b"center") {
             self.advance(6);
-            return Ok(DirectionalPosition::Center);
+            Ok(DirectionalPosition::Center)
         } else {
-            return Err(Error::InvalidString(
+            Err(Error::InvalidString(
                 vec![
                     self.slice_tail().to_string(),
                     "left".to_string(),
@@ -103,7 +99,7 @@ impl Stream<'_> {
                     "center".to_string(),
                 ],
                 self.calc_char_pos(),
-            ));
+            ))
         }
     }
 }

src/font.rs

@@ -134,6 +134,7 @@ impl<'a> FontShorthand<'a> {
     /// an owned value as a return type.
     ///
     /// [font]: https://www.w3.org/TR/css-fonts-3/#font-prop
+    #[allow(clippy::should_implement_trait)] // We aren't changing public API yet.
     pub fn from_str(text: &'a str) -> Result<Self, Error> {
         let mut stream = Stream::from(text);
         stream.skip_spaces();

src/transform.rs

@@ -363,13 +363,13 @@ mod tests {
     #[test]
     fn parse_err_4() {
         let mut ts = TransformListParser::from(" ");
-        assert_eq!(ts.next().is_none(), true);
+        assert!(ts.next().is_none());
     }
 
     #[test]
     fn parse_err_5() {
         let mut ts = TransformListParser::from("\x01");
-        assert_eq!(ts.next().unwrap().is_err(), true);
+        assert!(ts.next().unwrap().is_err());
     }
 
     test_err!(parse_err_6, "rect()", "unexpected data at position 1");