<div dir="ltr">Yeah,<div><br></div><div style>I agree something like that should be sufficient?</div><div style><br></div><div style>Regards,</div><div style>Nitesh Bansal</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Oct 21, 2013 at 10:17 PM, Scott Griepentrog <span dir="ltr"><<a href="mailto:sgriepentrog@digium.com" target="_blank">sgriepentrog@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_default" style="color:rgb(102,0,0)">If it wasn't already there, or somebody doesn't beat me too it, I was planning on doing a simple check of the valgrind output - not to parse it but just to detect it as having reported something amiss and then flag the test as failed (much in the same way that a core dump detection will).</div>
<div class="gmail_default" style="color:rgb(102,0,0)"></div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Mon, Oct 21, 2013 at 3:13 PM, nitesh bansal <span dir="ltr"><<a href="mailto:nitesh.bansal@gmail.com" target="_blank">nitesh.bansal@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>I have tracked it down in SVN and i would be happy to continue working on it. Paul, i have a question for you, the time when you had valgrind working for you, were you</div>
<div>
just running asterisk under valgrind or did you write something to parse valgrind output?</div><div><br></div><div>Regards,</div><div>Nitesh Bansal</div></div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">
On Mon, Oct 21, 2013 at 8:09 PM, Paul Belanger <span dir="ltr"><<a href="mailto:paul.belanger@polybeacon.com" target="_blank">paul.belanger@polybeacon.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>On <a href="tel:13-10-21%2001" value="+13102101" target="_blank">13-10-21 01</a>:41 PM, Scott Griepentrog wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Sounds like a job for Scott...<br>
<br>
I for one would feel a lot happier running patches through valgrind testing<br>
before posting it up for review.<br>
<br>
I understand the need to keep it as cmd line option though so it's not run<br>
by default.<br>
<br>
</blockquote></div>
At some point in time I actually had the testsuite working with valgrind, but that was a while ago. I know Matt added support for gdb, so I don't imagine it being to hard.<span><font color="#888888"><br>
<br>
-- <br>
Paul Belanger | PolyBeacon, Inc.<br>
Jabber: <a href="mailto:paul.belanger@polybeacon.com" target="_blank">paul.belanger@polybeacon.com</a> | IRC: pabelanger (Freenode)<br>
Github: <a href="https://github.com/pabelanger" target="_blank">https://github.com/pabelanger</a> | Twitter: <a href="https://twitter.com/pabelanger" target="_blank">https://twitter.com/pabelanger</a></font></span><div>
<div><br>
<br>
-- <br>
______________________________<u></u>______________________________<u></u>_________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.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" target="_blank">http://lists.digium.com/<u></u>mailman/listinfo/asterisk-dev</a><br>
</div></div></blockquote></div><br></div>
</div></div><br>--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.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" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br><br clear="all"><div><br></div></div></div><div class="im">
-- <br><div dir="ltr">
<img alt="Digium logo" width="288" height="50" style="font-size:12px;font-family:Arial,Helvetica,sans-serif"><br style="font-size:12px;font-family:Arial,Helvetica,sans-serif">
<strong style="font-size:12px;font-family:Arial,Helvetica,sans-serif">Scott Griepentrog</strong><br style="font-size:12px;font-family:Arial,Helvetica,sans-serif"><span style="font-size:12px;font-family:Arial,Helvetica,sans-serif">Digium, Inc · Software Developer</span><br style="font-size:12px;font-family:Arial,Helvetica,sans-serif">
<span style="font-size:12px;font-family:Arial,Helvetica,sans-serif">445 Jan Davis Drive NW · Huntsville, AL 35806 · US</span><br style="font-size:12px;font-family:Arial,Helvetica,sans-serif">
<span style="font-size:12px;font-family:Arial,Helvetica,sans-serif">direct/fax: <a href="tel:%2B1%20256%20428%206239" value="+12564286239" target="_blank">+1 256 428 6239</a> · mobile: <a href="tel:%2B1%20317%20507%204029" value="+13175074029" target="_blank">+1 317 507 4029</a></span><br style="font-size:12px;font-family:Arial,Helvetica,sans-serif">
<span style="font-size:12px;font-family:Arial,Helvetica,sans-serif">Check us out at: </span><a href="http://www.digium.com" style="font-family:Arial,Helvetica,sans-serif;font-size:12px" target="_blank">http://digium.com</a><span style="font-size:12px;font-family:Arial,Helvetica,sans-serif"> · </span><a href="http://www.asterisk.org" style="font-family:Arial,Helvetica,sans-serif;font-size:12px" target="_blank">http://asterisk.org</a><br>
</div>
</div></div>
<br>--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.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" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br></div>