Skip to content

Commit f29e18c

Browse files
committed
refactor: use cleaner api from clippy suggestion
1 parent 247f255 commit f29e18c

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

src/vbq/header.rs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -209,9 +209,8 @@ impl VBinseqHeader {
209209
let qual = buffer[13] != 0;
210210
let compressed = buffer[14] != 0;
211211
let paired = buffer[15] != 0;
212-
let reserved = match buffer[16..32].try_into() {
213-
Ok(reserved) => reserved,
214-
Err(_) => return Err(HeaderError::InvalidReservedBytes.into()),
212+
let Ok(reserved) = buffer[16..32].try_into() else {
213+
return Err(HeaderError::InvalidReservedBytes.into());
215214
};
216215
Ok(Self {
217216
magic,
@@ -245,9 +244,9 @@ impl VBinseqHeader {
245244
LittleEndian::write_u32(&mut buffer[0..4], self.magic);
246245
buffer[4] = self.format;
247246
LittleEndian::write_u64(&mut buffer[5..13], self.block);
248-
buffer[13] = if self.qual { 1 } else { 0 };
249-
buffer[14] = if self.compressed { 1 } else { 0 };
250-
buffer[15] = if self.paired { 1 } else { 0 }; // Fixed bug: was using self.compressed
247+
buffer[13] = self.qual.into();
248+
buffer[14] = self.compressed.into();
249+
buffer[15] = self.paired.into();
251250
buffer[16..32].copy_from_slice(&self.reserved);
252251
writer.write_all(&buffer)?;
253252
Ok(())

0 commit comments

Comments
 (0)