Module: libav Branch: master Commit: 78987a88a88b28d93d03ed6c228bcb33f178444f
Author: Janne Grunau <[email protected]> Committer: Janne Grunau <[email protected]> Date: Wed Jan 29 15:12:48 2014 +0100 lavf: include 60 fps in guessed standard frame rates Due to what looks like an off-by-one error 60 * 12 * 1001 / 12 * 1001 is not tested as standard frame rate in avformat_find_stream_info(). --- libavformat/utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index eaedf74..01215ef 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -2175,7 +2175,7 @@ static void compute_chapters_end(AVFormatContext *s) static int get_std_framerate(int i) { if (i < 60 * 12) - return i * 1001; + return (i + 1) * 1001; else return ((const int[]) { 24, 30, 60, 12, 15 })[i - 60 * 12] * 1000 * 12; } @@ -2458,7 +2458,7 @@ int avformat_find_stream_info(AVFormatContext *ic, AVDictionary **options) /* Round guessed framerate to a "standard" framerate if it's * within 1% of the original estimate. */ - for (j = 1; j < MAX_STD_TIMEBASES; j++) { + for (j = 0; j < MAX_STD_TIMEBASES; j++) { AVRational std_fps = { get_std_framerate(j), 12 * 1001 }; double error = fabs(av_q2d(st->avg_frame_rate) / av_q2d(std_fps) - 1); _______________________________________________ libav-commits mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-commits
