tree 7e8a99b6a92ce3ab82b859f0e2995283a1a1c7b7
parent 4cabd42a78d242650b1053520af308011061343e
author Eric Biggers <ebiggers@google.com> 1523068529 -0700
committer Eryu Guan <guaneryu@gmail.com> 1523530344 +0800

common/rc: add missing 'local' keywords

Many helper functions in xfstests are unnecessarily declaring variables
without the 'local' keyword, which pollutes the global namespace and can
collide with variables in tests.  Fix this for everything in common/rc
that I could find.

In _math(), just eliminate $result by removing the check for nonempty
$BC, which is redundant with _require_math() which the tests do.

Signed-off-by: Eric Biggers <ebiggers@google.com>
Reviewed-by: Eryu Guan <guaneryu@gmail.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>