Re: [PATCH] test: aggregate-results.sh: consistent style. zero forks.

2019-06-10 Thread Daniel Kahn Gillmor
On Tue 2019-06-04 22:46:24 +0300, Tomi Ollila wrote: > - 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. >

[PATCH] test: aggregate-results.sh: consistent style. zero forks.

2019-06-04 Thread Tomi Ollila
- 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