[asterisk-dev] [Code Review] Revise skinny debugging messages

wedhorn reviewboard at asterisk.org
Mon Jul 16 21:00:59 CDT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2040/
-----------------------------------------------------------

(Updated July 16, 2012, 9 p.m.)


Review request for Asterisk Developers and Matt Jordan.


Changes
-------

Changes as suggested. Think I got all the blobs.


Summary
-------

Rewrite of skinny debugging.

Debugging messages and associated controls only compiled in if configured with --enable-dev-mode

Debug messages provide more detail (including thread id) and are grouped so the user/dev can limit the type of messages displayed.

Functionally no real change to chan_skinny.

While the diff is fairly large, there are basically 3 components,
 - definition of SKINNY_DEBUG at lines 134-152
 - control functions at lines 3238-3374
 - numerous uses of SKINNY_DEBUG throughout


Diffs (updated)
-----

  trunk/channels/chan_skinny.c 369904 
  trunk/build_tools/cflags-devmode.xml 369904 

Diff: https://reviewboard.asterisk.org/r/2040/diff


Testing
-------

Been running for a while with --enable-dev-mode. Compiled and tested basic functionality without --enable-dev-mode.


Thanks,

wedhorn

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120717/88464d4c/attachment.htm>


More information about the asterisk-dev mailing list