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

[Corey Hickey]

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. :)

> 
>>@@ -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.

-Corey