rtp: Remove caps restrictions from RTP depayloader sink caps
authorOlivier Crête <olivier.crete@collabora.com>
Thu, 6 Mar 2014 17:06:43 +0000 (12:06 -0500)
committerOlivier Crête <olivier.crete@collabora.com>
Thu, 6 Mar 2014 17:06:43 +0000 (12:06 -0500)
Remove caps restrictions that correspond to the default and are not
required in SDP. With the new usage of having pads require a subset
of the caps, they will make the negotiation fail.

gst/rtp/gstrtpamrdepay.c
gst/rtp/gstrtpilbcdepay.c
gst/rtp/gstrtpsirendepay.c
gst/rtp/gstrtpspeexdepay.c

index e1208bf..4c6b722 100644 (file)
@@ -78,7 +78,8 @@ static GstStaticPadTemplate gst_rtp_amr_depay_sink_template =
         "media = (string) \"audio\", "
         "clock-rate = (int) 8000, "
         "encoding-name = (string) \"AMR\", "
-        "encoding-params = (string) \"1\", "
+        /* This is the default, so the peer doesn't have to specify it
+         * "encoding-params = (string) \"1\", " */
         /* NOTE that all values must be strings in orde to be able to do SDP <->
          * GstCaps mapping. */
         "octet-align = (string) \"1\";"
@@ -97,7 +98,8 @@ static GstStaticPadTemplate gst_rtp_amr_depay_sink_template =
         "media = (string) \"audio\", "
         "clock-rate = (int) 16000, "
         "encoding-name = (string) \"AMR-WB\", "
-        "encoding-params = (string) \"1\", "
+        /* This is the default, so the peer doesn't have to specify it
+         * "encoding-params = (string) \"1\", " */
         /* NOTE that all values must be strings in orde to be able to do SDP <->
          * GstCaps mapping. */
         "octet-align = (string) \"1\";"
index 8b4285e..5c7dc25 100644 (file)
@@ -49,8 +49,8 @@ GST_STATIC_PAD_TEMPLATE ("sink",
     GST_STATIC_CAPS ("application/x-rtp, "
         "media = (string) \"audio\", "
         "clock-rate = (int) 8000, "
-        "encoding-name = (string) \"ILBC\", "
-        "mode = (string) { \"20\", \"30\" }")
+        "encoding-name = (string) \"ILBC\"")
+     /* "mode = (string) { \"20\", \"30\" }" */
     );
 
 static GstStaticPadTemplate gst_rtp_ilbc_depay_src_template =
index 9d1e1ef..1bbbda7 100644 (file)
@@ -35,7 +35,9 @@ GST_STATIC_PAD_TEMPLATE ("sink",
     GST_STATIC_CAPS ("application/x-rtp, "
         "media = (string) \"audio\", "
         "clock-rate = (int) 16000, "
-        "encoding-name = (string) \"SIREN\", " "dct-length = (int) 320")
+        "encoding-name = (string) \"SIREN\"")
+    /* This is the default, so the peer doesn't have to specify it */
+    /*  " "dct-length = (int) 320") */
     );
 
 static GstStaticPadTemplate gst_rtp_siren_depay_src_template =
index 5a61fda..34aef39 100644 (file)
@@ -46,7 +46,7 @@ GST_STATIC_PAD_TEMPLATE ("sink",
     GST_STATIC_CAPS ("application/x-rtp, "
         "media = (string) \"audio\", "
         "clock-rate =  (int) [6000, 48000], "
-        "encoding-name = (string) \"SPEEX\"")
+        "encoding-name = (string) \"SPEEX\"")
     /*  "encoding-params = (string) \"1\"" */
     );