You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Hudson <gh...@MIT.EDU> on 2004/07/29 02:08:02 UTC

Re: svn commit: r10438 - in trunk/subversion: libsvn_fs_fs libsvn_ra_svn libsvn_subr tests/libsvn_delta

On Wed, 2004-07-28 at 20:42, jerenkrantz@tigris.org wrote:
> APR_STATUS_IS_SUCCESS is not recommended.

It isn't?  Why not?  I don't see any indication in apr_errno.h that it's
deprecated.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: r10438 - in trunk/subversion: libsvn_fs_fs libsvn_ra_svn libsvn_subr tests/libsvn_delta

Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
--On Wednesday, July 28, 2004 10:08 PM -0400 Greg Hudson <gh...@MIT.EDU> 
wrote:

> On Wed, 2004-07-28 at 20:42, jerenkrantz@tigris.org wrote:
>> APR_STATUS_IS_SUCCESS is not recommended.
>
> It isn't?  Why not?  I don't see any indication in apr_errno.h that it's
> deprecated.

There was a dev@apr thread today that brought up that APR_STATUS_IS_SUCCESS 
snuck back in to Subversion: we had previously gone through SVN and tossed 
those before.  (APR_STATUS_IS_SUCCESS was mis-used by some APR folks 
internally when they added that macro.  APR_SUCCESS is always defined to be 0 
- no other value is legitimate.)

*shrug*  -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org