[asterisk-commits] rmudgett: branch 12 r398498 - /branches/12/main/astobj2.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Thu Sep  5 16:46:46 2013
New Revision: 398498

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=398498
Log:
astobj2: Only define ao2_bt() once.

* Make ao2_bt() not use single char variable names.

* Fix ao2_bt() formatting.

Modified:
    branches/12/main/astobj2.c

Modified: branches/12/main/astobj2.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/main/astobj2.c?view=diff&rev=398498&r1=398497&r2=398498
==============================================================================
--- branches/12/main/astobj2.c (original)
+++ branches/12/main/astobj2.c Thu Sep  5 16:46:46 2013
@@ -114,27 +114,28 @@
 static struct ao2_stats ao2;
 #endif
 
-#ifndef HAVE_BKTR	/* backtrace support */
-void ao2_bt(void) {}
-#else
+#ifdef HAVE_BKTR
 #include <execinfo.h>    /* for backtrace */
+#endif
 
 void ao2_bt(void)
 {
-	int c, i;
+#ifdef HAVE_BKTR
+	int depth;
+	int idx;
 #define N1	20
 	void *addresses[N1];
 	char **strings;
 
-	c = backtrace(addresses, N1);
-	strings = ast_bt_get_symbols(addresses,c);
-	ast_verbose("backtrace returned: %d\n", c);
-	for(i = 0; i < c; i++) {
-		ast_verbose("%d: %p %s\n", i, addresses[i], strings[i]);
+	depth = backtrace(addresses, N1);
+	strings = ast_bt_get_symbols(addresses, depth);
+	ast_verbose("backtrace returned: %d\n", depth);
+	for (idx = 0; idx < depth; ++idx) {
+		ast_verbose("%d: %p %s\n", idx, addresses[idx], strings[idx]);
 	}
 	ast_std_free(strings);
-}
 #endif
+}
 
 #define INTERNAL_OBJ_MUTEX(user_data) \
 	((struct astobj2_lock *) (((char *) (user_data)) - sizeof(struct astobj2_lock)))