----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3919/#review13107 -----------------------------------------------------------
Ship it! Ship It! - rmudgett On Aug. 18, 2014, 1:47 p.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3919/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2014, 1:47 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > This one's been bugging me forever... > > When you call the CONFIG dialplan function with the name of a variable that > doesn't exist in the target context you get an ERROR. This does nothing but > clutter up the logs with messages that may be perfectly acceptable. Just > because a variable wasn't in the context doesn't mean it's an error. Maybe > it's optional or just needs to be defaulted or ignored. > > This patch changes the log level from ERROR to DEBUG. If a dialplan > developer wants to debug their dialplan they still can. > > > Diffs > ----- > > branches/12/funcs/func_config.c 421326 > > Diff: https://reviewboard.asterisk.org/r/3919/diff/ > > > Testing > ------- > > > Thanks, > > George Joseph > >
-- _____________________________________________________________________ -- 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