Fix cargo clippy warning and error (#877)

crossterm-rs · May 3, 2024 · 2545954 · 2545954
1 parent 39ef1e4
commit 2545954
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/macros.rs b/src/macros.rs
@@ -149,7 +149,7 @@ mod tests {
 
     // Helper for execute tests to confirm flush
     #[derive(Default, Debug, Clone)]
-    pub(self) struct FakeWrite {
+    struct FakeWrite {
         buffer: String,
         flushed: bool,
     }

diff --git a/src/style/types/color.rs b/src/style/types/color.rs
@@ -244,20 +244,20 @@ impl serde::ser::Serialize for Color {
             _ => "",
         };
 
-        if str == "" {
+        if str.is_empty() {
             match *self {
                 Color::AnsiValue(value) => {
-                    return serializer.serialize_str(&format!("ansi_({})", value));
+                    serializer.serialize_str(&format!("ansi_({})", value))
                 }
                 Color::Rgb { r, g, b } => {
-                    return serializer.serialize_str(&format!("rgb_({},{},{})", r, g, b));
+                    serializer.serialize_str(&format!("rgb_({},{},{})", r, g, b))
                 }
                 _ => {
-                    return Err(serde::ser::Error::custom("Could not serialize enum type"));
+                    Err(serde::ser::Error::custom("Could not serialize enum type"))
                 }
             }
         } else {
-            return serializer.serialize_str(str);
+            serializer.serialize_str(str)
         }
     }
 }