paint_down_to_common(): parse commit before relying on its timestamp
authorJunio C Hamano <gitster@pobox.com>
Thu, 4 Oct 2012 22:37:15 +0000 (15:37 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 4 Oct 2012 22:49:39 +0000 (15:49 -0700)
commitd866924a080215dd276c510abf255a8929b55643
treefe0ee5b7e358b77ab0ccfd32b9e29f21170f147c
parentf37d3c755209234acfc2ca280027ebdab8e9ea8a
paint_down_to_common(): parse commit before relying on its timestamp

When refactoring the merge-base computation to reduce the pairwise
O(n*(n-1)) traversals to parallel O(n) traversals, the code forgot
that timestamp based heuristics needs each commit to have been
parsed.  This caused an empty "git pull" to spend cycles, traversing
the history all the way down to 0 (because an unparsed commit object
has 0 timestamp, and any other commit object with positive timestamp
will be processed for its parents, all getting parsed), only to come
up with a merge message to be used.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
commit.c