[Bug 345830] qtdemux: better edit lists handling

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Jan 15 11:34:46 PST 2014


https://bugzilla.gnome.org/show_bug.cgi?id=345830
  GStreamer | gst-plugins-good | git

Thiago Sousa Santos <thiago.sousa.santos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.3.1

--- Comment #23 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2014-01-15 19:34:29 UTC ---
commit 90a5565229f39b181e8644c4b5482c614dbcb0fd
Author: Thiago Santos <ts.santos at sisa.samsung.com>
Date:   Fri Jan 3 10:59:35 2014 -0300

    qtdemux: do not ignore empty segments

    Make sure empty segments are used and pushed with a gap event
    to represent its data (or lack of it)

    Each QtSegment is mapped into a GstSegment with the corresponding
    media range. For empty QtSegments a gap event is pushed instead
    of GstBuffers and it advances to the next QtSegment.

    To make this work with seeks, need to keep track of the starting
    'base' to make sure it remains consistently increasing when
    pushing new segment events.
    For example: if a seek makes qtdemux start from 5s, the first
    segment will have a base=0. When the next segment is activated,
    its base time will be QtSegment.time - qtdemux.segment_base so
    that it doesn't include the first 5s that weren't played and
    shouldn't be accounted on the running time

    This purposedly will remove the fix made for
    https://bugzilla.gnome.org/show_bug.cgi?id=700264, at this
    point it was decided to respect the gaps, even if they cause
    a delay on playback, because that's the way the file was crafted.

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

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.


More information about the gstreamer-bugs mailing list