Remove a piece of dead code which made select_range() and
select_reverse_range() extremely inefficient (a new iterator
instance for each tuple) and confusing.

box.index.iterator() creates a closure which is sufficient
to produce all iteration results. Use of index:next()
was apparently left by some tricky merge.