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

[Jason Parker]

----- "Michiel van Baak" <michiel at vanbaak.info> wrote:
> On 18:52, Tue 12 Jun 07, Michiel van Baak wrote:
> > On 11:18, Tue 12 Jun 07, Russell Bryant wrote:
> > > Dmitry Andrianov has created a new macro, ast_debug() which
> simplifies 
> > > debug logging.  He has also converted main/pbx.c to use it.  Now,
> the 
> > > rest of the code base needs to get converted.  If you are
> interested in 
> > > helping, volunteer to take a section of the code in this thread so
> you 
> > > don't duplicate efforts.  :)
> > > 
> > > Currently, you will see code like:
> > > 
> > > if (option_debug)
> > >    ast_log(LOG_DEBUG, ...);
> > > 
> > > if (option debug > 3)
> > >    ast_log(LOG_DEBUG, ...);
> > > 
> > > Now, that would look like
> > > 
> > > ast_debug(1, ...);
> > > 
> > > ast_debug(3, ...);
> > 
> > chan_skinny is now being worked on by me :)
> 
> Added this to my patch 9762.
> If you want I can send a standalone patch for the ast_debug
> stuff (only 3 places)
> -- 
> 
> Michiel van Baak
> michiel at vanbaak.eu
> http://michiel.vanbaak.eu
> GnuPG key:
> http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x71C946BD
> 
> "Why is it drug addicts and computer afficionados are both called
> users?"

I would prefer it if you would open a new report with the ast_debug patch or if somebody here opens one, you can post yours there.

-- 
Jason Parker
Digium