On Mon, Apr 24, 2017 at 6:49 PM, Jeff King <peff@xxxxxxxx> wrote: > diff --git a/prio-queue.c b/prio-queue.c > index 17252d231..fc3860fdc 100644 > --- a/prio-queue.c > +++ b/prio-queue.c > @@ -21,7 +21,7 @@ void prio_queue_reverse(struct prio_queue *queue) > > if (queue->compare != NULL) > die("BUG: prio_queue_reverse() on non-LIFO queue"); > - for (i = 0; i <= (j = (queue->nr - 1) - i); i++) > + for (i = 0; i < (j = (queue->nr - 1) - i); i++) > swap(queue, i, j); > } FWIW This shuts valgrind-3.10.1 up. -- Duy