----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3756/#review12587 -----------------------------------------------------------
It's worth noting this updated script took 40 seconds to process the same file that took 30 seconds in the unmodified refcounter.py. Maybe a python wizard can suggest something to speed this up. If not the slower processing is worth not having false reports. - Corey Farrell On July 12, 2014, 2:13 a.m., Corey Farrell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3756/ > ----------------------------------------------------------- > > (Updated July 12, 2014, 2:13 a.m.) > > > Review request for Asterisk Developers and Matt Jordan. > > > Repository: Asterisk > > > Description > ------- > > This incorporates the ideas from r3572, with an improvement. REF_DEBUG is > still effective when ao2_ref(obj, -1) is run on an object that is already > destroyed. > > In addition refcounter.py is updated to use delta's to track the current > reference count. This works around a race condition where **destroyed** > could be printed before the unref from 2. > > > Diffs > ----- > > /branches/1.8/main/astobj2.c 417500 > /branches/1.8/contrib/scripts/refcounter.py 417486 > > Diff: https://reviewboard.asterisk.org/r/3756/diff/ > > > Testing > ------- > > Verified that all false reported leaks are no longer. > > One thing I noticed, the updated refcounter.py took 40 seconds to process a > file when the same file was processed in 30 seconds before the change. > > > Thanks, > > Corey Farrell > >
-- _____________________________________________________________________ -- 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
