[MPlayer-DOCS] [PATCH] document a couple more x264 options in mencoder.xml

The Wanderer inverseparadox at comcast.net
Tue May 31 02:38:18 CEST 2005


Diego Biurrun wrote:

> On Mon, May 30, 2005 at 11:16:33PM +0200, Guillaume POIRIER wrote:
> 
>> On 5/30/05, Jeff Clagg <snacky at ikaruga.co.uk> wrote:
>> 
>>> Added entries for the me (motion estimation search algorithm) and
>>> 4x4mv options.
>> 
>> Looks good to me, except that I think it would be a good idea to
>> explain what each me option means: (1=diamond search, radius 1,
>> 2=hexagon search, radius 2, 3=uneven multi-hexagon search)
>> 
>> Other than that, I'll wait until someone with more English
>> knowledge comments on it to apply it.
> 
> Jeff's English composition skills are completely above reproach.
> Commit whenever you feel like it.

Other than a couple of nitpicks which are mostly "yes, it's good
English, but I'm not sure it's sufficiently idiot-proof clear", and
which could well be just a product of lack of sleep on my part, I pretty
much agree. I have no objections to committing this as it stands.

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

A government exists to serve its citizens, not to control them.




More information about the MPlayer-DOCS mailing list