[Bug 765096] baseparse: When initializing DTS from PTS, remember that we did so

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|git master                  |1.8.1

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Merged into master and 1.8, it's a regression from 1.0 that happened for 1.2 :)

commit 895332e056735e0bc0c09cc462285d8387e8bb68
Author: Sebastian Dröge <sebastian at centricular.com>
Date:   Fri Apr 15 13:50:30 2016 +0300

    baseparse: When initializing DTS from PTS, remember that we did so

    If we don't store the value in prev_dts, we would over and over again
    initialize the DTS from the last known upstream PTS. If upstream only
provides
    PTS every now and then, then this causes DTS to be rather static.

    For example in adaptive streaming scenarios this means that all buffers in
a
    fragment will have exactly the same DTS while the PTS is properly updated.
As
    our queues are now preferring to do buffer fill level calculations on DTS,
    this is causing huge problems there.

    See https://bugzilla.gnome.org/show_bug.cgi?id=691481#c27 where this part
of
    the code was introduced.

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

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