[asterisk-dev] Janitor Project - ast_debug() conversion

Russell Bryant russell at digium.com
Tue Jun 12 20:51:06 CDT 2007


Caio Begotti wrote:
> On 12/06/2007, at 16:57, Russell Bryant wrote:
>>>> ast_debug(3, ...);
>>> Shouldn't this be ast_debug(4, ...) since ast_debug(level, ...) checks
>>> for option_debug >= level?
>>
>> Yes, you are right.  Please excuse my poor example.  :)
> 
> Ops! So we made tons of changes with the wrong level sequence? :-(
> 
> I had a few of ast_log level 3 and even 7 in some apps/*.c, and 
> according to Michael's mail the level counting should be 4 and 8, not 
> the same 3 and 7, right? I'd like to update my patches in 9957 if so...

Well, to be honest, I'm not terribly concerned.  The worst case is that some 
messages get moved up one debug level, and I'm fine with that.

-- 
Russell Bryant
Software Engineer
Digium, Inc.


More information about the asterisk-dev mailing list