[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