chore: remove useless braces

tonbo-io · Jul 22, 2024 · d260406 · d260406
1 parent 870d74f
commit d260406
Showing 1 changed file with 32 additions and 34 deletions.
diff --git a/src/compaction/mod.rs b/src/compaction/mod.rs
@@ -237,29 +237,28 @@ where
         let mut meet_scopes_ll = Vec::new();
         let mut start_ll = 0;
         let mut end_ll = 0;
-        {
-            if !version.level_slice[level + 1].is_empty() {
-                *min = &meet_scopes_l
-                    .first()
-                    .ok_or(CompactionError::EmptyLevel)?
-                    .min;
-                *max = &meet_scopes_l
-                    .iter()
-                    .last()
-                    .ok_or(CompactionError::EmptyLevel)?
-                    .max;
-
-                start_ll = Version::<R, FP>::scope_search(min, &version.level_slice[level + 1]);
-                end_ll = Version::<R, FP>::scope_search(max, &version.level_slice[level + 1]);
-
-                let next_level_len = version.level_slice[level + 1].len();
-                for scope in version.level_slice[level + 1]
-                    [start_ll..cmp::min(end_ll + 1, next_level_len - 1)]
-                    .iter()
-                {
-                    if scope.contains(min) || scope.contains(max) {
-                        meet_scopes_ll.push(scope);
-                    }
+
+        if !version.level_slice[level + 1].is_empty() {
+            *min = &meet_scopes_l
+                .first()
+                .ok_or(CompactionError::EmptyLevel)?
+                .min;
+            *max = &meet_scopes_l
+                .iter()
+                .last()
+                .ok_or(CompactionError::EmptyLevel)?
+                .max;
+
+            start_ll = Version::<R, FP>::scope_search(min, &version.level_slice[level + 1]);
+            end_ll = Version::<R, FP>::scope_search(max, &version.level_slice[level + 1]);
+
+            let next_level_len = version.level_slice[level + 1].len();
+            for scope in version.level_slice[level + 1]
+                [start_ll..cmp::min(end_ll + 1, next_level_len - 1)]
+                .iter()
+            {
+                if scope.contains(min) || scope.contains(max) {
+                    meet_scopes_ll.push(scope);
                 }
             }
         }
@@ -275,19 +274,18 @@ where
         let mut meet_scopes_l = Vec::new();
         let start_l = Version::<R, FP>::scope_search(min, &version.level_slice[level]);
         let mut end_l = start_l;
-        {
-            for scope in version.level_slice[level][start_l..].iter() {
-                if scope.contains(min) || scope.contains(max) {
-                    meet_scopes_l.push(scope);
-                    end_l += 1;
-                } else {
-                    break;
-                }
-            }
-            if meet_scopes_l.is_empty() {
-                return None;
+
+        for scope in version.level_slice[level][start_l..].iter() {
+            if scope.contains(min) || scope.contains(max) {
+                meet_scopes_l.push(scope);
+                end_l += 1;
+            } else {
+                break;
             }
         }
+        if meet_scopes_l.is_empty() {
+            return None;
+        }
         Some((meet_scopes_l, start_l, end_l))
     }