From 0c27886e469dcf81ed0ba7b2035e32a823224edc Mon Sep 17 00:00:00 2001 From: Paul Mackerras Date: Tue, 11 Dec 2007 20:09:53 +1100 Subject: [PATCH] gitk: Fix a couple of bugs First, if we invalidate the layout for all rows (i.e. from row 0 on), we were calling undolayout with an empty string as the argument. Second, the comparison in make_disporder that tests if we need to call update_arcrows was the wrong way around. Signed-off-by: Paul Mackerras --- gitk | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gitk b/gitk index 20e84e3b8..9d1dd77d6 100755 --- a/gitk +++ b/gitk @@ -600,7 +600,8 @@ proc modify_arc {v a} { while {$a != 0 && [lindex $varcrow($v) $a] eq {}} { set a [lindex $vupptr($v) $a] } - undolayout [lindex $varcrow($v) $a] + set r [expr {$a == 0? 0: [lindex $varcrow($v) $a]}] + undolayout $r } } @@ -739,7 +740,7 @@ proc make_disporder {start end} { set la $varcmod($curview) set lrow [lindex $varcrow($curview) $la] if {$la == 0 || $lrow eq {} || \ - $end < $lrow + [llength $varccommits($curview,$la)]} { + $end > $lrow + [llength $varccommits($curview,$la)]} { update_arcrows $curview } set ai [bsearch $vrownum($curview) $start] -- 2.30.2