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

Eliel Sardanons eliel at eliel.com.ar
Tue Jun 12 12:29:44 CDT 2007


We could reopen the bug of Dmitry Andrianov that submit the ast_debug macro
and the pbx.c patched.

On 6/12/07, Jason Parker <jparker at digium.com> wrote:
>
> ----- "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
>
> _______________________________________________
> --Bandwidth and Colocation provided by Easynews.com --
>
> asterisk-dev mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-dev
>



-- 
Eliel Sardañons
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/asterisk-dev/attachments/20070612/ed81d0eb/attachment.htm


More information about the asterisk-dev mailing list