mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Auto merge of #29811 - bluss:binary-heap-sift-less, r=gankro

Browse Source 
BinaryHeap: Simplify sift down

Sift down was doing all too much work: it can stop directly when the
current element obeys the heap property in relation to its children.

In the old code, sift down didn't compare the element to sift down at
all, so it was maximally sifted down and relied on the sift up call to
put it in the correct location.

This should speed up heapify and .pop().

Also rename Hole::removed() to Hole::element()
This commit is contained in:
bors 2015-11-13 14:28:57 +00:00
commit ec8ae4b0eb
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/libcollections/binary_heap.rs
View File

@ -522,29 +522,30 @@ impl<T: Ord> BinaryHeap<T> {
while hole.pos() > start {
let parent = (hole.pos() - 1) / 2;
if hole.removed() <= hole.get(parent) { break }
if hole.element() <= hole.get(parent) { break; }
hole.move_to(parent);
}
}
}
fn sift_down_range(&mut self, mut pos: usize, end: usize) {
let start = pos;
/// Take an element at `pos` and move it down the heap,
/// while its children are larger.
fn sift_down_range(&mut self, pos: usize, end: usize) {
unsafe {
let mut hole = Hole::new(&mut self.data, pos);
let mut child = 2 * pos + 1;
while child < end {
let right = child + 1;
// compare with the greater of the two children
if right < end && !(hole.get(child) > hole.get(right)) {
child = right;
}
// if we are already in order, stop.
if hole.element() >= hole.get(child) { break; }
hole.move_to(child);
child = 2 * hole.pos() + 1;
}
pos = hole.pos;
}
self.sift_up(start, pos);
}
fn sift_down(&mut self, pos: usize) {
@ -606,7 +607,7 @@ impl<'a, T> Hole<'a, T> {
/// Return a reference to the element removed
#[inline(always)]
fn removed(&self) -> &T {
fn element(&self) -> &T {
self.elt.as_ref().unwrap()
}