Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > In patch [5/11] you dropped whitespace following the redirection > operator, however, this patch introduces several new cases of unwanted > whitespace. > > Checking "master", I see that there are 209 instances of `: >` in > tests, but 1023 instances of `>`, which suggests that we should stick > with plain `>` rather than `: >` in this patch. Agreed on both points. Thanks for a careful review.