[FFmpeg-devel] [PATCH] lavf: inspect more frames when the time base is unreliable

[Michael Niedermayer]

On Sat, Jan 29, 2011 at 02:39:38AM +0200, Anssi Hannula wrote:
> 23.976 fps H.264 Matroska files are usually detected as 24 fps
> (r_frame_rate) by av_find_stream_info(). Fix that by inspecting the
> timestamps of 30 frames instead of just 20 for video codecs with
> unreliable time base.

While i agree with others that the header values should be used if they all
match and fit in, in absence
of any header values this code still seems to need to be improved as well
so i would suggest that the timebase be checked and if its to inaccurate
to detect this difference at 20 but detect it at 30 frames than 30 frames
should be used.
(more generic solutions may be possible too)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110129/10970776/attachment.pgp>