[asterisk-dev] [svn-commits] qwell: branch 1.4 r160764 - /branches/1.4/channels/chan_agent.c

[Miguel Molina]

I didn't know of the ast_verb() wrapper for 1.6, which is a good thing. 
Thanks for the follow-up.

Regards,

-- 
Ing. Miguel Molina
Grupo de Tecnología
Millenium Phone Center
PBX: (+57 1)6500800 ext. 1201
Fax: (+57 1)6500816
Móvil: (+57)3138873587 



Jason Parker escribió:
> Russell Bryant wrote:
>   
>> Julian Lyndon-Smith wrote:
>>     
>>> Not being a C programmer, but a lazy one, why doesn't the ast_verbose 
>>> function call check the current level and only output the message if the 
>>> current level is eq to or greater than the _prefix ?
>>>
>>> This would remove the need to have
>>>
>>> if (option_verbose > 2)
>>> 		ast_verbose(VERBOSE_PREFIX_3 ...
>>>
>>> and only have
>>>
>>> ast_verbose(VERBOSE_PREFIX_3 ...
>>>
>>> I know speed must be a consideration, but isn't the difference in order 
>>> of microseconds ?
>>>
>>> /me dons the heavy-duty flamesuit and awaits the derision of the -dev 
>>> list ...
>>>       
>> That's why we have ast_verb() in trunk, which is just such a wrapper.  :)
>>
>>     
>
> And why this commit was blocked in trunk, since that message is using the
> wrapper there.
>
> _______________________________________________
> --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
>   
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/asterisk-dev/attachments/20081205/111e78f2/attachment.htm