apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Garrett Rooney <roo...@electricjellyfish.net>
Date Thu, 29 Jul 2004 00:20:14 GMT
Greg Stein wrote:

> On Wed, Jul 28, 2004 at 08:08:05PM -0400, Ryan Bloom wrote:
>>Basically, the macro is wrong and needs to be removed.  The contract
>>that _all_ APR API's live up to is that on a successful result, they
>>must return APR_SUCCESS.  The reason we chose to use 0 as success is
> Yup. The contract is that success is 0. There is no "system success" --
> APR should translate any/all underlying success values into 0 upon return.
> So I fully ocncur with the original design contract -- both in terms of
> what the history and design said it should be, and how I (still) think it
> should operate.
> I'm with Ryan on calling that macro wrong, and just tossing it. We tossed
> its use from SVN a long time ago and just relied on the ==0 contract.

$ find . -name \*.c -exec grep -q APR_STATUS_IS_SUCCESS {} \; -print

Apparently it's crept back in.


View raw message