[Bug 743900] oggdemux gets first packet timestamp wrong - theora

[GStreamer (GNOME Bugzilla)]

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

Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> ---
That takes care of the issue you found. Please reopen if there's still
something.

commit ca136e3648d13fd81c54dc0fe7f5d018018a379d
Author: Vincent Penquerc'h <vincent.penquerch at collabora.co.uk>
Date:   Wed Feb 4 17:13:44 2015 +0000

    oggdemux: fix wrong first granule

    The code was using the first nonnegative granulepos to seed the
    granule tracking, which appeared to work since headers have zero
    granulepos. However, this does not work for files with a hole at
    start, which are common in live streaming.

    The correct behavior is to look for the first granule, and subtract
    the duration of all the packets finishing on this page.

    The function which does this relies on the fact that the ogg_stream
    structure can be duplicated by shallow copy, in order to pull the
    packets from the first page(s) on the copy without affecting the
    original stream state.

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