> On Dec. 10, 2013, 6:51 p.m., Mark Michelson wrote: > > /branches/12/channels/pjsip/dialplan_functions.c, lines 650-652 > > <https://reviewboard.asterisk.org/r/3038/diff/2/?file=49287#file49287line650> > > > > It may just be reviewboard, but the second and third lines of this are > > indented waaaaaaay to the right. > > > > I suspect that you used four-space tabs and lined up the arguments just > > past the opening paren on the preceding line, a la PEP-8 recommendations. > > However, such practice doesn't work well when dealing with tabs instead of > > spaces for indention. > > > > This applies to some other function invocations used in this function.
Yeah... I'll just single line these. - Matt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3038/#review10365 ----------------------------------------------------------- On Dec. 10, 2013, 3:51 a.m., Matt Jordan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3038/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2013, 3:51 a.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > This patch adds CHANNEL function support to chan_pjsip. Since things were > getting a bit large, all dialplan functions that were in chan_pjsip have also > been moved into their own file (dialplan_functions). > > Information that can be retrieved: > * rtp,type,[media_type] - Get RTP information, including media > source/destination addresses, whether or not the media is secure, etc. > * rtcp,statistic,[media_type] - Get RTCP statistic information > * endpoint - Get the name of the endpoint associated with this channel. Use > PJSIP_ENDPOINT to get more info. > * pjsip,type - Get signalling related information, including > source/destination addresses, URIs in the INVITE request, whether or not the > signalling is using a secure transport, etc. > > Note that after this patch is committed, we should go back through the > CHANNEL function documentation and move all of the channel technology > specific information into <info/> blocks, so that the documentation is > co-located with the channel drivers themselves. > > > Diffs > ----- > > /branches/12/res/res_pjsip_t38.c 403470 > /branches/12/include/asterisk/res_pjsip_session.h 403470 > /branches/12/funcs/func_channel.c 403470 > /branches/12/channels/pjsip/include/dialplan_functions.h PRE-CREATION > /branches/12/channels/pjsip/include/chan_pjsip.h PRE-CREATION > /branches/12/channels/pjsip/dialplan_functions.c PRE-CREATION > /branches/12/channels/chan_pjsip.c 403470 > /branches/12/channels/Makefile 403470 > > Diff: https://reviewboard.asterisk.org/r/3038/diff/ > > > Testing > ------- > > See https://reviewboard.asterisk.org/r/3037 > > > Thanks, > > Matt Jordan > >
-- _____________________________________________________________________ -- 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
