gitk: Fix bug introduced in commit 67a4f1a7
authorPaul Mackerras <paulus@samba.org>
Mon, 13 Aug 2007 04:52:00 +0000 (14:52 +1000)
committerPaul Mackerras <paulus@samba.org>
Mon, 13 Aug 2007 04:52:00 +0000 (14:52 +1000)
commit7b459a1c1cc5401295e3adb12031e39e35712f4a
treea8c78e51e44dc5b44b4f90cd5a6ea05acfd9558f
parent6c87d60cc6202d4de5ac6d136394602feefeafc6
gitk: Fix bug introduced in commit 67a4f1a7

In fixing the "can't unset idinlist" error, I moved the setting of
idinlist into the loop that splits the parents into "new" parents
(i.e. those of which this is the first child) and "old" parents.
Unfortunately this is incorrect in the case where we hit the break
statement a few lines further down, since when we come back in,
we'll see idinlist($p) set for some parents that aren't in the list.

This fixes it by moving the loop that sets up newolds and oldolds
further down.

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