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

Diego Biurrun diego at biurrun.de
Thu Aug 5 13:38:11 CEST 2004


Bernhard Rosenkraenzer writes:

I'm being nitpickish today ;-)

> +sets the bitrate to be used in kbits/\:second (default: 3000)

You should end sentences with a "." and start them capitalized..

> +Frequency of I frames (default: 60)

.. and start lowercase when it is not a complete sentence.

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

s/analyse/analysis/

You'll get extra points for removing the trailing whitespace and
breaking these overly long lines.

Diego




More information about the MPlayer-dev-eng mailing list