Skip to content

Commit a2283da

Browse files
committed
fix lints
1 parent 7f0bb21 commit a2283da

3 files changed

Lines changed: 8 additions & 16 deletions

File tree

src/iterator/range_del.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ impl FragmentedRangeTombstoneList {
214214
// Collect (seq, level) pairs from active tombstones, sorted by seq descending.
215215
let mut seq_levels: Vec<(SequenceNumber, usize)> =
216216
active.iter().map(|&idx| (raw[idx].2, raw[idx].3)).collect();
217-
seq_levels.sort_unstable_by(|a, b| b.0.cmp(&a.0));
217+
seq_levels.sort_unstable_by_key(|sl| std::cmp::Reverse(sl.0));
218218
seq_levels.dedup_by_key(|sl| sl.0);
219219

220220
fragments.push(TombstoneFragment {

src/manifest/version_set.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ impl VersionSet {
186186
}
187187

188188
// Sort L0 by file number descending (newest first)
189-
version.files[0].sort_by(|a, b| b.meta.number.cmp(&a.meta.number));
189+
version.files[0].sort_by_key(|file| std::cmp::Reverse(file.meta.number));
190190
// Sort L1+ by smallest key
191191
for level in 1..num_levels {
192192
version.files[level]

src/sst/block.rs

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -145,14 +145,10 @@ impl Block {
145145
let rp = self.restart_point(mid) as usize;
146146

147147
match decode_entry_at(&self.data, rp, &[]) {
148-
Some((key, _, _)) => {
149-
if compare(&key, target) == Ordering::Less {
150-
left = mid + 1;
151-
} else {
152-
right = mid;
153-
}
148+
Some((key, _, _)) if compare(&key, target) == Ordering::Less => {
149+
left = mid + 1;
154150
}
155-
None => {
151+
Some(_) | None => {
156152
right = mid;
157153
}
158154
}
@@ -204,14 +200,10 @@ impl Block {
204200
let rp = self.restart_point(mid) as usize;
205201

206202
match decode_entry_at(&self.data, rp, &[]) {
207-
Some((key, _, _)) => {
208-
if compare(&key, target) != Ordering::Greater {
209-
left = mid + 1;
210-
} else {
211-
right = mid;
212-
}
203+
Some((key, _, _)) if compare(&key, target) != Ordering::Greater => {
204+
left = mid + 1;
213205
}
214-
None => {
206+
Some(_) | None => {
215207
right = mid;
216208
}
217209
}

0 commit comments

Comments
 (0)