[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
PATCH: remove useless variable tests in runtest.exp
From: |
Jacob Bachmeyer |
Subject: |
PATCH: remove useless variable tests in runtest.exp |
Date: |
Sat, 01 Dec 2018 20:00:39 -0600 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.1.22) Gecko/20090807 MultiZilla/1.8.3.4e SeaMonkey/1.1.17 Mnenhy/0.7.6.0 |
I had overlooked this earlier but noticed it during the review for my
previous patch that added --local_init and --global_init options. The
"convenience abbreviations" "hex" and "decimal" were set only if they do
not already have a value, but this test is early enough that there is no
way for them to have been given a value.
----
ChangeLog entry:
----
* runtest.exp: Remove useless tests before setting "hex" and
"decimal" to regexp fragments.
----
patch:
----
diff --git a/runtest.exp b/runtest.exp
index c7a6f26..b0ddfed 100644
--- a/runtest.exp
+++ b/runtest.exp
@@ -99,12 +99,8 @@ unset -nocomplain env(CCACHE_NODISABLE)
#
# some convenience abbreviations
#
-if {![info exists hex]} {
- set hex "0x\[0-9A-Fa-f\]+"
-}
-if {![info exists decimal]} {
- set decimal "\[0-9\]+"
-}
+set hex "0x\[0-9A-Fa-f\]+"
+set decimal "\[0-9\]+"
#
# set the base dir (current working directory)
----
-- Jacob
- PATCH: remove useless variable tests in runtest.exp,
Jacob Bachmeyer <=