apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: svn commit: r438796 - /apr/apr/trunk/atomic/unix/apr_atomic.c
Date Thu, 31 Aug 2006 07:01:01 GMT
jerenkrantz@apache.org wrote:
> --- apr/apr/trunk/atomic/unix/apr_atomic.c (original)
> +++ apr/apr/trunk/atomic/unix/apr_atomic.c Wed Aug 30 22:10:22 2006
> @@ -179,7 +179,7 @@
>  #endif /* APR_OVERRIDE_ATOMIC_CAS32 */
>  
>  #if !defined(APR_OVERRIDE_ATOMIC_DEC32)
> -APR_DECLARE(apr_uint32_t) apr_atomic_dec32(volatile apr_uint32_t *mem)
> +APR_DECLARE(int) apr_atomic_dec32(volatile apr_uint32_t *mem)
>  {

This appears to be a binary break.

You realize we can play macro magic to enforce the correct type?

In any case, unless I'm imagining things, please revert

Mime
View raw message