Re: cvs commit: apr/include apr_atomic.h

[Joe Orton <jo...@manyfish.co.uk>]

On Tue, Apr 30, 2002 at 11:43:29PM -0000, Justin Erenkrantz wrote:
>   Modified:    include  apr_atomic.h
>   Log:
>   Remove Linux atomic support as this is most definitely not meant for
>   userspace applications.  We will have to come up with processor-specific
>   implementations, but we can not rely on Linux to help us here.  So,
>   remove this.

Great, thanks. There is a stale comment at line 96 which could probably
be deleted, and also this one:

--- apr_atomic.h	30 Apr 2002 23:45:27 -0000	1.25
+++ apr_atomic.h	1 May 2002 08:23:21 -0000
@@ -228,7 +228,7 @@
 #define APR_ATOMIC_NEED_CAS_DEFAULT 1
 #endif /* APR_HAS_THREADS */
 
-#endif /* !defined(WIN32) && !defined(__linux) */
+#endif /* !defined(WIN32) */
 
 #if defined(APR_ATOMIC_NEED_DEFAULT)
 #define apr_atomic_t apr_uint32_t

Re: cvs commit: apr/include apr_atomic.h

[Jeff Trawick <tr...@attglobal.net>]

Joe Orton <jo...@manyfish.co.uk> writes:

> On Tue, Apr 30, 2002 at 11:43:29PM -0000, Justin Erenkrantz wrote:
> >   Modified:    include  apr_atomic.h
> >   Log:
> >   Remove Linux atomic support as this is most definitely not meant for
> >   userspace applications.  We will have to come up with processor-specific
> >   implementations, but we can not rely on Linux to help us here.  So,
> >   remove this.
> 
> Great, thanks. There is a stale comment at line 96 which could probably
> be deleted, and also this one:

Thanks, committed!

-- 
Jeff Trawick | trawick@attglobal.net
Born in Roswell... married an alien...