[Bug 791979] aggregator: only process one event/query at a time

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sun Feb 4 14:52:15 UTC 2018


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

--- Comment #7 from Stefan Sauer (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> ---
Let me explain more. The pad queues take *serialized* events. The old code did:
foreach q in pad_queues:
  foreach d in q:
    break if d is buffer
    handle d

IMHO this will not allow the subclasses to sync on the queue items. Of could we
could implement another 'special case:
foreach q in pad_queues:
  foreach d in q:
    break if d is buffer
    break if d is event and d.type is segment-done
    handle d

but I don't think this is the right place to do this. The patch removes the
"foreach d in q" and this way subclasses can peek at the heads of the queue and
and act accordingly.

We've been taking that for segmented non flushing seeks we need queues to track
the events. We do have queues to track the events already. I think we need a
better way to trigger doing stuff depending on what kind of objects are in the
queues.

-- 
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