[MPlayer-DOCS] [PATCH] man page: update spp

[The Wanderer]

Corey Hickey wrote:

> The Wanderer wrote:
> 
>>> Michael just altered spp to prevent the amplification of B-frame
>>> flicker. This patch updates the man page accordingly; the wording
>>> is OK with Michael.
>> 
>>> Index: mplayer.1
>>> ===================================================================
>>> RCS file: /cvsroot/mplayer/main/DOCS/man/en/mplayer.1,v
>>> retrieving revision 1.1163
>>> diff -u -r1.1163 mplayer.1
>>> --- mplayer.1	6 Dec 2005 08:42:48 -0000	1.1163
>>> +++ mplayer.1	6 Dec 2005 21:37:07 -0000
>> 
>> Patches should be made from the root of the source tree, to make
>> applying them easier.
> 
> Ok.
> 
>> (Also, this patch was made against a man page two revisions old -
>> but it applies anyway.)
> 
> That's because you committed two revisions in the ten minutes before
> I sent this in. :)

Oh, I know - I just mentioned it because of the warning from cvs if you
try to commit while your own copy has not been synced with the remote
copy. (Why exactly this makes it worth mentioning I'm no longer sure,
but it made sense at the time.) I tend to be paranoid about making
absolutely certain that my copy and the remote one are in sync except
for the change I'm looking at, and...

>>> @@ -4900,6 +4900,10 @@
>>> 0: hard thresholding (default)
>>> .br
>>> 1: soft thresholding (better deringing, but blurrier)
>>> +.br
>>> +4: like 0, but also use B-frames' QP (may cause flicker)
>>> +.br
>>> +5: like 1, but also use B-frames' QP (may cause flicker)
>> 
>> Documentation policy is to avoid the use of hyphens, but I think
>> that this becomes ungrammatical if not unclear without them; should
>> we rephrase, or allow the hyphens in this case?
> 
> All the other instances of "B-frames" I can find in the man page are
> spelled that way, except when they are explicitly referring to an
> option spelled "bframes". I don't know if those are the correct
> spelling, but at least it's consistent.

Right - as Diego has pointed out, I was apparently misremembering
policy. Will commit in a moment.

-- 
       The Wanderer

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

Secrecy is the beginning of tyranny.