[Bug 656115] [PATCH] mpegpsdemux: recalculate adjust if diffenrence is negative

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Dec 9 07:40:53 PST 2011


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

Vincent Penquerc'h <vincent.penquerch> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |vincent.penquerch at collabora
                   |                            |.co.uk
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.23

--- Comment #3 from Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> 2011-12-09 15:40:50 UTC ---
I pushed this a while ago but forgot to close the bug:

commit 2099a3945957fa0ddf10cd8b2ddef0e54f86bcdf
Author: Alexey Fisher <bug-track at fisher-privat.net>
Date:   Tue Nov 29 12:20:51 2011 +0100

    mpegpsdemux: recalculate adjust if difference is negative

    One of my dvds jump on some position and miss about 1 minute of stream.
    The reason was mpeg timestamps. On some position scr difference is
negative.
    It produced negative timestamps. Since it was converted to unsigned value,
    gstreamer timestamps was invalid. Instead of increasing mpeg ts,
    they was decreasing till it started to be positive.

    The jump in timestamps caused mpeg2dec to skip frames to make QoS happy.

    This patch just make diff unsigned to avoid negative values.

    Signed-off-by: Alexey Fisher <bug-track at fisher-privat.net>

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

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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