Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Message-ID: <40CA2CAA.2090508@attglobal.net>
Date: Fri, 11 Jun 2004 18:05:30 -0400
From: Jeff Trawick <trawick@attglobal.net>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.6) Gecko/20040117
MIME-Version: 1.0
To: dev@apr.apache.org
Subject: [Fwd: Re: cvs commit: apr/include apr_thread_proc.h]
Content-Type: multipart/mixed;
 boundary="------------040708050004000207030301"

This is a multi-part message in MIME format.
--------------040708050004000207030301
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

(should have sent this to the list originally)

--------------040708050004000207030301
Content-Type: message/rfc822;
 name="Re: cvs commit: apr/include apr_thread_proc.h"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="Re: cvs commit: apr/include apr_thread_proc.h"

Message-ID: <40CA2C65.8030400@attglobal.net>
Date: Fri, 11 Jun 2004 18:04:21 -0400
From: Jeff Trawick <trawick@attglobal.net>
User-Agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.6) Gecko/20040117
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: Brad Nicholes <BNICHOLES@novell.com>
Subject: Re: cvs commit: apr/include apr_thread_proc.h
References: <s0c9d4c5.094@sinclair.provo.novell.com>
In-Reply-To: <s0c9d4c5.094@sinclair.provo.novell.com>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Brad Nicholes wrote:
>   We considered that but decided that adding a NetWare only enum was
> less intrusive than adding a NetWare only API to APR.  I'm fine with it
> either way, this just seemed a little cleaner.  All of the other enums
> are a no-ops for us since those concepts don't exist on NetWare.  We
> figured that it would be easier for NetWare to no-op the enums rather
> than all the other platforms having to no-op a new API.

what's cleaner for the application?  (no implication here of what is cleaner, 
but that's what is more important)

no-op-ing a new api is just copying these several lines into the other proc.c files

APR_DECLARE(apr_status_t) apr_procattr_FOO_set(apr_procattr_t *attr,
                                                FOO foo)
{
     return APR_SUCCESS;  /* or APR_ENOTIMPL depending on what promise is
                           * made */
}

but as always more opinions would be nice


--------------040708050004000207030301--