[prev in list] [next in list] [prev in thread] [next in thread] 

List:       apr-dev
Subject:    Re: cvs commit: apr/atomic/os390 atomic.c
From:       Greg Ames <gregames () remulak ! net>
Date:       2002-03-22 18:33:38
[Download RAW message or body]

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
[prev in list] [next in list] [prev in thread] [next in thread]