Hello Richard, On 17/08/15(Mon) 17:39, Richard Procter wrote: > Hi, > > This series of 29 small diffs slims pf.o by 2640 bytes and pf.c by > 113 non-comment lines.
We generally discuss one diff per mail. It makes it easier for people to comment and as you can imagine deal with possible conflicts in their tree :) As you might now, Sasha has some upcoming changes with pending review :) > pf_translate(), in particular, is now much shorter and clearer[0]. > [...] > [0] As pf_translate() is called only on state creation it isn't > terribly performance sensitive, either. This is a valid point, out of curiosity what drove you in this part of the code? Martin
