[MPlayer-DOCS] Re: Suggested correction in documentation (DVD compliant MPEG-2)

[Giacomo Comes]

On Mon, Dec 11, 2006 at 04:40:58PM +0100, Guillaume POIRIER wrote:
> Hi,
> 
> On 12/2/06, Diego Biurrun <diego at biurrun.de> wrote:
> >On Tue, Nov 28, 2006 at 08:09:15PM +0100, Reimar Döffinger wrote:
> >> On Tue, Nov 28, 2006 at 07:47:12PM +0100, Diego Biurrun wrote:
> >> > On Sun, Nov 26, 2006 at 03:36:24AM +0200, Eli Billauer wrote:
> >> > >
> >> > > I would suggest to update the page at
> >> > >
> >> > > http://www.mplayerhq.hu/DOCS/HTML/en/menc-feat-vcd-dvd.html
> >> > >
> >> > > so it states that vstrict=1 should be used for creating DVDs. Without
> >> > > this option, mencoder creates a stream which makes problem with some 
> >(or
> >> > > all?) standalone DVD players.
> >>
> >> vstrict=0 seems to be enough, though vstrict=1 probably won't hurt.
> >
> >Where are the MEncoder docs maintainers to fix this? :)
> 
> where ARE they? There's only me listed as a maintainer of this part of
> the docs! ;-)
> 
> Attached patch adds vstrict wherever it seems to be needed, please
> re-read in case I have missed smth, and also to fix my broken English.
> 

vstrict=1 is not necessary (altought allowed). vstrict=0 is enough.


> Index: DOCS/xml/en/encoding-guide.xml
> ===================================================================
> --- DOCS/xml/en/encoding-guide.xml	(revision 21592)
> +++ DOCS/xml/en/encoding-guide.xml	(working copy)
> @@ -4645,6 +4645,13 @@
>    use vbitrate=9800, but be warned that this could constrain you to less
>    than an hour of video on a single-layer DVD.
>  </para></listitem>
> +<listitem><para>
> +  <emphasis role="bold">vtrict</emphasis>:
> +  <option>vstrict</option>=1 should be used to create DVDs.
> +  Without this option, <application>MEncoder</application> creates a
> +  stream that cannot be decoded by most, if not all, standalone DVD

I would say:
     stream that cannot be correctly decoded by some standalone DVD

Giacomo

> +  players.
> +</para></listitem>
>  </itemizedlist>
>  </sect3>
>  

[...]