Skip to content

Commit 08d590e

Browse files
committed
formatting
1 parent 2235c98 commit 08d590e

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

src/algorithm/neighbour/cover_tree.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,10 @@ impl<T: Debug + PartialEq, D: Distance<T>> CoverTree<T, D> {
290290
self.new_leaf(p)
291291
} else {
292292
let max_dist = self.max(point_set);
293-
let next_scale = max_scale.checked_sub(1).map(|s| s.min(self.get_scale(max_dist))).unwrap_or(i64::MIN); // bugfix i64::MIN - 1 causes overflow
293+
let next_scale = max_scale // bugfix i64::MIN - 1 causes overflow
294+
.checked_sub(1) // returns None if overflow
295+
.map(|s| s.min(self.get_scale(max_dist)))
296+
.unwrap_or(i64::MIN); // safely getting required value
294297
if next_scale == i64::MIN {
295298
let mut children: Vec<Node> = Vec::new();
296299
let mut leaf = self.new_leaf(p);

0 commit comments

Comments
 (0)