Author: rmudgett
Date: Fri Aug 16 11:22:26 2013
New Revision: 396849

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=396849
Log:
utils.h: Minor formatting tweaks.

Modified:
    trunk/include/asterisk/utils.h

Modified: trunk/include/asterisk/utils.h
URL: 
http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/utils.h?view=diff&rev=396849&r1=396848&r2=396849
==============================================================================
--- trunk/include/asterisk/utils.h (original)
+++ trunk/include/asterisk/utils.h Fri Aug 16 11:22:26 2013
@@ -484,23 +484,26 @@
 
 #ifndef __AST_DEBUG_MALLOC
 
-/* This buffer is in static memory. We never intend to read it,
+/*
+ * This buffer is in static memory. We never intend to read it,
  * nor do we care about multiple threads writing to it at the
  * same time. We only want to know if we're recursing too deep
- * already. 60 entries should be more than enough. Function call
- * depth rarely exceeds 20 or so. */
+ * already. 60 entries should be more than enough.  Function
+ * call depth rarely exceeds 20 or so.
+ */
 #define _AST_MEM_BACKTRACE_BUFLEN 60
 extern void *_ast_mem_backtrace_buffer[_AST_MEM_BACKTRACE_BUFLEN];
 
+/*
+ * Ok, this sucks. But if we're already out of mem, we don't
+ * want the logger to create infinite recursion (and a crash).
+ */
 #define MALLOC_FAILURE_MSG \
-       { \
-               /* Ok, this sucks. But if we're already out of mem, we don't \
-                * want the logger to create infinite recursion (and a crash). 
*/ \
-               if (backtrace(_ast_mem_backtrace_buffer, 
_AST_MEM_BACKTRACE_BUFLEN) < \
-                               _AST_MEM_BACKTRACE_BUFLEN) { \
+       do { \
+               if (backtrace(_ast_mem_backtrace_buffer, 
_AST_MEM_BACKTRACE_BUFLEN) < _AST_MEM_BACKTRACE_BUFLEN) { \
                        ast_log(LOG_ERROR, "Memory Allocation Failure in 
function %s at line %d of %s\n", func, lineno, file); \
                } \
-       }
+       } while (0)
 
 /*!
  * \brief A wrapper for malloc()
@@ -518,8 +521,9 @@
 {
        void *p;
 
-       if (!(p = malloc(len)))
+       if (!(p = malloc(len))) {
                MALLOC_FAILURE_MSG;
+       }
 
        return p;
 }
@@ -541,8 +545,9 @@
 {
        void *p;
 
-       if (!(p = calloc(num, len)))
+       if (!(p = calloc(num, len))) {
                MALLOC_FAILURE_MSG;
+       }
 
        return p;
 }
@@ -577,8 +582,9 @@
 {
        void *newp;
 
-       if (!(newp = realloc(p, len)))
+       if (!(newp = realloc(p, len))) {
                MALLOC_FAILURE_MSG;
+       }
 
        return newp;
 }
@@ -605,8 +611,9 @@
        char *newstr = NULL;
 
        if (str) {
-               if (!(newstr = strdup(str)))
+               if (!(newstr = strdup(str))) {
                        MALLOC_FAILURE_MSG;
+               }
        }
 
        return newstr;
@@ -634,8 +641,9 @@
        char *newstr = NULL;
 
        if (str) {
-               if (!(newstr = strndup(str, len)))
+               if (!(newstr = strndup(str, len))) {
                        MALLOC_FAILURE_MSG;
+               }
        }
 
        return newstr;
@@ -673,8 +681,9 @@
 {
        int res;
 
-       if ((res = vasprintf(ret, fmt, ap)) == -1)
+       if ((res = vasprintf(ret, fmt, ap)) == -1) {
                MALLOC_FAILURE_MSG;
+       }
 
        return res;
 }


--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

svn-commits mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/svn-commits

Reply via email to