Re: [PATCH] negotiator/skipping: parse commit before queueing

2018-09-27 Thread Ævar Arnfjörð Bjarmason
On Thu, Sep 27 2018, Jonathan Tan wrote: > The skipping negotiator pushes entries onto the priority queue without > ensuring that the commit is parsed, resulting in the entry ending up in > the wrong position due to a lack of commit time. Fix this by parsing the > commit whenever we push an

[PATCH] negotiator/skipping: parse commit before queueing

2018-09-27 Thread Jonathan Tan
The skipping negotiator pushes entries onto the priority queue without ensuring that the commit is parsed, resulting in the entry ending up in the wrong position due to a lack of commit time. Fix this by parsing the commit whenever we push an entry onto the priority queue. Signed-off-by: Jonathan