Code

gitk: Fix a bug in make_disporder
authorPaul Mackerras <paulus@samba.org>
Sat, 12 Jan 2008 10:46:31 +0000 (21:46 +1100)
committerPaul Mackerras <paulus@samba.org>
Sat, 12 Jan 2008 10:46:31 +0000 (21:46 +1100)
commit17529cf9bc50e8b2bc0ea930a966b8b47056853e
treec101e983421cb3881de779fbe0b2e9b0e7ffae91
parent5be25a8f8560a88145ea31f0ca530393ee2094da
gitk: Fix a bug in make_disporder

The make_disporder function has an optimization where it assumed that
if displayorder was already long enough and the first entry in it for
a particular arc was non-null, then the whole arc was present.  This
turns out not to be true in some circumstances, since we can add a
commit to an arc (which truncates displayorder to the previous end of
that arc), then call make_disporder for later arcs (which will pad
displayorder with null elements), then call make_disporder for the
first arc - which won't update the null elements.

This fixes it by changing the optimization to check the last element
for the arc instead of the first.

Signed-off-by: Paul Mackerras <paulus@samba.org>
gitk