diff options
| author | David Disseldorp <ddiss@suse.de> | 2022-07-08 10:51:40 +0200 |
|---|---|---|
| committer | Zorro Lang <zlang@kernel.org> | 2022-07-09 01:13:22 +0800 |
| commit | 5c31ba9b72991dcf6f26931b646ae17b3ebeef72 (patch) | |
| tree | 75ddddd8c4ea13366bd5118212a2c7f06b895e01 /check | |
| parent | e47930fb73bcfe9c046fa5bb01ae293bbca559f9 (diff) | |
| download | xfstests-dev-5c31ba9b72991dcf6f26931b646ae17b3ebeef72.tar.gz | |
check: make a few variables local
The variables aren't used outside of function scope. Also convert one
timestamp output to use the helper.
Signed-off-by: David Disseldorp <ddiss@suse.de>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Zorro Lang <zlang@kernel.org>
Diffstat (limited to 'check')
| -rwxr-xr-x | check | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -179,10 +179,10 @@ get_all_tests() # the function from that list. trim_test_list() { - test_list="$*" + local test_list="$*" rm -f $tmp.grep - numsed=0 + local numsed=0 for t in $test_list do if [ $numsed -gt 100 ]; then @@ -207,7 +207,7 @@ _wallclock() _timestamp() { - now=`date "+%T"` + local now=`date "+%T"` echo -n " [$now]" } @@ -603,7 +603,7 @@ fi function run_section() { - local section=$1 + local section=$1 skip OLD_FSTYP=$FSTYP OLD_TEST_FS_MOUNT_OPTS=$TEST_FS_MOUNT_OPTS @@ -820,7 +820,7 @@ function run_section() rm -f core $seqres.notrun start=`_wallclock` - $timestamp && echo -n " ["`date "+%T"`"]" + $timestamp && _timestamp [ ! -x $seq ] && chmod u+x $seq # ensure we can run it $LOGGER_PROG "run xfstest $seqnum" if [ -w /dev/kmsg ]; then |