]> The Tcpdump Group git mirrors - tcpdump/commitdiff
Address all shellcheck warnings in update-test.sh.
authorDenis Ovsienko <[email protected]>
Mon, 5 Jul 2021 13:28:34 +0000 (14:28 +0100)
committerDenis Ovsienko <[email protected]>
Mon, 5 Jul 2021 13:28:34 +0000 (14:28 +0100)
update-test.sh

index 6ee18cae9e4f9fb03ed3ff82d285660d297ebbe5..ad2c7ecadfd2386cf556331eac15b04ca2eec8d3 100755 (executable)
@@ -7,13 +7,15 @@ TZ=GMT0; export TZ
 for TEST in "$@"; do
     PREFIX=tests
     MATCH=0
-    while read name input output options
+    while read -r name input output options
     do
-        [ _$name = _ ] && continue        # ignore empty lines
-        [ _${name#\#} != _$name ] && continue    # ignore comment lines
-        [ $name != "$TEST" ] && continue    # not the requested test
-        [ _$output = _ ] && continue    # ignore incomplete lines
+        [ -z "$name" ] && continue        # ignore empty lines
+        [ "${name#\#}" != "$name" ] && continue    # ignore comment lines
+        [ "$name" != "$TEST" ] && continue    # not the requested test
+        [ -z "$output" ] && continue    # ignore incomplete lines
         MATCH=1
+        # Word splitting is intentional for $options.
+        # shellcheck disable=SC2086
         ./tcpdump -# -n -r "$PREFIX/$input" $options >"$PREFIX/$output"
     done < $PREFIX/TESTLIST
     [ $MATCH = 0 ] && echo "test $TEST not found" >&2