BoundsError in MutableBinaryHeap delete!

[ancapdev]

If run with --check-bounds=yes, the following throws BoundsError:

x = MutableBinaryMinHeap{Int}()
h1 = push!(x, 1)
h2 = push!(x, 2)
delete!(x, h2)

Offending line

        @inbounds nodes[nd_id] = new_rt = pop!(nodes)

Looks to happen when deleting from the end of the heap. The code is safe under the assumption that pop!(::Vector) doesn't resize underlying storage, but not ideal to have this throw when bounds checks are forced.

[oxinabox]

Yes, the this is definately a bug.
Do you want to make a small PR to fix it, since you have found it now?