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

2019-06-11 Thread David Bremner
Tomi Ollila writes: > - all variables in $((...)) without leading $ > - all comparisons use -gt, -eq or -ne > - no -a nor -o inside [ ... ] expressions > - all indentation levels using one tab > pushed to master d ___ notmuch mailing list notmuch@notm

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

2019-06-11 Thread Daniel Kahn Gillmor
On Mon 2019-06-10 21:39:23 +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. >