kristopolous / TickTick

JSON in your Bash scripts
http://9ol.es/TheEmperorsNewClothes.html
Other
579 stars 55 forks source link

Remove an unnecessary grep from tickVars(); put functionality into existing sed #49

Closed XLTechie closed 4 years ago

XLTechie commented 4 years ago

In tickVars(), there was a pipe including grep and sed. Grep was only picking lines for the call to sed. Sed can do that itself, and doing it that way saves an extra program launch.

kristopolous commented 4 years ago

that was in there for compatibility reasons on systems I don't actually care about but seeing as this usage of sed works on this 1985 copy of sed I have running on this openstep/i386 vm I say screw it ... let's roll.

I bet I can qemu some hpux machine (I have some and alphas but keeping them on isn't worth the power bill). let me look into it.