res/res_pjsip_sdp_rtp: Remove left over reference to override_prefs
authorMatthew Jordan <mjordan@digium.com>
Sun, 19 Oct 2014 00:56:43 +0000 (00:56 +0000)
committerMatthew Jordan <mjordan@digium.com>
Sun, 19 Oct 2014 00:56:43 +0000 (00:56 +0000)
commitb263c8bdae3c3e40cc8d7d103af47f28ee1a9cec
treec161ade3ac63f7e1ca0ef49e8b6c91b328ae8b46
parentb8f687f27c06cb398270f6b4db9e38985e11563f
res/res_pjsip_sdp_rtp: Remove left over reference to override_prefs

The usage of the local override_prefs variable in create_outgoing_sdp_stream
was previously to track an override format preference set by PJSIP_MEDIA_OFFER.
Now, however, that function simply sets the joint capabilities structure,
session->req_caps. During the media format rework, the override_prefs was
instead used to check if there were any formats in session->req_caps.

However, this usage isn't useful in create_outgoing_sdp_stream.
session->req_caps contains the negotiated formats for *all* streams, not just
the current one being created. Thus, so long as any stream of any type has
provided a format, override_prefs will be non-zero. Hence, its usage in
checking whether or not we should look at the formats on the endpoint or
the joint capabilities is generally useless.

There's only two things useful to check:
(1) Does the endpoint have a format for the media type?
(2) Did we negotiate a format for the media type?

If either of those is a 'no', then we must kill the media stream.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@425924 65c4cc65-6c06-0410-ace0-fbb531ad65f3
res/res_pjsip_sdp_rtp.c