Re: The Performance Elephant is Dead

[Stefan Fuhrmann]
> * zlib: major deflate() and adler32() speedup, minor inflate() speedup

What is the story with the s->level > 0 thing in deflate.c? I note
that you slide the hash table only if s->level > 0, but the comment
right above this states:

            /* Slide the hash table (could be avoided with 32 bit values
               at the expense of memory usage). We slide even when level == 0
               to keep the hash table consistent if we switch back to level > 0
               later. (Using level 0 permanently is not an optimal usage of
               zlib, so we don't care about this pathological case.)
             */

From this comment, it looks like adding your "if (s->level > 0)" adds a
bug. The only time it would work properly is if you use what they call
"this pathological case", which Subversion does not.

-- 
Peter Samuelson | org-tld!p12n!peter | http://p12n.org/