[MPlayer-cvslog] CVS: main/DOCS/tech slave.txt,1.48,1.49

[The Wanderer]

syncmail at mplayerhq.hu wrote:


> -[brightness|contrast|gamma|hue|saturation] <-100 - 100> [<abs>]
> +[brightness|contrast|gamma|hue|saturation] <value> [<abs>]
>      Set/adjust video parameters.
> +    If abs is not given or zero, modifies parameter by <value>.

This could equally well be read "If abs is not given or is zero" or "If
abs is not given or is not zero". It could be fixed either by swapping
the two conditions ("is zero or not given") or by adding the verb ("is
not given or is zero"). Either works pretty much equally well, and I'm
not sure which would be considered the lesser change; any preference?

> +    If abs is non-zero, parameter is set to <value>

I know this isn't the man page, but you seem to be following the rule
anyway above - there's a missing period at the end of this sentence.

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