[MPlayer-DOCS] [PATCH] framerate updates

[D Richard Felker III]

On Tue, Mar 01, 2005 at 07:43:58PM -0800, Corey Hickey wrote:
> I've finished an update to the mplayer documentation.
> 
> Changes:
> - Updated help_mp-en.h to say
>   - "30000/1001 fps" insead of "30fps"
>   - "24000/1001 fps" instead of "24fps"
> - Updated documentation.xml, mencoder.xml, video.xml, mplayer.1 to say
>   - "60000/1001" instead of 59.94
>   - "30000/1001" instead of 29.97
>   - "24000/1001" instead of 23.976
> - Added a note about 60 vs. 59.94 vs. 60000/1001, etc. to the telecine
>   howto's introduction.
> 
> Please don't apply this without verifying that all the corrections
> should indeed be made. I updated everything I could find, which includes
> changes regarding parts of mplayer I don't use. If anything is incorrect
> I can easily remove the offending changes.

I'm very undecided on these changes. IMO it's important to use the
correct rational values whenever there are command lines involved, but
in the actual documentation to be read, it's very cumbersome. It's
kinda like the whole kilobyte/kabibyte crap.. You CAN use really
awkward precise wording if you like, but people "in the field" prefer
to just know what's meant by context. As such, I would just call the
video "24 fps" or "30 fps" unless there's a real need to be specific
that it's 24000/1001 or 30000/1001.

Or we could do like SI crap and come up with silly-sounding new
words... like call it 24000 mibifps or something. ;)))))

> Parts including changes I'm not sure of:
> - mplayer.1: -lavcopts vframerate

You mean mpegopts...

>  .B \-ovc <codec name>
> @@ -7932,7 +7932,7 @@
>  Sets the video bitrate in kbit/s for MPEG-1/2 video.
>  .
>  .TP
> -.B vframerate=<23.976 | 24 | 25 | 29.97 | 30 | 50 | 59.94 | 60 >
> +.B vframerate=<24000/1001 | 24 | 25 | 30000/1001 | 30 | 50 | 60000/1001 | 60 >

Wrong. This is a string variable, and you have to use the exact
strings the muxer wants! :( I agree it's bad. IMO it should take a
float (mplayer also accepts rationals for floats) and convert it to
the closest matching mpeg framerate.

Rich