rtpmanagerbad: name the element children
authorMarc Leeman <marc.leeman@gmail.com>
Mon, 23 Sep 2019 08:30:40 +0000 (10:30 +0200)
committerTim-Philipp Müller <tim@centricular.com>
Sun, 17 Nov 2019 16:00:19 +0000 (16:00 +0000)
As discussed with RIST, it is best to name the children of the elements
since these are now created at the element initialisation.

gst/rtp/gstrtpsink.c
gst/rtp/gstrtpsrc.c

index 0c0b7df..cc4b335 100644 (file)
@@ -546,7 +546,7 @@ gst_rtp_sink_init (GstRtpSink * self)
    *                                   | rtpbin |
    * udpsrc     -> [recv_rtcp_sink_%u]  --------  [send_rtcp_src_%u] -> * udpsink
    */
-  self->rtpbin = gst_element_factory_make ("rtpbin", NULL);
+  self->rtpbin = gst_element_factory_make ("rtpbin", "rtp_send_rtpbin0");
   if (self->rtpbin == NULL) {
     missing_plugin = "rtpmanager";
     goto missing_plugin;
@@ -566,31 +566,31 @@ gst_rtp_sink_init (GstRtpSink * self)
   gst_bin_set_suppressed_flags (GST_BIN (self),
       GST_ELEMENT_FLAG_SOURCE | GST_ELEMENT_FLAG_SINK);
 
-  self->funnel_rtp = gst_element_factory_make ("funnel", NULL);
+  self->funnel_rtp = gst_element_factory_make ("funnel", "rtp_rtp_funnel0");
   if (self->funnel_rtp == NULL) {
     missing_plugin = "funnel";
     goto missing_plugin;
   }
 
-  self->funnel_rtcp = gst_element_factory_make ("funnel", NULL);
+  self->funnel_rtcp = gst_element_factory_make ("funnel", "rtp_rtcp_funnel0");
   if (self->funnel_rtcp == NULL) {
     missing_plugin = "funnel";
     goto missing_plugin;
   }
 
-  self->rtp_sink = gst_element_factory_make ("udpsink", NULL);
+  self->rtp_sink = gst_element_factory_make ("udpsink", "rtp_rtp_udpsink0");
   if (self->rtp_sink == NULL) {
     missing_plugin = "udp";
     goto missing_plugin;
   }
 
-  self->rtcp_src = gst_element_factory_make ("udpsrc", NULL);
+  self->rtcp_src = gst_element_factory_make ("udpsrc", "rtp_rtcp_udpsrc0");
   if (self->rtcp_src == NULL) {
     missing_plugin = "udp";
     goto missing_plugin;
   }
 
-  self->rtcp_sink = gst_element_factory_make ("udpsink", NULL);
+  self->rtcp_sink = gst_element_factory_make ("udpsink", "rtp_rtcp_udpsink0");
   if (self->rtcp_sink == NULL) {
     missing_plugin = "udp";
     goto missing_plugin;
index 800b67d..97461a7 100644 (file)
@@ -663,7 +663,7 @@ gst_rtp_src_init (GstRtpSrc * self)
    * be added later.
    */
 
-  self->rtpbin = gst_element_factory_make ("rtpbin", NULL);
+  self->rtpbin = gst_element_factory_make ("rtpbin", "rtp_recv_rtpbin0");
   if (self->rtpbin == NULL) {
     missing_plugin = "rtpmanager";
     goto missing_plugin;
@@ -683,19 +683,20 @@ gst_rtp_src_init (GstRtpSrc * self)
   g_signal_connect (self->rtpbin, "on-ssrc-collision",
       G_CALLBACK (gst_rtp_src_rtpbin_on_ssrc_collision_cb), self);
 
-  self->rtp_src = gst_element_factory_make ("udpsrc", NULL);
+  self->rtp_src = gst_element_factory_make ("udpsrc", "rtp_rtp_udpsrc0");
   if (self->rtp_src == NULL) {
     missing_plugin = "udp";
     goto missing_plugin;
   }
 
-  self->rtcp_src = gst_element_factory_make ("udpsrc", NULL);
+  self->rtcp_src = gst_element_factory_make ("udpsrc", "rtp_rtcp_udpsrc0");
   if (self->rtcp_src == NULL) {
     missing_plugin = "udp";
     goto missing_plugin;
   }
 
-  self->rtcp_sink = gst_element_factory_make ("dynudpsink", NULL);
+  self->rtcp_sink =
+      gst_element_factory_make ("dynudpsink", "rtp_rtcp_dynudpsink0");
   if (self->rtcp_sink == NULL) {
     missing_plugin = "udp";
     goto missing_plugin;