apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Justin Erenkrantz <jerenkra...@apache.org>
Subject Re: proposal to add apr_check_dir_empty() to APR
Date Tue, 02 Jul 2002 21:36:12 GMT
On Tue, Jul 02, 2002 at 04:00:01PM -0500, Karl Fogel wrote:
> We should either make a new status code explicitly for this, or return
> via an int* parameter.  Personally I think the int* solution is
> cleaner: the function should return an error status iff something went
> wrong, and use an entirely separate channel for returning the answer.
> Would anyone object to this solution?

Count me with Ryan - int* is a really bad idea.  Either adding a new
status code or just using EEXISTS works for me.  -- justin

View raw message