----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3923/#review13179 -----------------------------------------------------------
Ship it! branches/1.8/main/callerid.c <https://reviewboard.asterisk.org/r/3923/#comment23649> Probably should have a comment stating what case this quote stripping is being done to handle. - rmudgett On Aug. 25, 2014, 8:08 p.m., opticron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3923/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2014, 8:08 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > This allows the callerid parsing function to handle malformed input strings > and strings containing escaped and unescaped double quotes. This also adds a > unittest to cover many of the cases where the parsing algorithm previously > failed. > > > Diffs > ----- > > branches/1.8/tests/test_utils.c 421534 > branches/1.8/tests/test_callerid.c PRE-CREATION > branches/1.8/main/utils.c 421534 > branches/1.8/main/callerid.c 421534 > branches/1.8/include/asterisk/utils.h 421534 > branches/1.8/channels/chan_sip.c 421534 > > Diff: https://reviewboard.asterisk.org/r/3923/diff/ > > > Testing > ------- > > Ran the unittest. > > > Thanks, > > opticron > >
-- _____________________________________________________________________ -- 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