SVN commits to the Asterisk project wrote:

> URL: http://svn.digium.com/view/asterisk?view=rev&rev=77810
> Log:
> Discovered in experiments on core files: if you wrap the lock and unlock 
> calls with sip_pvt_lock and sip_pvt_unlock, you lose the tracing info you 
> would normally get via DETECT_DEADLOCKS; so I turn these two functions into 
> macros when DETECT_DEADLOCKS is called. This way, you get meaningful stuff in 
> the file and func slots in the lock_info struct.

Two comments:

1) It was DEBUG_THREADS, not DETECT_DEADLOCKS, that was the real issue here.

2) I've removed this entirely and switched to macros in all cases; there
is no need for the inline function version, as any debugging code that
is needed can be put into the macros using the 'do { } while(0)' trick.

-- 
Kevin P. Fleming
Director of Software Technologies
Digium, Inc. - "The Genuine Asterisk Experience" (TM)

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

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

Reply via email to