[asterisk-bugs] [Asterisk 0010854]: [patch] remove extra logging from ast_add_extension2

noreply at bugs.digium.com noreply at bugs.digium.com
Sun Sep 30 18:55:02 CDT 2007


A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=10854 
====================================================================== 
Reported By:                dimas
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   10854
Category:                   Core/PBX
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 83433 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             09-30-2007 14:27 CDT
Last Modified:              09-30-2007 18:55 CDT
====================================================================== 
Summary:                    [patch] remove extra logging from ast_add_extension2
Description: 
ast_add_extension2 logs the fact it added extension twice - once with
ast_debug and second time with ast_verb. I believe there is no real need in
doubling this information.
====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 09-30-07 18:55  
---------------------------------------------------------------------- 
I'm not sure I agree.  Debug logging is only ever supposed to be turned on
when you're debugging an issue, and verbose is not meant to go to the log
-- it's meant to go to the console.

There are frequent cases where you'd want to see the verbose output, but
not the debug output, and for that reason (differing purposes), I don't
think this patch should be applied. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
09-30-07 18:55  Corydon76      Note Added: 0071218                          
======================================================================




More information about the asterisk-bugs mailing list