[FFmpeg-devel] [PATCH] avformat: offset fps calculation by one to include 60.00 fps

[Hendrik Leppkes]

Hi,

attached patch moves the fps calculation window slightly to include
60.00fps as a possible candidate for the fps, and instead don't consider 0
fps as a valid option.
60.00 fps is quite common for example when using a screen recorder on a PC
to capture a video game (eg. Fraps). Before this patch, the FPS guessing
code would end up on 59.94 as the closest match, instead of the true 60.00.

This was the easiest solution to include 60.00fps into the calculation,
instead the range could be extended by one to include both 0 fps and 60
fps, but since 0 fps didn't seem all that useful to me, i opted to go with
the more simple approach.

Feedback welcome.

- Hendrik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: avformat-offset-fps-calculation-by-one.patch
Type: application/octet-stream
Size: 786 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120621/5117b768/attachment.obj>