We could reopen the bug of <span class="q">Dmitry Andrianov that submit the ast_debug macro and the pbx.c patched.</span><br><br><div><span class="gmail_quote">On 6/12/07, <b class="gmail_sendername">Jason Parker</b> <<a href="mailto:jparker@digium.com">
jparker@digium.com</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">----- "Michiel van Baak" <<a href="mailto:michiel@vanbaak.info">
michiel@vanbaak.info</a>> wrote:<br>> On 18:52, Tue 12 Jun 07, Michiel van Baak wrote:<br>> > On 11:18, Tue 12 Jun 07, Russell Bryant wrote:<br>> > > Dmitry Andrianov has created a new macro, ast_debug() which
<br>> simplifies<br>> > > debug logging. He has also converted main/pbx.c to use it. Now,<br>> the<br>> > > rest of the code base needs to get converted. If you are<br>> interested in<br>> > > helping, volunteer to take a section of the code in this thread so
<br>> you<br>> > > don't duplicate efforts. :)<br>> > ><br>> > > Currently, you will see code like:<br>> > ><br>> > > if (option_debug)<br>> > > ast_log(LOG_DEBUG, ...);
<br>> > ><br>> > > if (option debug > 3)<br>> > > ast_log(LOG_DEBUG, ...);<br>> > ><br>> > > Now, that would look like<br>> > ><br>> > > ast_debug(1, ...);
<br>> > ><br>> > > ast_debug(3, ...);<br>> ><br>> > chan_skinny is now being worked on by me :)<br>><br>> Added this to my patch 9762.<br>> If you want I can send a standalone patch for the ast_debug
<br>> stuff (only 3 places)<br>> --<br>><br>> Michiel van Baak<br>> <a href="mailto:michiel@vanbaak.eu">michiel@vanbaak.eu</a><br>> <a href="http://michiel.vanbaak.eu">http://michiel.vanbaak.eu</a><br>> GnuPG key:
<br>> <a href="http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x71C946BD">http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x71C946BD</a><br>><br>> "Why is it drug addicts and computer afficionados are both called
<br>> users?"<br><br>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.<br><br>--<br>Jason Parker<br>Digium<br><br>_______________________________________________
<br>--Bandwidth and Colocation provided by <a href="http://Easynews.com">Easynews.com</a> --<br><br>asterisk-dev mailing list<br>To UNSUBSCRIBE or update options visit:<br> <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev">
http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br><br clear="all"><br>-- <br>Eliel Sardaņons