[MPlayer-dev-eng] [PATCH] H.264 encoding in mencoder

[Diego Biurrun]

snacky at ikaruga.co.uk writes:
> On Sun, Aug 22, 2004 at 09:57:38PM -0500, snacky at ikaruga.co.uk wrote:
> 
> > This is the first time I've ever generated a patch; let me know if I
> > did something wrongly.
> 
> As if to underscore my point, I screwed that up. Please don't use the
> patch in my last message. Attached is a patch that's not borked up.


> +.B bitrate=<value>
> +Sets the bitrate to be used in kbits/\:second (default: 3000). In order for
> +this to take effect, you must set cbr=1

You should start new sentences on a new line in the man page, same
below.

> +.B iframeref=<value>
> +Reference I-Frame (default: 1 - usually should not be changed)

No need to capitalize here, it's not a full sentence, similar things
below.

I'd write "should usually not be changed".

> +.B idrframe=<value>
> +Each <value> I-Frames are IDR-Frames

This is a full sentence, so it should be ended with a period.

> +.B fullinter
> +Use all available interframe analyse methods (i4x4, psub8x8, psub16x16) (default: 4x4, psub8x8) 

Please break overly long lines and do not add trailing whitespace.

Diego