----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4076/#review13515 -----------------------------------------------------------
Ship it! Ship It! - rmudgett On Oct. 14, 2014, 1:02 p.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4076/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2014, 1:02 p.m.) > > > Review request for Asterisk Developers and rmudgett. > > > Repository: Asterisk > > > Description > ------- > > Based on feedback from Richard, I created an accessor for > res_phoneprov/ast_phoneprov_std_variable_lookup and added load priority to > AST_MODULE_INFO. > > Richard, can you test this? I don't see any difference in behavior before > and after the change. res_phoneprov has AST_MODFLAG_GLOBAL_SYMBOLS set so it > should have always been loading before res_pjsip_phoneprov_provider which > doesn't have that flag set, regardless of load priority. You can't even > modify that behavior in modules.conf. > > > Diffs > ----- > > branches/12/res/res_pjsip_phoneprov_provider.c 425404 > branches/12/res/res_phoneprov.c 425404 > branches/12/include/asterisk/phoneprov.h 425404 > > Diff: https://reviewboard.asterisk.org/r/4076/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
