Skip to content

Commit

Permalink
gitk: Add braces around if expressions
Browse files Browse the repository at this point in the history
Apparently this simplifies things for the parser/compiler and makes
it go slightly faster (since without the braces, it potentially has
to do two levels of substitutions rather than one).

Signed-off-by: Paul Mackerras <paulus@samba.org>
  • Loading branch information
Paul Mackerras committed Feb 6, 2006
1 parent fd8ccbe commit 418c4c7
Showing 1 changed file with 11 additions and 9 deletions.
20 changes: 11 additions & 9 deletions gitk
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@ proc gitdir {} {
proc parse_args {rargs} {
global parsed_args

if [catch {
if {[catch {
set parse_args [concat --default HEAD $rargs]
set parsed_args [split [eval exec git-rev-parse $parse_args] "\n"]
}] {
}]} {
# if git-rev-parse failed for some reason...
if {$rargs == {}} {
set rargs HEAD
Expand All @@ -39,10 +39,10 @@ proc start_rev_list {rlargs} {
set startmsecs [clock clicks -milliseconds]
set nextupdate [expr {$startmsecs + 100}]
set ncmupdate 1
if [catch {
if {[catch {
set commfd [open [concat | git-rev-list --header --topo-order \
--parents $rlargs] r]
} err] {
} err]} {
puts stderr "Error executing git-rev-list: $err"
exit 1
}
Expand Down Expand Up @@ -181,7 +181,7 @@ proc doupdate {reading} {
}

proc readcommit {id} {
if [catch {set contents [exec git-cat-file commit $id]}] return
if {[catch {set contents [exec git-cat-file commit $id]}]} return
parsecommit $id $contents 0 {}
}

Expand Down Expand Up @@ -679,7 +679,7 @@ proc savestuff {w} {

proc resizeclistpanes {win w} {
global oldwidth
if [info exists oldwidth($win)] {
if {[info exists oldwidth($win)]} {
set s0 [$win sash coord 0]
set s1 [$win sash coord 1]
if {$w < 60} {
Expand Down Expand Up @@ -710,7 +710,7 @@ proc resizeclistpanes {win w} {

proc resizecdetpanes {win w} {
global oldwidth
if [info exists oldwidth($win)] {
if {[info exists oldwidth($win)]} {
set s0 [$win sash coord 0]
if {$w < 60} {
set sash0 [expr {int($w*3/4 - 2)}]
Expand Down Expand Up @@ -768,7 +768,7 @@ proc assigncolor {id} {
global parents nparents children nchildren
global cornercrossings crossings

if [info exists colormap($id)] return
if {[info exists colormap($id)]} return
set ncolors [llength $colors]
if {$nparents($id) <= 1 && $nchildren($id) == 1} {
set child [lindex $children($id) 0]
Expand Down Expand Up @@ -2912,7 +2912,9 @@ proc gettreediffs {ids} {
global treediff parents treepending
set treepending $ids
set treediff {}
if [catch {set gdtf [open [concat | git-diff-tree --no-commit-id -r $ids] r]}] return
if {[catch \
{set gdtf [open [concat | git-diff-tree --no-commit-id -r $ids] r]} \
]} return
fconfigure $gdtf -blocking 0
fileevent $gdtf readable [list gettreediffline $gdtf $ids]
}
Expand Down

0 comments on commit 418c4c7

Please sign in to comment.