apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan Bloom" <...@covalent.net>
Subject RE: proposal to add apr_check_dir_empty() to APR
Date Tue, 02 Jul 2002 23:11:04 GMT
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



Mime
View raw message