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

Re: svn commit: r15799 - trunk/subversion/include

From: <kfogel_at_collab.net>
Date: 2005-08-23 20:20:00 CEST

rooneg@tigris.org writes:
> Avoid hundreds of warnings about redefining macros now that APR has a
> version of the APR_ARRAY macros.
>
> * subversion/include/svn_types.h
> (APR_ARRAY_IDX, APR_ARRAY_PUSH): undef these before defining our own
> version if they were already defined by someone else.

Why are we preferring our own versions to the ones that now ship with APR?

Thanks,
-K

> Modified: trunk/subversion/include/svn_types.h
> Url: http://svn.collab.net/viewcvs/svn/trunk/subversion/include/svn_types.h?rev=15799&p1=trunk/subversion/include/svn_types.h&p2=trunk/subversion/include/svn_types.h&r1=15798&r2=15799
> ==============================================================================
> --- trunk/subversion/include/svn_types.h (original)
> +++ trunk/subversion/include/svn_types.h Thu Aug 18 00:50:14 2005
> @@ -66,6 +66,16 @@
>
>
>
> +/* this is now defined in APR, so undef it to avoid warnings here */
> +#ifdef APR_ARRAY_IDX
> +#undef APR_ARRAY_IDX
> +#endif
> +
> +/* ditto. */
> +#ifdef APR_ARRAY_PUSH
> +#undef APR_ARRAY_PUSH
> +#endif
> +
> /** index into an apr_array_header_t */
> #define APR_ARRAY_IDX(ary,i,type) (((type *)(ary)->elts)[i])
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: svn-help@subversion.tigris.org
>

-- 
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Aug 23 21:19:46 2005

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.