refactor: simplify expressions

Author: sgasho

src/codec/gzip/decoder.rs

@@ -166,19 +166,13 @@ impl Decode for GzipDecoder {
         output: &mut PartialBuffer<impl AsRef<[u8]> + AsMut<[u8]>>,
     ) -> Result<bool> {
         match &mut self.state {
-            State::Done => return Ok(true),
-            State::Header(parser) => {
-                // In this case, the input was an empty gzip. Exit successfully with an empty gzip.
-                if parser.has_no_content() {
-                    return Ok(true);
-                }
-            }
-            _ => {}
+            State::Done => Ok(true),
+            // In this case, the input was an empty gzip. Exit successfully with an empty gzip.
+            State::Header(parser) if parser.has_no_content() => Ok(true),
+            _ => Err(Error::new(
+                ErrorKind::UnexpectedEof,
+                "unexpected end of file",
+            )),
         }
-
-        Err(Error::new(
-            ErrorKind::UnexpectedEof,
-            "unexpected end of file",
-        ))
     }
 }

src/codec/gzip/header.rs

@@ -1,5 +1,5 @@
 use crate::util::PartialBuffer;
-use std::io;
+use std::{io, iter};
 
 #[derive(Debug, Default)]
 struct Flags {
@@ -163,9 +163,6 @@ impl Parser {
     }
 
     pub(super) fn has_no_content(&self) -> bool {
-        match &self.state {
-            State::Fixed(data) => data.buffer().iter().all(|&b| b == 0),
-            _ => false,
-        }
+        matches!(&self.state, State::Fixed(data) if data.buffer().iter().all(|&b| b == 0))
     }
 }