Return-Path: Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 76619 invoked by uid 500); 2 Jul 2002 23:05:52 -0000 Mailing-List: contact dev-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Delivered-To: mailing list dev@apr.apache.org Received: (qmail 76608 invoked from network); 2 Jul 2002 23:05:51 -0000 Date: Tue, 2 Jul 2002 17:57:04 -0500 (CDT) Message-Id: <200207022257.g62Mv4n10130@newton.ch.collab.net> From: Karl Fogel To: Cc: "'Justin Erenkrantz'" , "'William A. Rowe, Jr.'" , , Subject: Re: proposal to add apr_check_dir_empty() to APR Reply-To: kfogel@collab.net X-Windows: dissatisfaction guaranteed. X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N 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