diff options
author | Paul Mackerras <paulus@samba.org> | 2007-07-13 13:45:55 +1000 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2007-07-13 13:45:55 +1000 |
commit | 6ebedabf2d0f77c2c765ecc5effee1a7e3ffdedb (patch) | |
tree | 16f4afebf217f96ae07c2131483e4fcf10472e86 /gitk | |
parent | cdaee5db165ba8bae8d3b524950e61666fc36a84 (diff) | |
download | git-6ebedabf2d0f77c2c765ecc5effee1a7e3ffdedb.tar.gz git-6ebedabf2d0f77c2c765ecc5effee1a7e3ffdedb.tar.xz |
gitk: Fix bug introduced by previous commit
When I added the "--" case to the code scanning the arguments, I missed
the fact that since the switch statement uses -regexp, the "--" case
will match any argument containing "--", e.g. "--all". This fixes it
by taking out the -regexp (since we don't actually need regular
expression matching) and adjusting the match strings.
A side effect of this is that previously any argument starting with
"-d" would be taken to indicate date mode; now the argument has to be
exactly "-d" if you want date mode.
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'gitk')
-rwxr-xr-x | gitk | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -7478,9 +7478,9 @@ set revtreeargs {} set cmdline_files {} set i 0 foreach arg $argv { - switch -regexp -- $arg { - "^$" { } - "^-d" { set datemode 1 } + switch -- $arg { + "" { } + "-d" { set datemode 1 } "--" { set cmdline_files [lrange $argv [expr {$i + 1}] end] break |