Well, I still dislike having the APR_E*, but this would be fine with me.
Ryan
----------------------------------------------
Ryan Bloom rbb@covalent.net
645 Howard St. rbb@apache.org
San Francisco, CA
> -----Original Message-----
> From: Karl Fogel [mailto:kfogel@newton.ch.collab.net]
> Sent: Tuesday, July 02, 2002 3:57 PM
> To: rbb@covalent.net
> Cc: 'Justin Erenkrantz'; 'William A. Rowe, Jr.'; dev@apr.apache.org;
> dev@subversion.tigris.org
> Subject: Re: proposal to add apr_check_dir_empty() to APR
>
> Okay, wait a sec here:
>
> Why not just do this?
>
> APR_SUCCESS | APR_STATUS_IS_SUCCESS ===> the dir is empty
> APR_ENOTEMPTY | APR_STATUS_IS_ENOTEMPTY ===> the dir is not
empty
>
> No need for new status codes here. We have existing status codes that
> have the right meaning, and that cannot conflict with any true error
> that the function might actually need to return.
>
> -Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jul 3 01:11:34 2002