[0.11] gst-plugins-base: Revert "playsink: Only add a queue before the audio sink if visualizations are enabled"

Wim Taymans wtay at kemper.freedesktop.org
Wed Mar 30 12:41:10 PDT 2011


Module: gst-plugins-base
Branch: 0.11
Commit: 65320a04abec3afd7de27f93713a7a27932d723b
URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=65320a04abec3afd7de27f93713a7a27932d723b

Author: Sebastian Dröge <sebastian.droege at collabora.co.uk>
Date:   Thu Mar 24 14:21:01 2011 +0100

Revert "playsink: Only add a queue before the audio sink if visualizations are enabled"

This reverts commit df886c0622257bb8635e5bd0fc7fc3da20bfc3be.

---

 gst/playback/gstplaysink.c |   31 +++++++++++++------------------
 1 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/gst/playback/gstplaysink.c b/gst/playback/gstplaysink.c
index 879f9c4..169a541 100644
--- a/gst/playback/gstplaysink.c
+++ b/gst/playback/gstplaysink.c
@@ -1645,7 +1645,7 @@ notify_mute_cb (GObject * object, GParamSpec * pspec, GstPlaySink * playsink)
  *  +-------------------------------------------------------------+
  */
 static GstPlayAudioChain *
-gen_audio_chain (GstPlaySink * playsink, gboolean raw, gboolean queue)
+gen_audio_chain (GstPlaySink * playsink, gboolean raw)
 {
   GstPlayAudioChain *chain;
   GstBin *bin;
@@ -1691,24 +1691,19 @@ gen_audio_chain (GstPlaySink * playsink, gboolean raw, gboolean queue)
 
   /* we have to add a queue when we need to decouple for the video sink in
    * visualisations */
-  if (queue) {
-    GST_DEBUG_OBJECT (playsink, "adding audio queue");
-    chain->queue = gst_element_factory_make ("queue", "aqueue");
-    if (chain->queue == NULL) {
-      post_missing_element_message (playsink, "queue");
-      GST_ELEMENT_WARNING (playsink, CORE, MISSING_PLUGIN,
-          (_("Missing element '%s' - check your GStreamer installation."),
-              "queue"), ("audio playback and visualizations might not work"));
-      head = chain->sink;
-      prev = NULL;
-    } else {
-      g_object_set (chain->queue, "silent", TRUE, NULL);
-      gst_bin_add (bin, chain->queue);
-      prev = head = chain->queue;
-    }
-  } else {
+  GST_DEBUG_OBJECT (playsink, "adding audio queue");
+  chain->queue = gst_element_factory_make ("queue", "aqueue");
+  if (chain->queue == NULL) {
+    post_missing_element_message (playsink, "queue");
+    GST_ELEMENT_WARNING (playsink, CORE, MISSING_PLUGIN,
+        (_("Missing element '%s' - check your GStreamer installation."),
+            "queue"), ("audio playback and visualizations might not work"));
     head = chain->sink;
     prev = NULL;
+  } else {
+    g_object_set (chain->queue, "silent", TRUE, NULL);
+    gst_bin_add (bin, chain->queue);
+    prev = head = chain->queue;
   }
 
   /* find ts-offset element */
@@ -2398,7 +2393,7 @@ gst_play_sink_reconfigure (GstPlaySink * playsink)
 
     if (!playsink->audiochain) {
       GST_DEBUG_OBJECT (playsink, "creating new audio chain");
-      playsink->audiochain = gen_audio_chain (playsink, raw, need_vis);
+      playsink->audiochain = gen_audio_chain (playsink, raw);
     }
 
     if (!playsink->audio_sinkpad_stream_synchronizer) {



More information about the gstreamer-commits mailing list