# HG changeset patch # User michael # Date 1159295647 0 # Node ID 18d9db9590e6edf343d8a3363722797e53219e95 # Parent c55f7573fb9508ff4c00b9f97b5c2d39be8c68d5 fix r_frame_rate init code for groundhog.vob diff -r c55f7573fb95 -r 18d9db9590e6 utils.c --- a/utils.c Tue Sep 26 18:03:17 2006 +0000 +++ b/utils.c Tue Sep 26 18:34:07 2006 +0000 @@ -1873,7 +1873,10 @@ duration_sum[index]= duration; duration_count[index]=1; }else{ - int factor= av_rescale(duration, duration_count[index], duration_sum[index]); + int factor= av_rescale(2*duration, duration_count[index], duration_sum[index]); + if(factor==3) + duration_count[index] *= 2; + factor= av_rescale(duration, duration_count[index], duration_sum[index]); duration_sum[index] += duration; duration_count[index]+= factor; } @@ -1931,7 +1934,9 @@ if(st->codec->codec_id == CODEC_ID_RAWVIDEO && !st->codec->codec_tag && !st->codec->bits_per_sample) st->codec->codec_tag= avcodec_pix_fmt_to_codec_tag(st->codec->pix_fmt); - if(duration_count[i] && st->codec->time_base.num*101LL <= st->codec->time_base.den && + if(duration_count[i] + && (st->codec->time_base.num*101LL <= st->codec->time_base.den || st->codec->codec_id == CODEC_ID_MPEG2VIDEO) && + //FIXME we should not special case mpeg2, but this needs testing with non mpeg2 ... st->time_base.num*duration_sum[i]/duration_count[i]*101LL > st->time_base.den){ int64_t num, den, error, best_error; @@ -1946,11 +1951,12 @@ av_reduce(&st->r_frame_rate.num, &st->r_frame_rate.den, j, 12, INT_MAX); } } - for(j=24; j<=30; j+=6){ - error= ABS(1001*12*num - 1000*12*j*den); + for(j=0; j<3; j++){ + static const int ticks[]= {24,30,60}; + error= ABS(1001*12*num - 1000*12*den * ticks[j]); if(error < best_error){ best_error= error; - av_reduce(&st->r_frame_rate.num, &st->r_frame_rate.den, j*1000, 1001, INT_MAX); + av_reduce(&st->r_frame_rate.num, &st->r_frame_rate.den, ticks[j]*1000, 1001, INT_MAX); } } }