----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3671/#review12307 -----------------------------------------------------------
/team/group/media_formats-reviewed-trunk/include/asterisk/codec.h <https://reviewboard.asterisk.org/r/3671/#comment22476> Where'd this come from? Should it just go away? /team/group/media_formats-reviewed-trunk/include/asterisk/format.h <https://reviewboard.asterisk.org/r/3671/#comment22477> This says moved... but I don't see it... is it still around? /team/group/media_formats-reviewed-trunk/main/codec.c <https://reviewboard.asterisk.org/r/3671/#comment22478> Provided we documented that it gets bumped in ref, we could. We'd also have to clean up afterwards. /team/group/media_formats-reviewed-trunk/main/format.c <https://reviewboard.asterisk.org/r/3671/#comment22479> Just curious - does this happen? /team/group/media_formats-reviewed-trunk/res/res_rtp_asterisk.c <https://reviewboard.asterisk.org/r/3671/#comment22480> SRSLY! For cases where is it doing a cleanup/copy it might be useful to add a BUGBUG for when replace exists. - Joshua Colp On June 25, 2014, 7:42 a.m., Corey Farrell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3671/ > ----------------------------------------------------------- > > (Updated June 25, 2014, 7:42 a.m.) > > > Review request for Asterisk Developers, Joshua Colp and Matt Jordan. > > > Repository: Asterisk > > > Description > ------- > > This update gives media_formats the ability to receive a call using chan_sip. > Possibly other channel drivers might work, I haven't tried them. > > * ast_format_cap_is_compatible_format needs to be checked against > AST_FORMAT_CMP_NOT_EQUAL, not zero/non-zero. All calls to > ast_format_cap_is_compatible_format were fixed. > * res_rtp_asterisk was updated by Matt Jordan, along with related changes to > codec.c, codec.h, format.c, format.c and codec_builtin.c. > * Switch ast_format_copy from function to macro to ao2_bump. This allows > REF_DEBUG to give better results. > > > Diffs > ----- > > /team/group/media_formats-reviewed-trunk/res/res_speech.c 417190 > /team/group/media_formats-reviewed-trunk/res/res_rtp_asterisk.c 417190 > /team/group/media_formats-reviewed-trunk/main/translate.c 417190 > /team/group/media_formats-reviewed-trunk/main/frame.c 417190 > /team/group/media_formats-reviewed-trunk/main/format.c 417190 > /team/group/media_formats-reviewed-trunk/main/codec_builtin.c 417190 > /team/group/media_formats-reviewed-trunk/main/codec.c 417190 > /team/group/media_formats-reviewed-trunk/main/channel.c 417190 > /team/group/media_formats-reviewed-trunk/main/bridge.c 417190 > /team/group/media_formats-reviewed-trunk/include/asterisk/format.h 417190 > /team/group/media_formats-reviewed-trunk/include/asterisk/codec.h 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_unistim.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_skinny.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_sip.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_pjsip.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_oss.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_nbs.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_motif.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_mgcp.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_jingle.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_h323.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_gtalk.c 417190 > /team/group/media_formats-reviewed-trunk/channels/chan_alsa.c 417190 > /team/group/media_formats-reviewed-trunk/addons/chan_ooh323.c 417190 > /team/group/media_formats-reviewed-trunk/addons/chan_mobile.c 417190 > > Diff: https://reviewboard.asterisk.org/r/3671/diff/ > > > Testing > ------- > > Called from Asterisk 11 to a test server with this code, I was able to hear > the 'invalid' message, everything seemed during the call. I received TONS of > ao2 frack's when stopping Asterisk. The sip.conf peer on both Asterisk > servers was setup for disallow=all / allow=ulaw. > > > Thanks, > > Corey Farrell > >
-- _____________________________________________________________________ -- 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
