----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3536/#review11883 -----------------------------------------------------------
Ship it! Ship It! - Jonathan Rose On May 13, 2014, 9:27 a.m., wdoekes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3536/ > ----------------------------------------------------------- > > (Updated May 13, 2014, 9:27 a.m.) > > > Review request for Asterisk Developers and opticron. > > > Repository: Asterisk > > > Description > ------- > > A subset of https://reviewboard.asterisk.org/r/2742/ > > Parts of that patch have been committed already, others have been fixed in a > different way, and others will be moved into a new review. > > This patch does: > > (5) free()'s were encountered that should be ast_free()'s. > > (6) Added comment about LOGIC by tilghman:^H^H^H^H^H^H^H^H^H > Reverted the code to the original, because the old comment seems wrong. > > (8) Reorder a bit of code so a lock is paired with a single unlock. > > > Diffs > ----- > > /branches/1.8/res/res_musiconhold.c 413786 > > Diff: https://reviewboard.asterisk.org/r/3536/diff/ > > > Testing > ------- > > It compiles. > > > Thanks, > > wdoekes > >
-- _____________________________________________________________________ -- 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
