[Bug 770107] qtdemux: reports only one encryption system even if it can support more than one

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sun May 21 13:34:36 UTC 2017


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

--- Comment #22 from Xabier Rodríguez Calvar <calvaris at igalia.com> ---
(In reply to Tim-Philipp Müller from comment #20)
> >+  if (stream->protection_scheme_event_queue.length) {
> >+    ...
> >+    walk = stream->protection_scheme_event_queue.tail;
> >+  } else {
> >+    ...
> >+    walk = qtdemux->protection_event_queue.tail;
> >+  }
> >+
> >+  g_value_init (&event_list, GST_TYPE_LIST);
> >+  for (; walk; walk = g_list_previous (walk)) {
> >+    ...
> >+  }
> 
> OOC, what is the reason we're walking the event queues backwards here?

Cause that way, we queue the protection events in the same order they are found
in the media. If we don't walk the queue backwards, we'll have to do it in the
client, but I think pushing the events in the same order they are found is a
good idea.

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