[asterisk-bugs] [Asterisk 0011107]: [patch] to better debug main DSP busydetect function

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11107 
====================================================================== 
Reported By:                agx
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11107
Category:                   Core/BuildSystem
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.4.13  
SVN Branch (only for SVN checkouts, not tarball releases): N/A  
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             10-29-2007 12:20 CDT
Last Modified:              12-27-2007 12:46 CST
====================================================================== 
Summary:                    [patch] to better debug main DSP busydetect function
Description: 
In Italy there is no availability for polarityreversal switch so i've to do
a little enhanchement to the code to capture dps.c handling the busy signal
detection.

This patch adds:
- menuselect option to allow to choose the already in-place option about
the different BUSYDETECT routines already implemented;
- main/dsp.c enhanched on-deman debugging using MENUSELECT_DEBUG flag and
the common option_debug global variable;
- zap show channel ability to display wich busydetection parameters have
been configured for the zap channel;
- documentation enhancement;

Platform: Linux Debian 4.0

The patch is again 1.4.13 svn: svn info
URL: http://svn.digium.com/svn/asterisk/tags/1.4.13
Repository Root: http://svn.digium.com/svn/asterisk
Revision: 87121
Last Changed Rev: 85320
Last Changed Date: 2007-10-10 18:06:23 +0200 (Wed, 10 Oct 2007)

====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 12-27-07 12:46  
---------------------------------------------------------------------- 
1) Please don't use option_debug in combination with LOG_WARNING.  In fact,
you should just be using the ast_debug() macro call.

2) Don't use multiple ast_cli calls to print out a single line.  Instead,
separate out the format string on multiple lines and use the #ifd #elif
#endif set of preprocessor commands to select which string to use. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
12-27-07 12:46  Corydon76      Note Added: 0076054                          
======================================================================