style(clippy): fix

Author: noamteyssier

src/record/sequencing_record.rs

@@ -161,15 +161,15 @@ impl<'a> SequencingRecord<'a> {
     /// extra data in the record that the writer won't use.
     ///
     /// The VBQ record layout is:
-    /// - Flag (8 bytes, if has_flags)
-    /// - s_len (8 bytes)
-    /// - x_len (8 bytes)
-    /// - s_seq (encoded, rounded up to 8-byte words)
-    /// - s_qual (raw bytes, if has_qualities)
-    /// - s_header_len + s_header (8 + len bytes, if has_headers and s_header present)
-    /// - x_seq (encoded, rounded up to 8-byte words, if paired)
-    /// - x_qual (raw bytes, if has_qualities and paired)
-    /// - x_header_len + x_header (8 + len bytes, if has_headers and x_header present)
+    /// - Flag (8 bytes, if `has_flags`)
+    /// - `s_len` (8 bytes)
+    /// - `x_len` (8 bytes)
+    /// - `s_seq` (encoded, rounded up to 8-byte words)
+    /// - `s_qual` (raw bytes, if `has_qualities`)
+    /// - `s_header_len` + `s_header` (8 + len bytes, if `has_headers` and `s_header` present)
+    /// - `x_seq` (encoded, rounded up to 8-byte words, if paired)
+    /// - `x_qual` (raw bytes, if `has_qualities` and paired)
+    /// - `x_header_len` + `x_header` (8 + len bytes, if `has_headers` and `x_header` present)
     #[inline]
     #[must_use]
     pub fn configured_size_vbq(
@@ -222,10 +222,8 @@ impl<'a> SequencingRecord<'a> {
             if let Some(h) = self.s_header {
                 size += 8 + h.len(); // length prefix + header bytes
             }
-            if is_paired {
-                if let Some(h) = self.x_header {
-                    size += 8 + h.len(); // length prefix + header bytes
-                }
+            if is_paired && let Some(h) = self.x_header {
+                size += 8 + h.len(); // length prefix + header bytes
             }
         }
 

src/utils/fastx.rs

@@ -104,6 +104,7 @@ impl FastxEncoderBuilder {
     ///     .run()?;
     /// # Ok::<(), binseq::Error>(())
     /// ```
+    #[must_use]
     pub fn input_stdin(mut self) -> Self {
         self.input = Some(FastxInput::Stdin);
         self
@@ -150,6 +151,7 @@ impl FastxEncoderBuilder {
     ///     .run()?;
     /// # Ok::<(), binseq::Error>(())
     /// ```
+    #[must_use]
     pub fn threads(mut self, n: usize) -> Self {
         self.threads = n;
         self

src/vbq/writer.rs

@@ -897,18 +897,18 @@ impl BlockWriter {
         self.write_buffer(sbuf)?;
 
         // Write primary quality (only if configured)
-        if self.has_qualities {
-            if let Some(qual) = record.s_qual {
-                self.write_u8buf(qual)?;
-            }
+        if self.has_qualities
+            && let Some(qual) = record.s_qual
+        {
+            self.write_u8buf(qual)?;
         }
 
         // Write primary header (only if configured)
-        if self.has_headers {
-            if let Some(sheader) = record.s_header {
-                self.write_length(sheader.len() as u64)?;
-                self.write_u8buf(sheader)?;
-            }
+        if self.has_headers
+            && let Some(sheader) = record.s_header
+        {
+            self.write_length(sheader.len() as u64)?;
+            self.write_u8buf(sheader)?;
         }
 
         // Write the optional extended sequence
@@ -917,18 +917,18 @@ impl BlockWriter {
         }
 
         // Write extended quality (only if configured)
-        if self.has_qualities {
-            if let Some(qual) = record.x_qual {
-                self.write_u8buf(qual)?;
-            }
+        if self.has_qualities
+            && let Some(qual) = record.x_qual
+        {
+            self.write_u8buf(qual)?;
         }
 
         // Write extended header (only if configured)
-        if self.has_headers {
-            if let Some(xheader) = record.x_header {
-                self.write_length(xheader.len() as u64)?;
-                self.write_u8buf(xheader)?;
-            }
+        if self.has_headers
+            && let Some(xheader) = record.x_header
+        {
+            self.write_length(xheader.len() as u64)?;
+            self.write_u8buf(xheader)?;
         }
 
         Ok(())