From dev-return-6065-apmail-apr-dev-archive=apr.apache.org@apr.apache.org Fri Mar 22 18:37:03 2002 Return-Path: Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 84400 invoked by uid 500); 22 Mar 2002 18:37:03 -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 84387 invoked from network); 22 Mar 2002 18:37:01 -0000 Sender: gregames@Mail.MeepZor.Com Message-ID: <3C9B7902.15A5BA72@remulak.net> Date: Fri, 22 Mar 2002 13:33:38 -0500 From: Greg Ames X-Mailer: Mozilla 4.77 [en] (X11; U; Linux 2.2.19-10mdk i686) MIME-Version: 1.0 To: dev@apr.apache.org Subject: Re: cvs commit: apr/atomic/os390 atomic.c References: <20020322182916.61097.qmail@icarus.apache.org> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N 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. Greg