dejagnu
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

PATCH: fix a FIXME in dg.exp


From: Ben Elliston
Subject: PATCH: fix a FIXME in dg.exp
Date: Sun, 20 Mar 2016 21:46:36 +1100
User-agent: Mutt/1.5.23 (2014-03-12)

Here's a patch to remove another FIXME.  I don't think there is any
need to escape any of the +'s -- for example the expressions
\[0-9\]+ and \[ \t\]+ clearly don't need escaped +.

Okay to commit?

Ben


diff --git a/lib/dg.exp b/lib/dg.exp
index dfd6a6c..8b4e3c0 100644
--- a/lib/dg.exp
+++ b/lib/dg.exp
@@ -189,9 +189,7 @@ proc dg-get-options { prog } {
     set tmp [grep $prog "{\[ \t\]\+dg-\[-a-z\]\+\[ \t\]\+.*\[ \t\]\+}" line]
     if {![string match "" $tmp]} {
        foreach i $tmp {
-           # FIXME: When to use "+" and "\+" isn't clear.
-           # Seems to me it took awhile to get this to work.
-           regexp "(\[0-9\]\+)\[ \t\]\+{\[ \t\]+(dg-\[-a-z\]+)\[ \t\]\+(.*)\[ 
\t\]+}\[^\}\]*(\n|$)" $i i line cmd args
+           regexp "(\[0-9\]+)\[ \t\]+{\[ \t\]+(dg-\[-a-z\]+)\[ \t\]+(.*)\[ 
\t\]+}\[^\}\]*(\n|$)" $i i line cmd args
            append result " { $cmd $line $args }"
        }
     }

Attachment: signature.asc
Description: Digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]