Changeset 4577 for pjproject/trunk/pjmedia/src/pjmedia/sdp_neg.c

Timestamp:

Aug 1, 2013 4:18:15 AM (10 years ago)

Author:

ming

Message:

Closed #1692: Allow multiple codecs in SDP answer
By default, the setting is disabled, to change it during run-time, use the function pjmedia_sdp_neg_set_allow_multiple_codecs().

File:

• pjproject/trunk/pjmedia/src/pjmedia/sdp_neg.c (modified) (9 diffs)

pjproject/trunk/pjmedia/src/pjmedia/sdp_neg.c

@@ -34 +34 @@
     pjmedia_sdp_neg_state state;            /**< Negotiator state.           */
     pj_bool_t             prefer_remote_codec_order;
+    pj_bool_t             answer_with_multiple_codecs;
     pj_bool_t             has_remote_answer;
     pj_bool_t             answer_was_remote;
@@ -115 +116 @@
     neg->state = PJMEDIA_SDP_NEG_STATE_LOCAL_OFFER;
     neg->prefer_remote_codec_order = PJMEDIA_SDP_NEG_PREFER_REMOTE_CODEC_ORDER;
+    neg->answer_with_multiple_codecs = PJMEDIA_SDP_NEG_ANSWER_MULTIPLE_CODECS;
     neg->initial_sdp = pjmedia_sdp_session_clone(pool, local);
     neg->neg_local_sdp = pjmedia_sdp_session_clone(pool, local);
@@ -179 +181 @@
     PJ_ASSERT_RETURN(neg, PJ_EINVAL);
     neg->prefer_remote_codec_order = prefer_remote;
+    return PJ_SUCCESS;
+}
+
+
+/*
+ * Set multiple codec answering.
+ */
+PJ_DEF(pj_status_t) pjmedia_sdp_neg_set_answer_multiple_codecs(
+                        pjmedia_sdp_neg *neg,
+                        pj_bool_t answer_multiple)
+{
+    PJ_ASSERT_RETURN(neg, PJ_EINVAL);
+    neg->answer_with_multiple_codecs = answer_multiple;
     return PJ_SUCCESS;
 }
@@ -1011 +1026 @@
 static pj_status_t match_offer(pj_pool_t *pool,
                                pj_bool_t prefer_remote_codec_order,
+                               pj_bool_t answer_with_multiple_codecs,
                                const pjmedia_sdp_media *offer,
                                const pjmedia_sdp_media *preanswer,
@@ -1076 +1092 @@
                 master_has_codec = 1;
 
-                /* We just need to select one codec. 
+                /* We just need to select one codec if not allowing multiple.
                  * Continue if we have selected matching codec for previous 
                  * payload.
                  */
-                if (found_matching_codec)
+                if (!answer_with_multiple_codecs && found_matching_codec)
                     continue;
 
@@ -1120 +1136 @@
                 } else {
                     master_has_codec = 1;
-                    if (found_matching_codec)
+                    if (!answer_with_multiple_codecs && found_matching_codec)
                         continue;
                     is_codec = 1;
@@ -1278 +1294 @@
 static pj_status_t create_answer( pj_pool_t *pool,
                                   pj_bool_t prefer_remote_codec_order,
+                                  pj_bool_t answer_with_multiple_codecs,
                                   const pjmedia_sdp_session *initial,
                                   const pjmedia_sdp_session *offer,
@@ -1327 +1344 @@
 
                 /* See if it has matching codec. */
-                status2 = match_offer(pool, prefer_remote_codec_order, 
+                status2 = match_offer(pool, prefer_remote_codec_order,
+                                      answer_with_multiple_codecs,
                                       om, im, initial, &am);
                 if (status2 == PJ_SUCCESS) {
@@ -1425 +1443 @@
         pjmedia_sdp_session *answer = NULL;
 
-        status = create_answer(pool, neg->prefer_remote_codec_order, 
+        status = create_answer(pool, neg->prefer_remote_codec_order,
+                               neg->answer_with_multiple_codecs,
                                neg->neg_local_sdp, neg->neg_remote_sdp,
                                &answer);