From 874edd44fc41aff555b8cc39d91b6576095135df Mon Sep 17 00:00:00 2001 From: Ben Elliston Date: Wed, 12 Dec 2018 12:57:00 +1100 Subject: * config/gdb-comm.exp, config/gdb_stub.exp, config/vxworks.exp, lib/dg.exp, lib/ftp.exp, lib/kermit.exp, lib/rlogin.exp, lib/telnet.exp, runtest.exp, testsuite/lib/libsup.exp: Simplify some regular expressions in constant strings by placing them inside braces instead of quotes. This allows one level of backslash quoting to be removed. --- runtest.exp | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) (limited to 'runtest.exp') diff --git a/runtest.exp b/runtest.exp index 0bfca7d..09ffcf0 100644 --- a/runtest.exp +++ b/runtest.exp @@ -1563,16 +1563,16 @@ proc process_target_variants { target_list } { set result {} foreach x $target_list { if {[regexp "\\(" $x]} { - regsub "^.*\\((\[^()\]*)\\)$" "$x" "\\1" variant_list - regsub "\\(\[^(\]*$" "$x" "" x + regsub {^.*\(([^()]*)\)$} $x {\1} variant_list + regsub {\([^(]*$} $x "" x set list [process_target_variants $x] set result {} foreach x $list { set result [concat $result [iterate_target_variants $x [split $variant_list ","]]] } } elseif {[regexp "\{" $x]} { - regsub "^.*\{(\[^\{\}\]*)\}$" "$x" "\\1" variant_list - regsub "\{\[^\{\]*$" "$x" "" x + regsub "^.*\{(\[^\{\}\]*)\}$" $x {\1} variant_list + regsub "\{\[^\{\]*$" $x "" x set list [process_target_variants $x] foreach x $list { foreach i [split $variant_list ","] { @@ -1639,11 +1639,12 @@ if {[info exists errorInfo]} { unset errorInfo } + # make sure we have only single path delimiters -regsub -all "\(\[^/\]\)//*" $srcdir "\\1/" srcdir -regsub -all "\(\[^/\]\)//*" $objdir "\\1/" objdir -regsub -all "\(\[^/\]\)//*" $testsuitedir "\\1/" testsuitedir -regsub -all "\(\[^/\]\)//*" $testbuilddir "\\1/" testbuilddir +regsub -all {([^/])//*} $srcdir {\1/} srcdir +regsub -all {([^/])//*} $objdir {\1/} objdir +regsub -all {([^/])//*} $testsuitedir {\1/} testsuitedir +regsub -all {([^/])//*} $testbuilddir {\1/} testbuilddir if {![info exists target_list]} { # Make sure there is at least one target machine. It's probably a Unix box, -- cgit v1.1