[Bug 703350] New: qtdemux reports wrong framerate

[GStreamer (bugzilla.gnome.org)]

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

           Summary: qtdemux reports wrong framerate
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: matej.knopp at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=248087)
 --> (https://bugzilla.gnome.org/attachment.cgi?id=248087)
Patch

Currently qtdemux takes one sample from first 20 samples and uses its duration
to compute framerate. For some files this can give invalid results (one file I
tried this with shows 35fps). 

Also the timescale that is usually used (2400) means that single sample
duration can not be used to compute framerate of 24000/1001. Best case scenario
currently is that the sample will have duration of 100, resulting in 24fps, but
that's not precise. Some samples have duration 99 making the average framerate
23.98.

Patch uses average sample duration and scales the timescale appropriately (so
that it gives proper framerate for 24000/1001).

-- 
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.