diff options
author | Gary V. Vaughan <gary@gnu.org> | 2011-11-20 21:08:24 +0700 |
---|---|---|
committer | Gary V. Vaughan <gary@gnu.org> | 2011-11-25 14:53:42 +0700 |
commit | 8fad6be03e8eca44e5e771cfbb907d3840a8e01e (patch) | |
tree | 82ccc492a8aef1fef65fdaecca176331e43aa5a0 /tests/flags.at | |
parent | 4757bf0637658fd2e298042d9f7a0d51c5f49495 (diff) | |
download | libtool-8fad6be03e8eca44e5e771cfbb907d3840a8e01e.tar.gz |
syntax-check: fix violations and implement sc_useless_braces_in_variable_derefs.
Until now, libtool sources have used braced variable names
seemingly at random! Almost always the braces are just noise, so
remove all the unnecessary ones.
* cfg.mk (sc_useless_braces_in_variable_derefs): New syntax
check rule to ensure we only reintroduce braced variable
dereferences if they are followed by a valid variable name
character.
build-aux/general.m4sh, build-aux/git-hooks/commit-msg,
build-aux/ltmain.m4sh, build-aux/options-parser, configure.ac,
libltdl/configure.ac, m4/libtool.m4, m4/ltdl.m4,
m4/ltoptions.m4, tests/defs.m4sh, tests/demo-nopic.test,
tests/depdemo/configure.ac, tests/flags.at, tests/link.test,
tests/objectlist.test, tests/quote.test, tests/static.at: Remove
spurious braces.
Signed-off-by: Gary V. Vaughan <gary@gnu.org>
Diffstat (limited to 'tests/flags.at')
-rw-r--r-- | tests/flags.at | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/flags.at b/tests/flags.at index f865757d..7eebb365 100644 --- a/tests/flags.at +++ b/tests/flags.at @@ -70,7 +70,7 @@ eval "`$LIBTOOL --tag=lt_tag --config | $EGREP '^(wl|archive_cmds)='`" AT_CHECK([$LIBTOOL --tag=lt_tag --mode=compile $compile -c $source], [], [ignore], [ignore]) -# Linker flags are prefixed with ${wl} iff they are passed to the +# Linker flags are prefixed with $wl iff they are passed to the # compiler driver, instead of directly to the linker. case $archive_cmds in *\$LD*\$linker_flags*) maybe_wl= compiler_flags_in_liblink=false ;; |