[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Revert to using zlib-provided compress bound

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Fri, 6 Aug 2010 16:52:44 -0500

On Fri, Aug 6, 2010 at 3:51 PM, Peter Samuelson <peter_at_p12n.org> wrote:
>
> [Michael Spang]
>> This reverts to using the zlib-provided version, since the old version
>> of zlib that was missing this function should be quite rare these
>> days.
>
> Maybe I'm just old ... but I bet there's still some zlib 1.1.4
> out there.  Maybe do the following instead?  (Untested.)

You're not old...you just run Debian. ;)

/me ducks.

>
> [[[
> * subversion/libsvn_delta/svndiff.c
>  (svnCompressBound): Use zlib compressBound() if available.
> ]]]
>
> Index: libsvn_delta/svndiff.c
> ===================================================================
> --- subversion/libsvn_delta/svndiff.c      (revisione 980368)
> +++ subversion/libsvn_delta/svndiff.c      (copia locale)
> @@ -31,10 +31,12 @@
>  #include "svn_private_config.h"
>  #include <zlib.h>
>
> -/* This macro is taken from zlib, and was originally the function
> -   compressBound.  It shouldn't ever change, but once every millenium,
> -   it may be useful for someone to make sure. */
> +/* The zlib compressBound function was not exported until 1.2.0. */
> +#if ZLIB_VERNUM >= 0x1200
> +#define svnCompressBound(LEN) compressBound(LEN)
> +#else
>  #define svnCompressBound(LEN) ((LEN) + ((LEN) >> 12) + ((LEN) >> 14) + 11)
> +#endif
>
>  /* For svndiff1, address/instruction/new data under this size will not
>    be compressed using zlib as a secondary compressor.  */
>
Received on 2010-08-06 23:53:26 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.