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

Re: svn commit: r21561 - in trunk/subversion: include libsvn_client libsvn_subr

From: David Glasser <glasser_at_mit.edu>
Date: 2006-09-20 08:15:44 CEST

On 9/19/06, dlr@tigris.org <dlr@tigris.org> wrote:
> --- trunk/subversion/libsvn_subr/hash.c (original)
> +++ trunk/subversion/libsvn_subr/hash.c Tue Sep 19 15:42:07 2006
> @@ -21,6 +21,7 @@
> #include <stdlib.h>
> #include <limits.h>
> #include <assert.h>
> +#include <apr_version.h>
> #include <apr_pools.h>
> #include <apr_hash.h>
> #include <apr_file_io.h>
> @@ -429,3 +430,25 @@
>
> return SVN_NO_ERROR;
> }
> +
> +
> +/*** Misc. hash APIs ***/
> +
> +svn_error_t *
> +svn_hash_clear(apr_hash_t *hash)
> +{
> +#if APR_VERSION_AT_LEAST(1, 3, 0)

APR_VERSION_AT_LEAST is new in 1.3.0, so this breaks the build for pre-1.3.0.

I am committing a quick fix to change this to "#ifdef
APR_VERSION_AT_LEAST" to preserve the ability to build trunk; somebody
who knows APR better than me might want to make a better change.
(#ifdef does work for macros, right?)

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 20 08:15:59 2006

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.