[MPlayer-cvslog] r25811 - trunk/mencoder.c

[Reimar Döffinger]

Hello,
On Mon, Feb 11, 2008 at 01:18:22PM -0500, Rich Felker wrote:
> On Sun, Jan 20, 2008 at 12:19:42AM +0100, iive wrote:
> > Author: iive
> > Date: Sun Jan 20 00:19:41 2008
> > New Revision: 25811
> > 
> > Log:
> > Extend the precision of rationale conversion so
> > it would give proper result for framerate 60000/1001 .
> > Otherwise framerate restrained encoders like lavc mpeg2video
> > would refuse to encode, even if -ofps 60000/1001 is given.
> 
> This commit seems to have seriously broken MEncoder. See the thread on
> -users..

Mostly it should be changed to use double, but how about
av_d2q(fps, fps*1001+1);
or so as a quick hack?

Greetings,
Reimar Döffinger