[MPlayer-DOCS] CVS: main/DOCS/xml/en mencoder.xml,1.39,1.40

[The Wanderer]

A few things which got missed before - some your fault, some mine:

Guillaume Poirier CVS wrote:

> CVS change done by Guillaume Poirier CVS
> 
> Update of /cvsroot/mplayer/main/DOCS/xml/en
> In directory mail:/var2/tmp/cvs-serv23999
> 
> Modified Files:
> 	mencoder.xml 

> @@ -590,8 +591,10 @@
>    <option>vqmin</option>, which is 2 by default). As soon as you specify a
>    low enough bitrate that <systemitem class="library">libavcodec</systemitem>
>    is forced to use a higher quantizer, then you're almost certainly ruining
> -  the quality of your video. In general, you should avoid CBR altogether if
> -  you care about quality.
> +  the quality of your video.
> +  In order to avoid that, you should probably down-scale your video, according
> +  to the method which will be exposed later on that guide.
> +  In general, you should avoid CBR altogether if you care about quality.

Here, you still use "exposed" - if you want to keep rephrasing to an
absolute minimum, albeit at slight expense in grammatical terms, you
could simply replace it with the words "gone over". Either that or I'm
completely misunderstanding what is actually meant here, in which case
the sentence most definitely should be rephrased.

Also, there is an inconsistency: in at least one place you use
"down-scale", in at least one place you use "downscale", and in at least
one place you use "scale up". These forms should probably be made
consistent; I'm personally inclined towards the last of the three, but
if you prefer another I'm not likely to object too strongly.

> +  You may want to normalize the sound before encoding, as DVD audio tracks
> +  are commonly recorded at low volumes.
> +  You can use the tool <application>normalize</application> for instance,
> +  which is available in most distribution.

Pluralize: "distributions". If there exists an adequately comparable
tool for use under Windows, it might be worth mentioning that here - or
in the Windows-specific section of the documentation, minimal though
that is.

> @@ -801,6 +912,153 @@
>  
>  </sect2>
>  
> +<sect2 id="menc-feat-dvd-mpeg4-encoding-options">
> +<title>Encoding options</title>
> +
> +<para>
> +  Ideally, you'd probably want to be able to just tell the encoder to switch
> +  into "high quality" mode and move on.
> +  That would probably be nice, but unfortunately hard to implement as different
> +  encoding options yield different qualities depending on the source material.
> +  Anime and live action are for example two very different materials that
> +  require different care.
> +  The good news is that some options should never be left out, like
> +  <option>mbd=2</option>, <option>trell</option>, and <option>v4mv</option>.
> +  See below a detailed description of common encoding options. 

I missed this one before - "for a detailed distribution". The word "for"
is missing.

> +<listitem><para>
> +  <emphasis role="bold">qns</emphasis>: very slow, especially when combined
> +  with qprd.
> +  This option will make the encoder reduce as much as possible noise due to
> +  compression artifacts instead making the encoded video strictly match the
> +  source.

This sentence is awkwardly phrased, largely because the verb qualifier
("as much as possible") is placed before the noun phrase ('noise due to
compression artifacts"). The simplest fix would be to swap them; a more
complicated fix would involve rephrasing, and possibly use of the word
"minimize". Also, you want to either insert a comma before "instead" or
the word "of" after "instead"; the two forms have opposing meanings, so
which one you should use depends on what you actually mean here.

> +<listitem><para>
> +  <emphasis role="bold">lumi_mask, dark_mask</emphasis>: Psychovisual adaptive
> +  quantization.
> +  You don't want to play with those options if you care about quality.
> +  Reasonable values may be effective in your case, but be warned this is a very
> +  subjective.

Either "this is a very subjective <something>", or "this is very
subjective".

-- 
       The Wanderer

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

A government exists to serve its citizens, not to control them.