[Bug 778426] qtdemux: Properly handle edit list in push mode

[GStreamer (GNOME Bugzilla)]

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

--- Comment #33 from Seungha Yang <pudding8757 at gmail.com> ---
(In reply to Thibault Saunier from comment #27)
> Review of attachment 370371 [details] [review]:
> 
> ::: gst/isomp4/qtdemux.c
> @@ +9158,3 @@
>        buffer += entry_size;
> +
> +      if (!GST_CLOCK_TIME_IS_VALID (stime)) {
> 
> In what case is `stime` going to be CLOCK_TIME_NONE? I do not have the
> impression that this is going to happen with the file you provided.
> 
> Otherwise I understand the idea, and it seems sensible.

Not sure whether it's possible or not, but in that function, "qtdemux->duration
== -1" can cause CLOCK_TIME_NONE. As I mentioned before, this patch is not
required for this bug.

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