[MPlayer-dev-eng] [PATCH] MPEG-2 TS 60000/1001 FPS Conversion Error in mencoder

[Trent Piepho]

On Sat, 25 Nov 2006, Bill Adams wrote:
> [snip]
> [mpeg2video @ 0x861accc]MPEG1/2 does not support 19001/317 fps

You can find an even older thread, where I explain that av_d2q doesn't
work.  Certain people refused to acknowledge that the numbers were wrong.
I can see 19001/317, you can see it, others can see it; we know it is
wrong.  But these certain people will not believe it!  They say,
"Theoretically, av_d2q will work.  Therefore, you are just imagining seeing
19001/317 when it should be 60000/1001."

> After digging around for a while, I came across the bug. In mencoder.c
> the line:
>
>  AVRational q=
> av_d2q(force_ofps?force_ofps:sh_video->fps*playback_speed, 30000);
>
> caps the numerator to 30,000 (I think). Changing this to:

yep

>
>  AVRational q=
> av_d2q(force_ofps?force_ofps:sh_video->fps*playback_speed, 60000);
>
> fixes the problem.

But now try mencoder with -ofps 24000/1001, you get this:

[mpeg2video @ 0x86aa018]MPEG1/2 does not support 31001/1293 fps