[MPlayer-DOCS] Improved vi_qfactor and (tcplx|scplx|p)_mask

[Diego Biurrun]

Guillaume POIRIER writes:
> Le dim 26/09/2004 à 15:12, Diego Biurrun a écrit :
> 
> > >  .B tcplx_mask=<0.0\-1.0>
> > >  temporal complexity masking (default: 0.0 (disabled))
> > > +Imagine a scene with a bird flying across the whole scene, tcplx_mask
> > > +will decrease the bird's macroblocks quantizers (thus decreasing their
> > 
> > macroblock
> 
> I think it needs to be plural since the bird is made out of several
> macroblocks.

Then you would have to say something to the effect of

  decrease the bird's macroblocks' quantizers

although I prefer the equivalent (I hope)

  decrease the quantizers of the bird's macroblocks

> > > +(default: 0.0 (disabled)).
> > > +p_mask=1.0 doubles the bits allocated to each intra block.
> > 
> > Same question here as in the paragraph above: Are we talking about
> > blocks or frames?
> 
> My understanding of masking functions is that they would on a
> block-by-block basis, so for consistency reasons, it only reads about
> blocks. 
> I believe that P-blocks are included in P-frames, so talking about blocs
> is probably more specific and precise than talking about frames.
> Does that make sense?

OK

> The attached patch includes all your suggestions (even those about
> macroblocks, but leaves untouched the issue about (blocks|frames).
> Please tell me if you agree with it.

Yes, commit, possibly with the change I suggested above.

Diego