[asterisk-commits] russell: trunk r77790 - /trunk/main/manager.c

[SVN commits to the Asterisk project]

Author: russell
Date: Mon Jul 30 14:31:27 2007
New Revision: 77790

URL: http://svn.digium.com/view/asterisk?view=rev&rev=77790
Log:
Remove an XXX comment noting that it would be nice for a declaration to be
inside of a function.  (Yes, it would!)  Replace it with a note that explains
why it can't be done using the way that the AST_THREADSTORAGE macro is
currently defined.

Modified:
    trunk/main/manager.c

Modified: trunk/main/manager.c
URL: http://svn.digium.com/view/asterisk/trunk/main/manager.c?view=diff&rev=77790&r1=77789&r2=77790
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Mon Jul 30 14:31:27 2007
@@ -832,8 +832,15 @@
 	return n < 0 ? -1 : 0;
 }
 
-/* XXX see if it can be moved inside the function */
+/*!
+ * \brief thread local buffer for astman_append
+ *
+ * \note This can not be defined within the astman_append() function
+ *       because it declares a couple of functions that get used to
+ *       initialize the thread local storage key.
+ */
 AST_THREADSTORAGE(astman_append_buf);
+/*! \brief initial allocated size for the astman_append_buf */
 #define ASTMAN_APPEND_BUF_INITSIZE   256
 
 /*!