]> git.notmuchmail.org Git - notmuch/commit
test: aggregate-results.sh: consistent style. zero forks.
authorTomi Ollila <tomi.ollila@iki.fi>
Mon, 10 Jun 2019 18:39:23 +0000 (21:39 +0300)
committerDavid Bremner <david@tethera.net>
Tue, 11 Jun 2019 10:20:01 +0000 (07:20 -0300)
commit00c63bf7364778a75591fe494e029233736af04d
tree808c4281b8f1ae7e83f442bfb5c91d61a859b17f
parentbc396c967c7cd8e7a109858e428d7bf97173f7a7
test: aggregate-results.sh: consistent style. zero forks.

- all variables in $((...)) without leading $
- all comparisons use -gt, -eq or -ne
- no -a nor -o inside [ ... ] expressions
- all indentation levels using one tab

Dropped unnecessary empty string check when reading results files.

Replaced pluralize() which was executed in subshell with
pluralize_s(). pluralize_s sets $s to 's' or '' based on value of
$1. Calls to pluralize_s are done in context of current shell, so
no forks to subshells executed.
test/aggregate-results.sh