Re: [PATCH v2] Fix revision walk for commits with the same dates

From

Kacper Kornet <draenog@pld-linux.org>

Date

Mar 22, 2013, 21:07 UTC

Message-ID

<20130322210741.GC18210@camk.edu.pl>

In-Reply-To

<7va9pv6u4k.fsf@alter.siamese.dyndns.org>

On Fri, Mar 22, 2013 at 01:45:47PM -0700, Junio C Hamano wrote:

> Kacper Kornet <draenog@pld-linux.org> writes:

> > Logic in still_interesting function allows to stop the commits
> > traversing if the oldest processed commit is not older then the
> > youngest commit on the list to process and the list contains only
> > commits marked as not interesting ones. It can be premature when dealing
> > with a set of coequal commits. For example git rev-list A^! --not B
> > provides wrong answer if all commits in the range A..B had the same
> > commit time and there are more then 7 of them.

> > To fix this problem the relevant part of the logic in still_interesting
> > is changed to: the walk can be stopped if the oldest processed commit is
> > younger then the youngest commit on the list to processed.

> Is the made-up test case to freeze the clock even interesting?  The
> slop logic is merely a heuristic to compensate for effects caused by
> skewed or non-monototic clocks, so in a different repository you may
> even need to fuzz the timestamp comparison further

> 	if (date - 10 < src->item->date)

> or something silly like that.

-- 
  Kacper Kornet