rtpbin: Cleanup dead code
authorNicolas Dufresne <nicolas.dufresne@collabora.com>
Wed, 23 Sep 2020 17:26:51 +0000 (13:26 -0400)
committerNicolas Dufresne <nicolas.dufresne@collabora.com>
Wed, 23 Sep 2020 19:48:24 +0000 (15:48 -0400)
The rtpjitterbuffer is now part of the session elements, we no longer need
to do the ref_sink dance when signalling it. It is already owned by the bin
when signalled. Also, the code that handles generic session elements already
handle the ref_sink() calls since:

03dc22951bacb6fdc3868c8f801e6a52c33a745f

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/merge_requests/735>

gst/rtpmanager/gstrtpbin.c

index 5e88e47..bdb16a0 100644 (file)
@@ -1790,19 +1790,12 @@ create_stream (GstRtpBinSession * session, guint32 ssrc)
     g_object_set (buffer, "max-ts-offset-adjustment",
         rtpbin->max_ts_offset_adjustment, NULL);
 
-  /* need to sink the jitterbufer or otherwise signal handlers from bindings will
-   * take ownership of it and we don't own it anymore */
-  gst_object_ref_sink (buffer);
   g_signal_emit (rtpbin, gst_rtp_bin_signals[SIGNAL_NEW_JITTERBUFFER], 0,
       buffer, session->id, ssrc);
 
   if (!rtpbin->ignore_pt)
     gst_bin_add (GST_BIN_CAST (rtpbin), demux);
 
-  /* unref the jitterbuffer again, the bin has a reference now and
-   * we don't need it anymore */
-  gst_object_unref (buffer);
-
   /* link stuff */
   if (demux)
     gst_element_link_pads_full (buffer, "src", demux, "sink",