Re: refactor gzip methods

2016-09-02 Thread Todd C. Miller
On Fri, 02 Sep 2016 12:02:20 -0400, "Ted Unangst" wrote: > this diff doesn't change any behavior, but creates separate read > and write open functions (which currently share quite a bit of code) > that only take the necessary arguments. > > it also deletes a never used zopen function. and removes

refactor gzip methods

2016-09-02 Thread Ted Unangst
the code in gzip uses a function pointer abstraction but tries to shove too much code into the same functions. this is impeding work to refactor the code to fork + pledge helper processes. in many cases, the duplication results absurd code like this: error = (decomp ? dodecompress : docom