Return-Path: Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 99386 invoked by uid 500); 23 Mar 2002 03:45:33 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 99328 invoked from network); 23 Mar 2002 03:45:33 -0000 Message-ID: <3C9BFA5F.9050600@apache.org> Date: Fri, 22 Mar 2002 19:45:35 -0800 From: Ian Holsman User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.9+) Gecko/20020320 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Greg Ames , dev@httpd.apache.org Subject: Re: cvs commit: apr/atomic/os390 atomic.c References: <3C9B7902.15A5BA72@remulak.net> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Greg Ames wrote: > gregames@apache.org wrote: > > >> -apr_atomic_t apr_atomic_add(apr_atomic_t *mem, apr_uint32_t val) >> +apr_int32_t apr_atomic_add(apr_atomic_t *mem, apr_int32_t val) > > > Ian, > > Should we change the 2nd input param to apr_int32_t everywhere? This occured to > me when I used -1 there to implement apr_atomic_dec. And the return value for > apr_atomic_dec? Letting it go negative could be useful in some apps. > I can't see why not. t I'll go make sure the solaris stuff works > Greg >