[asterisk-dev] [Code Review] 3763: media formats: Fix appending of compatible formats

[opticron]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3763/
-----------------------------------------------------------

Review request for Asterisk Developers, Corey Farrell and Matt Jordan.


Repository: Asterisk


Description
-------

When ast_format_cap_append was replaced with ast_format_cap_append_from_cap, the behavior of the original function was not preserved. This change introduces ast_format_cap_append_compatible_from_cap which reproduces the original behavior of ast_format_cap_append and alters the original users of ast_format_cap_append to use ast_format_cap_append_compatible_from_cap.

The primary difference in behavior is that ast_format_cap_append_from_cap appends unconditionally from the source format capability while ast_format_cap_append checks for compatibility with ast_format_cap_iscompatible.


Diffs
-----

  team/group/media_formats-reviewed-trunk/res/res_pjsip_session.c 418441 
  team/group/media_formats-reviewed-trunk/res/res_pjsip_sdp_rtp.c 418441 
  team/group/media_formats-reviewed-trunk/main/format_cap.c 418441 
  team/group/media_formats-reviewed-trunk/include/asterisk/format_cap.h 418441 
  team/group/media_formats-reviewed-trunk/channels/chan_sip.c 418441 
  team/group/media_formats-reviewed-trunk/addons/chan_ooh323.c 418441 

Diff: https://reviewboard.asterisk.org/r/3763/diff/


Testing
-------

Ensured that calling a channel via Dial() produced the same behavior as trunk.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140713/9d2ece5b/attachment.html>