> On Sat, Jul 01, 2017 at 11:01:00PM -0600, Theo de Raadt wrote: > >Don't understand your point. What is this fixing? > Nothing gets "fixed", it just avoids unnecessary forking and makes clear > that the inner commands do not require a subshell - there are no side > effects on the environment so a command list will do just fine for the > sake of bundling output.
So you fixed nothing, and you don't have an explanation. That's not good enough. Let me explain the reality. These scripts are written in idiomatic shell. So that many people can understand them. I barely understand the distinction you are making, and must presume many others in the future won't understand it either. The change doesn't make sense.