From: Paul Mackerras <paulus@samba.org>
Date: Fri, 13 Jul 2007 03:45:55 +0000 (+1000)
Subject: gitk: Fix bug introduced by previous commit
X-Git-Tag: v1.5.3-rc2~18^2
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=6ebedabf2d0f77c2c765ecc5effee1a7e3ffdedb;p=git.git

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>
---

diff --git a/gitk b/gitk
index b9219c2c7..39e452aba 100755
--- a/gitk
+++ b/gitk
@@ -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