[Bug 754851] aggregator: Use the whole aggregator latency for the queue size, not just the latency property

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Sep 18 00:26:03 PDT 2015


https://bugzilla.gnome.org/show_bug.cgi?id=754851

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #311589|none                        |reviewed
             status|                            |

--- Comment #5 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 311589:
 --> (https://bugzilla.gnome.org/review?bug=754851&attachment=311589)

Makes sense, let's get this in :)

::: gst-libs/gst/base/gstaggregator.c
@@ +2054,3 @@
+  /* We also want at least two buffers, one is being processed and one is
ready
+   * for the next iteration when we operate in live mode. */
+  if (self->priv->peer_latency_live && aggpad->priv->num_buffers < 2)

Why not g_queue_get_length() ?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list