Well, I still dislike having the APR_E*, but this would be fine with me. Ryan
---------------------------------------------- Ryan Bloom [EMAIL PROTECTED] 645 Howard St. [EMAIL PROTECTED] San Francisco, CA > -----Original Message----- > From: Karl Fogel [mailto:[EMAIL PROTECTED] > Sent: Tuesday, July 02, 2002 3:57 PM > To: [EMAIL PROTECTED] > Cc: 'Justin Erenkrantz'; 'William A. Rowe, Jr.'; dev@apr.apache.org; > [EMAIL PROTECTED] > 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
