From c46642523c84974cf9e6d7ed7dc9fbcd16b6d40b Mon Sep 17 00:00:00 2001 From: Stefan Kost Date: Thu, 6 May 2010 15:40:33 +0300 Subject: [PATCH] pulsesrc: reflow to truncate caps just once We get writable cpas from the intersection (unless it failed). As we truncate those anyway, we don't need to manyaly copy the first structure. --- ext/pulse/pulsesrc.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/ext/pulse/pulsesrc.c b/ext/pulse/pulsesrc.c index 0ddaa43b36..b3126f4813 100644 --- a/ext/pulse/pulsesrc.c +++ b/ext/pulse/pulsesrc.c @@ -884,23 +884,17 @@ gst_pulsesrc_negotiate (GstBaseSrc * basesrc) peercaps = gst_pad_peer_get_caps (GST_BASE_SRC_PAD (basesrc)); GST_DEBUG_OBJECT (basesrc, "caps of peer: %" GST_PTR_FORMAT, peercaps); if (peercaps) { - GstCaps *icaps; - /* get intersection */ - icaps = gst_caps_intersect (thiscaps, peercaps); - GST_DEBUG_OBJECT (basesrc, "intersect: %" GST_PTR_FORMAT, icaps); + caps = gst_caps_intersect (thiscaps, peercaps); + GST_DEBUG_OBJECT (basesrc, "intersect: %" GST_PTR_FORMAT, caps); gst_caps_unref (thiscaps); gst_caps_unref (peercaps); - if (icaps) { - /* take first (and best, since they are sorted) possibility */ - caps = gst_caps_copy_nth (icaps, 0); - gst_caps_unref (icaps); - } } else { /* no peer, work with our own caps then */ caps = thiscaps; } if (caps) { + /* take first (and best, since they are sorted) possibility */ caps = gst_caps_make_writable (caps); gst_caps_truncate (caps);