On Wed, Jan 1, 2020 at 3:25 AM Denton Liu <liu.denton@xxxxxxxxx> wrote: > I assumed that letting these programs open their own input was one of > those minor "style cleanup" things that was done such as removing > spaces after redirection operators, indenting compound command blocks, > etc. Whenever I've been cleaning tests up, if I noticed a `sed <` in the > area, I'd apply the transformation as a style cleanup. > > Anyway, if no one else feels strongly about this, I can drop this patch > and I'll stop doing this cleanup in the future. As a reviewer, I'd just as soon not see that particular change which increases reviewer burden yet provides no practical benefit. This is especially true for these lengthy patch series which can easily lead to reviewer fatigue. (In fact, after reviewing this series -- and its predecessors -- I was strongly considering asking you to limit these cleanup series to 10 or fewer patches rather than 16, precisely due to feeling reviewer fatigue.)
