changeset 32740:01e248c1b369

Remove needless fontid. The parameter fontid isn't used anywhere. The documentation is wrong and very confusing to skin designers.

--- a/DOCS/xml/de/skin.xml	Fri Jan 28 19:06:27 2011 +0000
+++ b/DOCS/xml/de/skin.xml	Sat Jan 29 15:13:55 2011 +0000
@@ -547,15 +547,15 @@
         <varlistentry>
           <term>
             <literal>
-              <anchor id="skin-font"/>font = fontfile, fontid
+              <anchor id="skin-font"/>font = fontfile
             </literal>
           </term>
           <listitem>
             <para>
               Definiert eine Schrift. <literal>fontfile</literal> ist der Name der
               Schrift-Beschreibungsdatei mit der Erweiterung <filename>.fnt</filename>
-              (gieb hier keine Erweiterung an).
-              <literal>fontid</literal> wird verwendet, um auf die Schrift zu verweisen
+              (gib hier keine Erweiterung an) und
+              wird verwendet, um auf die Schrift zu verweisen
               (siehe <link linkend="skin-dlabel">dlabel</link>
               und <link linkend="skin-slabel">slabel</link>). Bis zu 25 Schriften können
               definiert werden.
@@ -566,18 +566,18 @@
         <varlistentry>
           <term>
             <literal>
-              <anchor id="skin-slabel"/>slabel = X, Y, fontid, "text"
+              <anchor id="skin-slabel"/>slabel = X, Y, fontfile, "text"
             </literal>
           </term>
           <listitem>
             <para>
               Platziere ein statisches Label an Position <literal>X,Y</literal>.
               <literal>text</literal> wird mittels der identifizierten
-              <literal>fontid</literal> angezeigt. Der Text ist einfach ein
+              <literal>fontfile</literal> angezeigt. Der Text ist einfach ein
               eine Ausgangszeichenkette (<literal>$x</literal>-Variablen funktionieren nicht),
               eingeschlossen von doppelten Anführungszeichen (das "-Zeichen kann jedoch
               nicht Teil des Textes sein).
-              Das Label wird mittels der über die <literal>fontid</literal> identifizierten
+              Das Label wird mittels der über die <literal>fontfile</literal> identifizierten
               Schrift angezeigt.
             </para>
           </listitem>
@@ -586,7 +586,7 @@
         <varlistentry>
           <term>
             <literal>
-              <anchor id="skin-dlabel"/>dlabel = X, Y, length, align, fontid, "text"
+              <anchor id="skin-dlabel"/>dlabel = X, Y, length, align, fontfile, "text"
             </literal>
           </term>
           <listitem>
@@ -607,7 +607,7 @@
               Der anzuzeigende Text wird mit <literal>text</literal> festgelegt:
               Er muss zwischen doppelten Anführungszeichen stehen (das "-Zeichen kann jedoch
               nicht Teil des Textes sein).
-              Das Label wird mittels der über die <literal>fontid</literal> identifizierten
+              Das Label wird mittels der über die <literal>fontfile</literal> identifizierten
               Schrift angezeigt.
               Du kannst folgende Variablen im Text verwenden:
             </para>

--- a/DOCS/xml/en/skin.xml	Fri Jan 28 19:06:27 2011 +0000
+++ b/DOCS/xml/en/skin.xml	Sat Jan 29 15:13:55 2011 +0000
@@ -446,12 +446,12 @@
 
 <varlistentry>
   <term><literal>
-  <anchor id="skin-font"/>font = fontfile, fontid
+  <anchor id="skin-font"/>font = fontfile
   </literal></term>
   <listitem><para>
   Defines a font. <literal>fontfile</literal> is the name of a font description
   file with a <filename>.fnt</filename> extension (do not specify the extension
-  here).  <literal>fontid</literal> is used to refer to the font
+  here) and is used to refer to the font
   (see <link linkend="skin-dlabel">dlabel</link>
   and <link linkend="skin-slabel">slabel</link>). Up to 25 fonts can be defined.
   </para></listitem>
@@ -459,21 +459,21 @@
 
 <varlistentry>
   <term><literal>
-  <anchor id="skin-slabel"/>slabel = X, Y, fontid, "text"
+  <anchor id="skin-slabel"/>slabel = X, Y, fontfile, "text"
   </literal></term>
   <listitem><para>
   Place a static label at the position <literal>X,Y</literal>.
   <literal>text</literal> is displayed using the font identified by
-  <literal>fontid</literal>. The text is just a raw string
+  <literal>fontfile</literal>. The text is just a raw string
   (<literal>$x</literal> variables do not work) that must be enclosed between
   double quotes (but the " character cannot be part of the text). The
-  label is displayed using the font identified by <literal>fontid</literal>.
+  label is displayed using the font identified by <literal>fontfile</literal>.
   </para></listitem>
 </varlistentry>
 
 <varlistentry>
   <term><literal>
-  <anchor id="skin-dlabel"/>dlabel = X, Y, length, align, fontid, "text"
+  <anchor id="skin-dlabel"/>dlabel = X, Y, length, align, fontfile, "text"
   </literal></term>
   <listitem>
   <para>
@@ -490,7 +490,7 @@
   The text to be displayed is given by <literal>text</literal>: It must be
   written between double quotes (but the " character cannot be part of the
   text). The label is displayed using the font identified by
-  <literal>fontid</literal>. You can use the following variables in the text:
+  <literal>fontfile</literal>. You can use the following variables in the text:
   </para>
 
   <informaltable>

--- a/gui/skin/skin.c	Fri Jan 28 19:06:27 2011 +0000
+++ b/gui/skin/skin.c	Sat Jan 29 15:13:55 2011 +0000
@@ -471,9 +471,8 @@
 }
 
 static int cmd_font( char * in )
-{ // font=fontname,fontid
+{ // font=fontname
  char    name[512];
- char    id[512];
  wItem * item;
 
  CHECKDEFLIST( "font" );
@@ -483,7 +482,6 @@
  CHECK( "menu" );
 
  cutItem( in,name,',',0 );
- cutItem( in,id,',',1 );
 
  mp_dbg( MSGT_GPLAYER,MSGL_DBG2,"\n[skin] font\n" );
  mp_dbg( MSGT_GPLAYER,MSGL_DBG2,"[skin]  name: %s\n",name );
@@ -542,7 +540,7 @@
 }
 
 static int cmd_dlabel( char * in )
-{ // dlabel=x,y,sx,align,fontid,string ...
+{ // dlabel=x,y,sx,align,fontfile,string ...
  char    tmp[512];
  char    sid[63];
  int     x,y,sx,id,a;