audiobasesrc: Avoid unnecessary configuration
authorJan Schmidt <jan@centricular.com>
Thu, 2 Jan 2014 15:18:20 +0000 (02:18 +1100)
committerJan Schmidt <jan@centricular.com>
Thu, 2 Jan 2014 15:20:39 +0000 (02:20 +1100)
Port a change from audiobasesink from def07410, to ignore setcaps
when the caps don't actually change, and avoid a reconfiguration
and reset of the ringbuffer in that case.

gst-libs/gst/audio/gstaudiobasesrc.c

index db2f462..872116b 100644 (file)
@@ -536,11 +536,17 @@ gst_audio_base_src_setcaps (GstBaseSrc * bsrc, GstCaps * caps)
   GstAudioRingBufferSpec *spec;
   gint bpf, rate;
 
+  spec = &src->ringbuffer->spec;
+
+  if (G_UNLIKELY (spec->caps && gst_caps_is_equal (spec->caps, caps))) {
+    GST_DEBUG_OBJECT (src,
+        "Ringbuffer caps haven't changed, skipping reconfiguration");
+    return TRUE;
+  }
+
   GST_DEBUG ("release old ringbuffer");
   gst_audio_ring_buffer_release (src->ringbuffer);
 
-  spec = &src->ringbuffer->spec;
-
   spec->buffer_time = src->buffer_time;
   spec->latency_time = src->latency_time;