You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by da...@apache.org on 2007/06/29 19:09:34 UTC

svn commit: r551962 - in /apr/apr/trunk: include/apr_atomic.h memory/unix/apr_pools.c

Author: davi
Date: Fri Jun 29 10:09:33 2007
New Revision: 551962

URL: http://svn.apache.org/viewvc?view=rev&rev=551962
Log:
Update apr_atomic_init documentation. PR: 42760.

Modified:
    apr/apr/trunk/include/apr_atomic.h
    apr/apr/trunk/memory/unix/apr_pools.c

Modified: apr/apr/trunk/include/apr_atomic.h
URL: http://svn.apache.org/viewvc/apr/apr/trunk/include/apr_atomic.h?view=diff&rev=551962&r1=551961&r2=551962
==============================================================================
--- apr/apr/trunk/include/apr_atomic.h (original)
+++ apr/apr/trunk/include/apr_atomic.h Fri Jun 29 10:09:33 2007
@@ -40,6 +40,9 @@
  * atomic operation's internal structures
  * @param p pool
  * @return APR_SUCCESS on successful completion
+ * @remark Programs do NOT need to call this directly. APR will call this
+ *         automatically from apr_initialize.
+ * @internal
  */
 APR_DECLARE(apr_status_t) apr_atomic_init(apr_pool_t *p);
 

Modified: apr/apr/trunk/memory/unix/apr_pools.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/memory/unix/apr_pools.c?view=diff&rev=551962&r1=551961&r2=551962
==============================================================================
--- apr/apr/trunk/memory/unix/apr_pools.c (original)
+++ apr/apr/trunk/memory/unix/apr_pools.c Fri Jun 29 10:09:33 2007
@@ -556,6 +556,9 @@
 
     /* This has to happen here because mutexes might be backed by
      * atomics.  It used to be snug and safe in apr_initialize().
+     *
+     * Warning: apr_atomic_init() must always be called, by any
+     * means possible, from apr_initialize().
      */
     if ((rv = apr_atomic_init(global_pool)) != APR_SUCCESS) {
         return rv;