[MPlayer-DOCS] incomprehensibility in encoding guide

[Corey Hickey]

Guillaume POIRIER wrote:
>> > > > As far as I can tell from the code and a couple quick tests,
>> > > > the situation with mbd and mbcmp is still the same: when all
>> > > > other *cmp options are 0 (the default), making mbcmp nonzero
>> > > > will have a beneficial effect on quality regardless of what
>> > > > mbd is set to. When mbd is set to 1 or 2, the specific value
>> > > > of mbcmp is ignored, except to check whether it is nonzero.
>> > > > When mbd is 0, the specific value of mbcmp matters.
>> > >
>> > > So, maybe manpage should be updated to clarify this?
>> >
>> > Like this?
>>
>> Almost, but IMHO it would be nice to mention that "When mbd is set
>> to 1 or 2, the specific value of mbcmp is ignored, except to check
>> whether it is nonzero." or something similar.
> 
> Isn't that overly verbose?

Actually, it might not be verbose enough to be to precise. This isn't 
any more fun to read, but should match the actual behavior:

"When mbd is set to 1 or 2, the value of mbcmp is ignored when comparing 
macroblocks. If any comparison setting (precmp, subcmp, cmp, or mbcmp) 
is nonzero, however, a slower but better half-pel motion search will be 
used, regardless of what mbd is set to. If qpel is set, quarter-pel 
motion search will be used regardless."

If someone wants to make the slower but better half-pel motion search a 
separate option, instead, that might be less complicated.

-Corey