[gstreamer-bugs] [Bug 339678] New: [matroskamux] wrong timestamps of B-frames

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=339678
 GStreamer | gst-plugins-good | Ver: HEAD CVS

           Summary: [matroskamux] wrong timestamps of B-frames
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: major
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: josef.zlomek at itonis.tv
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


When using B-frames, some frames have lower timestamps than the preceding ones
(because of decoding order of frames, e.g. IPBB). The code is computing the
timestamp relative to cluster time in unsigned integers. When the difference is
negative, the result is completely wierd.
The relative timestamp should be signed according to the description in the end
of http://www.matroska.org/technical/specs/index.html  

The attached patch changes the computation of relative timestamp to signed
integers and adds rounding (i.e. division with rounding).


-- 
Configure bugmail: http://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.