Skip to content

Commit c6a8cfb

Browse files
committed
BTreeMap: code readability tweaks
1 parent a0925fb commit c6a8cfb

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

library/alloc/src/collections/btree/map.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1719,7 +1719,7 @@ impl<'a, K: 'a, V: 'a> DrainFilterInner<'a, K, V> {
17191719
/// Allow Debug implementations to predict the next element.
17201720
pub(super) fn peek(&self) -> Option<(&K, &V)> {
17211721
let edge = self.cur_leaf_edge.as_ref()?;
1722-
edge.reborrow().next_kv().ok().map(|kv| kv.into_kv())
1722+
edge.reborrow().next_kv().ok().map(Handle::into_kv)
17231723
}
17241724

17251725
/// Implementation of a typical `DrainFilter::next` method, given the predicate.

library/alloc/src/collections/btree/navigate.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ impl<BorrowType, K, V> Handle<NodeRef<BorrowType, K, V, marker::Leaf>, marker::E
218218
let mut edge = self.forget_node_type();
219219
loop {
220220
edge = match edge.right_kv() {
221-
Ok(internal_kv) => return Ok(internal_kv),
221+
Ok(kv) => return Ok(kv),
222222
Err(last_edge) => match last_edge.into_node().ascend() {
223223
Ok(parent_edge) => parent_edge.forget_node_type(),
224224
Err(root) => return Err(root),
@@ -239,7 +239,7 @@ impl<BorrowType, K, V> Handle<NodeRef<BorrowType, K, V, marker::Leaf>, marker::E
239239
let mut edge = self.forget_node_type();
240240
loop {
241241
edge = match edge.left_kv() {
242-
Ok(internal_kv) => return Ok(internal_kv),
242+
Ok(kv) => return Ok(kv),
243243
Err(last_edge) => match last_edge.into_node().ascend() {
244244
Ok(parent_edge) => parent_edge.forget_node_type(),
245245
Err(root) => return Err(root),

library/alloc/src/collections/btree/node.rs

+4-4
Original file line numberDiff line numberDiff line change
@@ -927,14 +927,14 @@ impl<'a, K: 'a, V: 'a> Handle<NodeRef<marker::Mut<'a>, K, V, marker::Leaf>, mark
927927
/// The returned pointer points to the inserted value.
928928
fn insert(mut self, key: K, val: V) -> (InsertResult<'a, K, V, marker::Leaf>, *mut V) {
929929
if self.node.len() < CAPACITY {
930-
let ptr = self.insert_fit(key, val);
930+
let val_ptr = self.insert_fit(key, val);
931931
let kv = unsafe { Handle::new_kv(self.node, self.idx) };
932-
(InsertResult::Fit(kv), ptr)
932+
(InsertResult::Fit(kv), val_ptr)
933933
} else {
934934
let (middle_kv_idx, insertion) = splitpoint(self.idx);
935935
let middle = unsafe { Handle::new_kv(self.node, middle_kv_idx) };
936936
let (mut left, k, v, mut right) = middle.split();
937-
let ptr = match insertion {
937+
let val_ptr = match insertion {
938938
InsertionPlace::Left(insert_idx) => unsafe {
939939
Handle::new_edge(left.reborrow_mut(), insert_idx).insert_fit(key, val)
940940
},
@@ -946,7 +946,7 @@ impl<'a, K: 'a, V: 'a> Handle<NodeRef<marker::Mut<'a>, K, V, marker::Leaf>, mark
946946
.insert_fit(key, val)
947947
},
948948
};
949-
(InsertResult::Split(SplitResult { left: left.forget_type(), k, v, right }), ptr)
949+
(InsertResult::Split(SplitResult { left: left.forget_type(), k, v, right }), val_ptr)
950950
}
951951
}
952952
}

0 commit comments

Comments
 (0)