[MPlayer-DOCS] Some install.xml suggestions

Torinthiel torinthiel at megapolis.pl
Mon Nov 27 11:15:09 CET 2006


After the last changes in install.xml, I've reached a few suggestions:

The periods are used inconsistently, as some non-sentences end in
period, most don't. See patch

FreeType description stands out from the croud, as it's formatted
differently. See patch

libjpeg is said to be DEcoder, but using it as vo requires an ENcoder
IMHO

libpng is similar, but here both en/decoding capability is used.

I'm not sure I like the
"optional, non-sentence. At least *.* is required." form, as it mixes
non-sentences and sentences. Not that I have a good alternative.
Torinthiel

-- 
 Waclaw "Torinthiel" Schiller       GG#: 3073512
   torinthiel(at)megapolis(dot)pl
   gpg: 0906A2CE fpr: EE3E DFB4 C4D6 E22E 8999  D714 7CEB CDDC 0906 A2CE
 "No classmates may be used during this examination"
-------------- next part --------------
Index: DOCS/xml/en/install.xml
===================================================================
--- DOCS/xml/en/install.xml	(revision 21283)
+++ DOCS/xml/en/install.xml	(working copy)
@@ -23,7 +23,7 @@
 <itemizedlist>
 <listitem><para>
   <emphasis role="bold">binutils</emphasis> - recommended version is
-  <emphasis role="bold">2.11.x</emphasis>.
+  <emphasis role="bold">2.11.x</emphasis>
 </para></listitem>
 <listitem><para>
   <emphasis role="bold">gcc</emphasis> - recommended versions are 2.95
@@ -42,20 +42,20 @@
   3.79.x or later. To build the XML documentation you need 3.80.
   </para></listitem>
 <listitem><para>
-  <emphasis role="bold">FreeType</emphasis> 2.0.9 or later is required
-  to have a font for the OSD and subtitles.
+  <emphasis role="bold">FreeType</emphasis> - optional,
+  required for font for the OSD and subtitles. At least 2.0.9 is required.
   </para></listitem>
 <listitem><para>
   <emphasis role="bold">libjpeg</emphasis> - optional JPEG decoder,
-  required for the JPEG video output driver.
+  required for the JPEG video output driver
   </para></listitem>
 <listitem><para>
   <emphasis role="bold">libpng</emphasis> - optional (M)PNG decoder,
-  required for the GUI and the PNG video output driver.
+  required for the GUI and the PNG video output driver
   </para></listitem>
 <listitem><para>
   <emphasis role="bold">lame</emphasis> - 3.90 or later is recommended,
-  necessary for encoding MP3 audio with <application>MEncoder</application>.
+  necessary for encoding MP3 audio with <application>MEncoder</application>
   </para></listitem>
 <listitem><para>
   <emphasis role="bold">zlib</emphasis> - recommended, necessary for compressed
@@ -66,7 +66,7 @@
   - optional, needed for some RTSP/RTP streams
   </para></listitem>
 <listitem><para>
-  <emphasis role="bold">directfb</emphasis> - optional, use 0.9.13 or later.
+  <emphasis role="bold">directfb</emphasis> - optional, use 0.9.13 or later
   </para></listitem>
 <listitem><para>
   <emphasis role="bold">cdparanoia</emphasis> - optional, for CDDA support
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-docs/attachments/20061127/4b50f65e/attachment.pgp>


More information about the MPlayer-DOCS mailing list