[MPlayer-DOCS] [PATCH] some ideas

[Diego Biurrun]

On Wed, Aug 29, 2007 at 12:33:40AM -0400, Compn wrote:
> remove mencoder 'planned features' from documentation.xml
> it belongs in wishlist or todo...
> 
> changed 'sub-option' to 'suboption' in -vivo entry for consistency.
> also need to document the suboptions for -vivo (or delete the entry).
> 
> ok to apply?

Yes, see comment below.

> also i was thinking about changing this line in documentation.xml
> but i dont know what to write...
> 
> "MPlayer is a movie player for Linux (runs on many other Unices, and
> non-x86 CPUs, see Ports)."

I've been meaning to rewrite this for centuries.  Suggestions welcome.

> --- DOCS/man/en/mplayer.1	(revision 24280)
> +++ DOCS/man/en/mplayer.1	(working copy)
> @@ -1934,9 +1934,10 @@
>  .
>  .TP
> -.B \-vivo <sub-options> (DEBUG CODE)
> +.B \-vivo <suboption> (DEBUG CODE)
>  Force audio parameters for the VIVO demuxer (for debugging purposes).
>  .
> +FIXME: Document this.

Add the FIXME directly below the option not after the line with the
period.

> --- mplayer/DOCS/xml/en/documentation.xml	(revision 24266)
> +++ mplayer/DOCS/xml/en/documentation.xml	(working copy)
> @@ -161,14 +161,6 @@
>  
> -<itemizedlist>
> -<title>Planned features</title>
> -<listitem><para>
> -  Even wider variety of available en/decoding formats/codecs
> -  (creating VOB files with DivX4/Indeo5/VIVO streams :).
> -</para></listitem>
> -</itemizedlist>

Apply.

Diego