[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: proposal to add apr_check_dir_empty() to APR

From: Justin Erenkrantz <jerenkrantz_at_apache.org>
Date: 2002-07-02 23:36:12 CEST

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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 2 23:36:59 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.