From 3e79d060d4484d061df5eea888b49c5349ad938b Mon Sep 17 00:00:00 2001
From: marvin-j97 <marvin.janke.97@gmail.com>
Date: Thu, 21 Mar 2024 19:21:43 +0100
Subject: [PATCH] refactor

---
 src/merge.rs   | 20 ++++++++++----------
 src/version.rs |  8 ++------
 2 files changed, 12 insertions(+), 16 deletions(-)

diff --git a/src/merge.rs b/src/merge.rs
index 2bafffa..cb1c904 100644
--- a/src/merge.rs
+++ b/src/merge.rs
@@ -262,13 +262,13 @@ mod tests {
 
     #[test]
     fn test_snapshot_iter() -> crate::Result<()> {
-        let vec0 = vec![
+        let vec0 = [
             crate::Value::new(1u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"old", 0, ValueType::Value),
         ];
 
-        let vec1 = vec![
+        let vec1 = [
             crate::Value::new(1u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"new", 1, ValueType::Value),
@@ -350,13 +350,13 @@ mod tests {
 
     #[test]
     fn test_mixed() -> crate::Result<()> {
-        let vec0 = vec![
+        let vec0 = [
             crate::Value::new(1u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"new", 2, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"old", 0, ValueType::Value),
         ];
 
-        let vec1 = vec![
+        let vec1 = [
             crate::Value::new(1u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"new", 1, ValueType::Value),
@@ -382,13 +382,13 @@ mod tests {
 
     #[test]
     fn test_forward_merge() -> crate::Result<()> {
-        let vec0 = vec![
+        let vec0 = [
             crate::Value::new(1u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"old", 0, ValueType::Value),
         ];
 
-        let vec1 = vec![
+        let vec1 = [
             crate::Value::new(1u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"new", 1, ValueType::Value),
@@ -414,13 +414,13 @@ mod tests {
 
     #[test]
     fn test_forward_tombstone_shadowing() -> crate::Result<()> {
-        let vec0 = vec![
+        let vec0 = [
             crate::Value::new(1u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"old", 0, ValueType::Value),
         ];
 
-        let vec1 = vec![
+        let vec1 = [
             crate::Value::new(1u64.to_be_bytes(), *b"", 1, ValueType::Tombstone),
             crate::Value::new(2u64.to_be_bytes(), *b"", 1, ValueType::Tombstone),
             crate::Value::new(3u64.to_be_bytes(), *b"", 1, ValueType::Tombstone),
@@ -446,13 +446,13 @@ mod tests {
 
     #[test]
     fn test_rev_merge() -> crate::Result<()> {
-        let vec0 = vec![
+        let vec0 = [
             crate::Value::new(1u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"old", 0, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"old", 0, ValueType::Value),
         ];
 
-        let vec1 = vec![
+        let vec1 = [
             crate::Value::new(1u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(2u64.to_be_bytes(), *b"new", 1, ValueType::Value),
             crate::Value::new(3u64.to_be_bytes(), *b"new", 1, ValueType::Value),
diff --git a/src/version.rs b/src/version.rs
index 6147dee..685746d 100644
--- a/src/version.rs
+++ b/src/version.rs
@@ -39,14 +39,10 @@ impl Version {
     }
 
     pub fn parse_file_header(bytes: &[u8]) -> Option<Self> {
-        let Some(first_three) = bytes.get(0..3) else {
-            return None;
-        };
+        let first_three = bytes.get(0..3)?;
 
         if first_three == MAGIC_BYTES {
-            let Some(next_two) = bytes.get(3..5) else {
-                return None;
-            };
+            let next_two = bytes.get(3..5)?;
 
             let mut bytes = [0; 2];
             bytes.copy_from_slice(next_two);