(composition_compute_stop_pos): Fix previous change.
[bpt/emacs.git] / src / xfont.c
index 94fe280..67cb6e5 100644 (file)
@@ -1,15 +1,15 @@
 /* xfont.c -- X core font driver.
 /* xfont.c -- X core font driver.
-   Copyright (C) 2006 Free Software Foundation, Inc.
-   Copyright (C) 2006
+   Copyright (C) 2006, 2007, 2008 Free Software Foundation, Inc.
+   Copyright (C) 2006, 2007, 2008
      National Institute of Advanced Industrial Science and Technology (AIST)
      Registration Number H13PRO009
 
 This file is part of GNU Emacs.
 
      National Institute of Advanced Industrial Science and Technology (AIST)
      Registration Number H13PRO009
 
 This file is part of GNU Emacs.
 
-GNU Emacs is free software; you can redistribute it and/or modify
+GNU Emacs is free software: you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
 it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2, or (at your option)
-any later version.
+the Free Software Foundation, either version 3 of the License, or
+(at your option) any later version.
 
 GNU Emacs is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 
 GNU Emacs is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -17,12 +17,11 @@ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
 GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
-along with GNU Emacs; see the file COPYING.  If not, write to
-the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
-Boston, MA 02110-1301, USA.  */
+along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include <config.h>
 #include <stdio.h>
 
 #include <config.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <X11/Xlib.h>
 
 #include "lisp.h"
 #include <X11/Xlib.h>
 
 #include "lisp.h"
@@ -34,57 +33,23 @@ Boston, MA 02110-1301, USA.  */
 #include "charset.h"
 #include "fontset.h"
 #include "font.h"
 #include "charset.h"
 #include "fontset.h"
 #include "font.h"
+#include "ccl.h"
 
 \f
 /* X core font driver.  */
 
 
 \f
 /* X core font driver.  */
 
+struct xfont_info
+{
+  struct font font;
+  Display *display;
+  XFontStruct *xfont;
+};
+
 /* Prototypes of support functions.  */
 extern void x_clear_errors P_ ((Display *));
 
 /* Prototypes of support functions.  */
 extern void x_clear_errors P_ ((Display *));
 
-static char *xfont_query_font P_ ((Display *, char *, Lisp_Object));
 static XCharStruct *xfont_get_pcm P_ ((XFontStruct *, XChar2b *));
 static XCharStruct *xfont_get_pcm P_ ((XFontStruct *, XChar2b *));
-static int xfont_registry_charsets P_ ((Lisp_Object, struct charset **,
-                                       struct charset **));
-
-static char *
-xfont_query_font (display, name, spec)
-     Display *display;
-     char *name;
-     Lisp_Object spec;
-{
-  XFontStruct *font;
-
-  BLOCK_INPUT;
-  x_catch_errors (display);
-  font = XLoadQueryFont (display, name);
-  name = NULL;
-  if (x_had_errors_p (display))
-    {
-      /* This error is perhaps due to insufficient memory on X
-        server.  Let's just ignore it.  */
-      x_clear_errors (display);
-    }
-  else if (font)
-    {
-      unsigned long value;
-
-      if (XGetFontProperty (font, XA_FONT, &value))
-       {
-         char *n = (char *) XGetAtomName (display, (Atom) value);
-
-         if (font_parse_xlfd (n, spec) >= 0)
-           name = n;
-         else
-           XFree (n);
-       }
-      XFreeFont (display, font);
-    }
-  x_uncatch_errors ();
-  UNBLOCK_INPUT;
-
-  return name;
-}
-
+static void xfont_find_ccl_program P_ ((struct font *));
 
 /* Get metrics of character CHAR2B in XFONT.  Value is null if CHAR2B
    is not contained in the font.  */
 
 /* Get metrics of character CHAR2B in XFONT.  Value is null if CHAR2B
    is not contained in the font.  */
@@ -97,7 +62,7 @@ xfont_get_pcm (xfont, char2b)
   /* The result metric information.  */
   XCharStruct *pcm = NULL;
 
   /* The result metric information.  */
   XCharStruct *pcm = NULL;
 
-  xassert (xfont && char2b);
+  font_assert (xfont && char2b);
 
   if (xfont->per_char != NULL)
     {
 
   if (xfont->per_char != NULL)
     {
@@ -154,25 +119,60 @@ xfont_get_pcm (xfont, char2b)
          ? NULL : pcm);
 }
 
          ? NULL : pcm);
 }
 
-static Lisp_Object xfont_get_cache P_ ((Lisp_Object));
+/* Find a CCL program for a font specified by FONTP, and set the member
+ `encoder' of the structure.  */
+
+static void
+xfont_find_ccl_program (font)
+     struct font *font;
+{
+  Lisp_Object list, elt;
+
+  elt = Qnil;
+  for (list = Vfont_ccl_encoder_alist; CONSP (list); list = XCDR (list))
+    {
+      elt = XCAR (list);
+      if (CONSP (elt)
+         && STRINGP (XCAR (elt))
+         && ((fast_string_match_ignore_case (XCAR (elt),
+                                             font->props[FONT_NAME_INDEX])
+              >= 0)
+             || (fast_string_match_ignore_case (XCAR (elt),
+                                                font->props[FONT_FULLNAME_INDEX])
+                 >= 0)))
+       break;
+    }
+
+  if (! NILP (list))
+    {
+      struct ccl_program *ccl
+       = (struct ccl_program *) xmalloc (sizeof (struct ccl_program));
+
+      if (setup_ccl_program (ccl, XCDR (elt)) < 0)
+       xfree (ccl);
+      else
+       font->font_encoder = ccl;
+    }
+}
+
+static Lisp_Object xfont_get_cache P_ ((FRAME_PTR));
 static Lisp_Object xfont_list P_ ((Lisp_Object, Lisp_Object));
 static Lisp_Object xfont_match P_ ((Lisp_Object, Lisp_Object));
 static Lisp_Object xfont_list_family P_ ((Lisp_Object));
 static Lisp_Object xfont_list P_ ((Lisp_Object, Lisp_Object));
 static Lisp_Object xfont_match P_ ((Lisp_Object, Lisp_Object));
 static Lisp_Object xfont_list_family P_ ((Lisp_Object));
-static struct font *xfont_open P_ ((FRAME_PTR, Lisp_Object, int));
+static Lisp_Object xfont_open P_ ((FRAME_PTR, Lisp_Object, int));
 static void xfont_close P_ ((FRAME_PTR, struct font *));
 static int xfont_prepare_face P_ ((FRAME_PTR, struct face *));
 static void xfont_close P_ ((FRAME_PTR, struct font *));
 static int xfont_prepare_face P_ ((FRAME_PTR, struct face *));
-#if 0
-static void xfont_done_face P_ ((FRAME_PTR, struct face *));
-#endif
 static int xfont_has_char P_ ((Lisp_Object, int));
 static unsigned xfont_encode_char P_ ((struct font *, int));
 static int xfont_text_extents P_ ((struct font *, unsigned *, int,
                                   struct font_metrics *));
 static int xfont_draw P_ ((struct glyph_string *, int, int, int, int, int));
 static int xfont_has_char P_ ((Lisp_Object, int));
 static unsigned xfont_encode_char P_ ((struct font *, int));
 static int xfont_text_extents P_ ((struct font *, unsigned *, int,
                                   struct font_metrics *));
 static int xfont_draw P_ ((struct glyph_string *, int, int, int, int, int));
+static int xfont_check P_ ((FRAME_PTR, struct font *));
 
 struct font_driver xfont_driver =
   {
     0,                         /* Qx */
 
 struct font_driver xfont_driver =
   {
     0,                         /* Qx */
+    0,                         /* case insensitive */
     xfont_get_cache,
     xfont_list,
     xfont_match,
     xfont_get_cache,
     xfont_list,
     xfont_match,
@@ -181,26 +181,37 @@ struct font_driver xfont_driver =
     xfont_open,
     xfont_close,
     xfont_prepare_face,
     xfont_open,
     xfont_close,
     xfont_prepare_face,
-    NULL /*xfont_done_face*/,
+    NULL,
     xfont_has_char,
     xfont_encode_char,
     xfont_text_extents,
     xfont_draw,
     xfont_has_char,
     xfont_encode_char,
     xfont_text_extents,
     xfont_draw,
+    NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,
+    xfont_check
   };
 
 extern Lisp_Object QCname;
 
 static Lisp_Object
   };
 
 extern Lisp_Object QCname;
 
 static Lisp_Object
-xfont_get_cache (frame)
-     Lisp_Object frame;
+xfont_get_cache (f)
+     FRAME_PTR f;
 {
 {
-  Display_Info *dpyinfo = FRAME_X_DISPLAY_INFO (XFRAME (frame));
+  Display_Info *dpyinfo = FRAME_X_DISPLAY_INFO (f);
 
   return (dpyinfo->name_list_element);
 }
 
 extern Lisp_Object Vface_alternative_font_registry_alist;
 
 
   return (dpyinfo->name_list_element);
 }
 
 extern Lisp_Object Vface_alternative_font_registry_alist;
 
+static int
+compare_font_names (const void *name1, const void *name2)
+{
+  return xstrcasecmp (*(const unsigned char **) name1,
+                     *(const unsigned char **) name2);
+}
+
+static Lisp_Object xfont_list_pattern P_ ((Lisp_Object, Display *, char *));
+
 static Lisp_Object
 xfont_list_pattern (frame, display, pattern)
      Lisp_Object frame;
 static Lisp_Object
 xfont_list_pattern (frame, display, pattern)
      Lisp_Object frame;
@@ -230,60 +241,63 @@ xfont_list_pattern (frame, display, pattern)
       XFreeFontNames (names);
     }
 
       XFreeFontNames (names);
     }
 
-  for (i = 0; i < num_fonts; i++)
+  if (num_fonts > 0)
     {
     {
-      Lisp_Object entity = Fmake_vector (make_number (FONT_ENTITY_MAX), Qnil);
-      int result;
+      char **indices = alloca (sizeof (char *) * num_fonts);
 
 
-      ASET (entity, FONT_TYPE_INDEX, Qx);
-      ASET (entity, FONT_FRAME_INDEX, frame);
+      for (i = 0; i < num_fonts; i++)
+       indices[i] = names[i];
+      qsort (indices, num_fonts, sizeof (char *), compare_font_names);
 
 
-      result = font_parse_xlfd (names[i], entity);
-      if (result < 0)
+      for (i = 0; i < num_fonts; i++)
        {
        {
-         /* This may be an alias name.  Try to get the full XLFD name
-            from XA_FONT property of the font.  */
-         XFontStruct *font = XLoadQueryFont (display, names[i]);
-         unsigned long value;
+         Lisp_Object entity;
+         int result;
 
 
-         if (! font)
+         if (i > 0 && xstrcasecmp (indices[i - 1], indices[i]) == 0)
            continue;
            continue;
-         if (XGetFontProperty (font, XA_FONT, &value))
-           {
-             char *name = (char *) XGetAtomName (display, (Atom) value);
-             int len = strlen (name);
-
-             /* If DXPC (a Differential X Protocol Compressor)
-                 Ver.3.7 is running, XGetAtomName will return null
-                 string.  We must avoid such a name.  */
-             if (len > 0)
-               result = font_parse_xlfd (name, entity);
-             XFree (name);
-           }
-         XFreeFont (display, font);
-       }
 
 
-      if (result == 0)
-       {
-         Lisp_Object val = AREF (entity, FONT_EXTRA_INDEX);
-         char *p = (char *) SDATA (SYMBOL_NAME (val));
+         entity = font_make_entity ();
+         ASET (entity, FONT_TYPE_INDEX, Qx);
 
 
-         /* P == "RESX-RESY-SPACING-AVGWIDTH.  We rejust this font if
-            it's an autoscaled one (i.e. RESX > 0 && AVGWIDTH == 0).  */
-         if (atoi (p) > 0)
+         result = font_parse_xlfd (indices[i], entity);
+         if (result < 0)
            {
            {
-             p += SBYTES (SYMBOL_NAME (val));
-             while (p[-1] != '-') p--;
-             if (atoi (p) == 0)
+             /* This may be an alias name.  Try to get the full XLFD name
+                from XA_FONT property of the font.  */
+             XFontStruct *font = XLoadQueryFont (display, indices[i]);
+             unsigned long value;
+
+             if (! font)
                continue;
                continue;
+             if (XGetFontProperty (font, XA_FONT, &value))
+               {
+                 char *name = (char *) XGetAtomName (display, (Atom) value);
+                 int len = strlen (name);
+
+                 /* If DXPC (a Differential X Protocol Compressor)
+                    Ver.3.7 is running, XGetAtomName will return null
+                    string.  We must avoid such a name.  */
+                 if (len > 0)
+                   result = font_parse_xlfd (name, entity);
+                 XFree (name);
+               }
+             XFreeFont (display, font);
            }
            }
-         list = Fcons (entity, list);
+
+         if (result == 0
+             /* Avoid auto-scaled fonts.  */
+             && (XINT (AREF (entity, FONT_DPI_INDEX)) == 0
+                 || XINT (AREF (entity, FONT_AVGWIDTH_INDEX)) > 0))
+           list = Fcons (entity, list);
        }
        }
+      XFreeFontNames (names);
     }
 
   x_uncatch_errors ();
   UNBLOCK_INPUT;
 
     }
 
   x_uncatch_errors ();
   UNBLOCK_INPUT;
 
+  font_add_log ("xfont-list", build_string (pattern), list);
   return list;
 }
 
   return list;
 }
 
@@ -293,63 +307,73 @@ xfont_list (frame, spec)
 {
   FRAME_PTR f = XFRAME (frame);
   Display *display = FRAME_X_DISPLAY_INFO (f)->display;
 {
   FRAME_PTR f = XFRAME (frame);
   Display *display = FRAME_X_DISPLAY_INFO (f)->display;
-  Lisp_Object list, val, extra, font_name;
+  Lisp_Object registry, list, val, extra;
   int len;
   char name[256];
   
   extra = AREF (spec, FONT_EXTRA_INDEX);
   int len;
   char name[256];
   
   extra = AREF (spec, FONT_EXTRA_INDEX);
-  font_name = Qnil;
   if (CONSP (extra))
     {
       val = assq_no_quit (QCotf, extra);
       if (! NILP (val))
   if (CONSP (extra))
     {
       val = assq_no_quit (QCotf, extra);
       if (! NILP (val))
-       return null_vector;
+       return Qnil;
       val = assq_no_quit (QCscript, extra);
       if (! NILP (val))
       val = assq_no_quit (QCscript, extra);
       if (! NILP (val))
-       return null_vector;
-      val = assq_no_quit (QClanguage, extra);
+       return Qnil;
+      val = assq_no_quit (QClang, extra);
       if (! NILP (val))
       if (! NILP (val))
-       return null_vector;
-      val = assq_no_quit (QCname, extra);
-      if (CONSP (val))
-       font_name = XCDR (val);
+       return Qnil;
     }
 
     }
 
-  if (STRINGP (font_name)
-      && ! strchr ((char *) SDATA (font_name), ':'))
-    list = xfont_list_pattern (frame, display, (char *) SDATA (font_name));
-  else if ((len = font_unparse_xlfd (spec, 0, name, 256)) < 0)
-    return null_vector;
-  else
+  registry = AREF (spec, FONT_REGISTRY_INDEX);
+  len = font_unparse_xlfd (spec, 0, name, 256);
+  ASET (spec, FONT_REGISTRY_INDEX, registry);
+  if (len < 0)
+    return Qnil;
+  list = xfont_list_pattern (frame, display, name);
+  if (NILP (list) && NILP (registry))
     {
     {
-      list = xfont_list_pattern (frame, display, name);
-      if (NILP (list))
+      /* Try iso10646-1 */
+      char *r = name + len - 9;        /* 9 == strlen (iso8859-1) */
+
+      if (r - name + 10 < 256) /* 10 == strlen (iso10646-1) */
        {
        {
-         Lisp_Object registry = AREF (spec, FONT_REGISTRY_INDEX);
-         Lisp_Object alter;
+         strcpy (r, "iso10646-1");
+         list = xfont_list_pattern (frame, display, name);
+       }
+    }
+  if (NILP (list) && ! NILP (registry))
+    {
+      /* Try alternate registries.  */
+      Lisp_Object alter;
 
 
-         if (! NILP (registry)
-             && (alter = Fassoc (SYMBOL_NAME (registry),
-                                 Vface_alternative_font_registry_alist),
-                 CONSP (alter)))
-           {
-             /* Pointer to REGISTRY-ENCODING field.  */
-             char *r = name + len - SBYTES (SYMBOL_NAME (registry));
-
-             for (alter = XCDR (alter); CONSP (alter); alter = XCDR (alter))
-               if (STRINGP (XCAR (alter))
-                   && ((r - name) + SBYTES (XCAR (alter))) < 255)
-                 {
-                   strcpy (r, (char *) SDATA (XCAR (alter)));
-                   list = xfont_list_pattern (frame, display, name);
-                   if (! NILP (list))
-                     break;
-                 }
-           }
+      if ((alter = Fassoc (SYMBOL_NAME (registry),
+                          Vface_alternative_font_registry_alist),
+          CONSP (alter)))
+       {
+         /* Pointer to REGISTRY-ENCODING field.  */
+         char *r = name + len - SBYTES (SYMBOL_NAME (registry));
+
+         for (alter = XCDR (alter); CONSP (alter); alter = XCDR (alter))
+           if (STRINGP (XCAR (alter))
+               && ((r - name) + SBYTES (XCAR (alter))) < 256)
+             {
+               strcpy (r, (char *) SDATA (XCAR (alter)));
+               list = xfont_list_pattern (frame, display, name);
+               if (! NILP (list))
+                 break;
+             }
        }
     }
        }
     }
+  if (NILP (list))
+    {
+      /* Try alias.  */
+      val = assq_no_quit (QCname, AREF (spec, FONT_EXTRA_INDEX));
+      if (CONSP (val) && STRINGP (XCDR (val)))
+       list = xfont_list_pattern (frame, display, (char *) SDATA (XCDR (val)));
+    }
 
 
-  return (NILP (list) ? null_vector : Fvconcat (1, &list));
+  return list;
 }
 
 static Lisp_Object
 }
 
 static Lisp_Object
@@ -359,18 +383,23 @@ xfont_match (frame, spec)
   FRAME_PTR f = XFRAME (frame);
   Display *display = FRAME_X_DISPLAY_INFO (f)->display;
   Lisp_Object extra, val, entity;
   FRAME_PTR f = XFRAME (frame);
   Display *display = FRAME_X_DISPLAY_INFO (f)->display;
   Lisp_Object extra, val, entity;
-  char *name;
+  char buf[256], *name;
   XFontStruct *xfont;
   unsigned long value;
 
   extra = AREF (spec, FONT_EXTRA_INDEX);
   val = assq_no_quit (QCname, extra);
   if (! CONSP (val) || ! STRINGP (XCDR (val)))
   XFontStruct *xfont;
   unsigned long value;
 
   extra = AREF (spec, FONT_EXTRA_INDEX);
   val = assq_no_quit (QCname, extra);
   if (! CONSP (val) || ! STRINGP (XCDR (val)))
-    return Qnil;
+    {
+      if (font_unparse_xlfd (spec, 0, buf, 256) < 0)
+       return Qnil;
+      name = buf;
+    }
+  else
+    name = (char *) SDATA (XCDR (val));
 
   BLOCK_INPUT;
   entity = Qnil;
 
   BLOCK_INPUT;
   entity = Qnil;
-  name = (char *) SDATA (XCDR (val));
   xfont = XLoadQueryFont (display, name);
   if (xfont)
     {
   xfont = XLoadQueryFont (display, name);
   if (xfont)
     {
@@ -386,9 +415,8 @@ xfont_match (frame, spec)
             string.  We must avoid such a name.  */
          if (len > 0)
            {
             string.  We must avoid such a name.  */
          if (len > 0)
            {
-             entity = Fmake_vector (make_number (FONT_ENTITY_MAX), Qnil);
+             entity = font_make_entity ();
              ASET (entity, FONT_TYPE_INDEX, Qx);
              ASET (entity, FONT_TYPE_INDEX, Qx);
-             ASET (entity, FONT_FRAME_INDEX, frame);
              if (font_parse_xlfd (name, entity) < 0)
                entity = Qnil;
            }
              if (font_parse_xlfd (name, entity) < 0)
                entity = Qnil;
            }
@@ -398,18 +426,10 @@ xfont_match (frame, spec)
     }
   UNBLOCK_INPUT;
 
     }
   UNBLOCK_INPUT;
 
+  font_add_log ("xfont-match", spec, entity);
   return entity;
 }
 
   return entity;
 }
 
-static int
-memq_no_quit (elt, list)
-     Lisp_Object elt, list;
-{
-  while (CONSP (list) && ! EQ (XCAR (list), elt))
-    list = XCDR (list);
-  return (CONSP (list));
-}
-
 static Lisp_Object
 xfont_list_family (frame)
      Lisp_Object frame;
 static Lisp_Object
 xfont_list_family (frame)
      Lisp_Object frame;
@@ -453,8 +473,8 @@ xfont_list_family (frame)
        continue;
       last_len = p1 - p0;
       last_family = p0;
        continue;
       last_len = p1 - p0;
       last_family = p0;
-      family = intern_downcase (p0, last_len);
-      if (! memq_no_quit (family, list))
+      family = font_intern_prop (p0, last_len, 1);
+      if (NILP (assq_no_quit (family, list)))
        list = Fcons (family, list);
     }
 
        list = Fcons (family, list);
     }
 
@@ -465,7 +485,9 @@ xfont_list_family (frame)
   return list;
 }
 
   return list;
 }
 
-static struct font *
+extern Lisp_Object QCavgwidth;
+
+static Lisp_Object
 xfont_open (f, entity, pixel_size)
      FRAME_PTR f;
      Lisp_Object entity;
 xfont_open (f, entity, pixel_size)
      FRAME_PTR f;
      Lisp_Object entity;
@@ -478,20 +500,35 @@ xfont_open (f, entity, pixel_size)
   unsigned long value;
   Lisp_Object registry;
   struct charset *encoding, *repertory;
   unsigned long value;
   Lisp_Object registry;
   struct charset *encoding, *repertory;
+  Lisp_Object font_object, fullname;
   struct font *font;
   XFontStruct *xfont;
   struct font *font;
   XFontStruct *xfont;
+  int i;
 
   /* At first, check if we know how to encode characters for this
      font.  */
   registry = AREF (entity, FONT_REGISTRY_INDEX);
   if (font_registry_charsets (registry, &encoding, &repertory) < 0)
 
   /* At first, check if we know how to encode characters for this
      font.  */
   registry = AREF (entity, FONT_REGISTRY_INDEX);
   if (font_registry_charsets (registry, &encoding, &repertory) < 0)
-    return NULL;
+    {
+      font_add_log ("  x:unknown registry", registry, Qnil);
+      return Qnil;
+    }
 
   if (XINT (AREF (entity, FONT_SIZE_INDEX)) != 0)
     pixel_size = XINT (AREF (entity, FONT_SIZE_INDEX));
 
   if (XINT (AREF (entity, FONT_SIZE_INDEX)) != 0)
     pixel_size = XINT (AREF (entity, FONT_SIZE_INDEX));
+  else if (pixel_size == 0)
+    {
+      if (FRAME_FONT (f))
+       pixel_size = FRAME_FONT (f)->pixel_size;
+      else
+       pixel_size = 14;
+    }
   len = font_unparse_xlfd (entity, pixel_size, name, 256);
   if (len <= 0)
   len = font_unparse_xlfd (entity, pixel_size, name, 256);
   if (len <= 0)
-    return NULL;
+    {
+      font_add_log ("  x:unparse failed", entity, Qnil);
+      return Qnil;
+    }
 
   BLOCK_INPUT;
   x_catch_errors (display);
 
   BLOCK_INPUT;
   x_catch_errors (display);
@@ -503,147 +540,158 @@ xfont_open (f, entity, pixel_size)
       x_clear_errors (display);
       xfont = NULL;
     }
       x_clear_errors (display);
       xfont = NULL;
     }
+  else if (! xfont)
+    {
+      /* Some version of X lists:
+          -misc-fixed-medium-r-normal--20-*-75-75-c-100-iso8859-1
+          -misc-fixed-medium-r-normal--20-*-100-100-c-100-iso8859-1
+        but can open only:
+          -misc-fixed-medium-r-normal--20-*-100-100-c-100-iso8859-1
+        and
+          -misc-fixed-medium-r-normal--20-*-*-*-c-100-iso8859-1
+        So, we try again with wildcards in RESX and RESY.  */
+      Lisp_Object temp;
+
+      temp = Fcopy_font_spec (entity);
+      ASET (temp, FONT_DPI_INDEX, Qnil);
+      len = font_unparse_xlfd (temp, pixel_size, name, 256);
+      if (len <= 0)
+       {
+         font_add_log ("  x:unparse failed", temp, Qnil);
+         return Qnil;
+       }
+      xfont = XLoadQueryFont (display, name);
+      if (x_had_errors_p (display))
+       {
+         /* This error is perhaps due to insufficient memory on X server.
+            Let's just ignore it.  */
+         x_clear_errors (display);
+         xfont = NULL;
+       }
+    }
+  fullname = Qnil;
+  /* Try to get the full name of FONT.  */
+  if (xfont && XGetFontProperty (xfont, XA_FONT, &value))
+    {
+      char *p0, *p;
+      int dashes = 0;
+
+      p0 = p = (char *) XGetAtomName (FRAME_X_DISPLAY (f), (Atom) value);;
+      /* Count the number of dashes in the "full name".
+        If it is too few, this isn't really the font's full name,
+        so don't use it.
+        In X11R4, the fonts did not come with their canonical names
+        stored in them.  */
+      while (*p)
+       {
+         if (*p == '-')
+           dashes++;
+         p++;
+       }
+
+      if (dashes >= 13)
+       fullname = Fdowncase (make_unibyte_string (p0, p - p0));
+      XFree (p0);
+    }
   x_uncatch_errors ();
   UNBLOCK_INPUT;
 
   if (! xfont)
   x_uncatch_errors ();
   UNBLOCK_INPUT;
 
   if (! xfont)
-    return NULL;
-  font = malloc (sizeof (struct font));
-  font->format = Qx;
-  font->font.font = xfont;
-  font->entity = entity;
-  font->pixel_size = pixel_size;
-  font->driver = &xfont_driver;
-  font->font.name = malloc (len + 1);
-  if (! font->font.name)
     {
     {
-      XFreeFont (display, xfont);
-      free (font);
-      return NULL;
+      font_add_log ("  x:open failed", build_string (name), Qnil);
+      return Qnil;
     }
     }
-  bcopy (name, font->font.name, len + 1);
-  font->font.charset = encoding->id;
+
+  font_object = font_make_object (VECSIZE (struct xfont_info),
+                                 entity, pixel_size);
+  ASET (font_object, FONT_TYPE_INDEX, Qx);
+  if (STRINGP (fullname))
+    font_parse_xlfd ((char *) SDATA (fullname), font_object);
+  if (STRINGP (fullname))
+    ASET (font_object, FONT_NAME_INDEX, fullname);
+  else
+    ASET (font_object, FONT_NAME_INDEX, make_unibyte_string (name, len));
+  ASET (font_object, FONT_FULLNAME_INDEX, fullname);
+  ASET (font_object, FONT_FILE_INDEX, Qnil);
+  ASET (font_object, FONT_FORMAT_INDEX, Qx);
+  font = XFONT_OBJECT (font_object);
+  ((struct xfont_info *) font)->xfont = xfont;
+  ((struct xfont_info *) font)->display = FRAME_X_DISPLAY (f);
+  font->pixel_size = pixel_size;
+  font->driver = &xfont_driver;
   font->encoding_charset = encoding->id;
   font->repertory_charset = repertory ? repertory->id : -1;
   font->ascent = xfont->ascent;
   font->descent = xfont->descent;
   font->encoding_charset = encoding->id;
   font->repertory_charset = repertory ? repertory->id : -1;
   font->ascent = xfont->ascent;
   font->descent = xfont->descent;
-
+  font->height = font->ascent + font->descent;
+  font->min_width = xfont->min_bounds.width;
   if (xfont->min_bounds.width == xfont->max_bounds.width)
     {
       /* Fixed width font.  */
   if (xfont->min_bounds.width == xfont->max_bounds.width)
     {
       /* Fixed width font.  */
-      font->font.average_width = font->font.space_width
-       = xfont->min_bounds.width;
+      font->average_width = font->space_width = xfont->min_bounds.width;
     }
   else
     {
     }
   else
     {
-      XChar2b char2b;
       XCharStruct *pcm;
       XCharStruct *pcm;
+      XChar2b char2b;
+      Lisp_Object val;
 
       char2b.byte1 = 0x00, char2b.byte2 = 0x20;
       pcm = xfont_get_pcm (xfont, &char2b);
       if (pcm)
 
       char2b.byte1 = 0x00, char2b.byte2 = 0x20;
       pcm = xfont_get_pcm (xfont, &char2b);
       if (pcm)
-       font->font.space_width = pcm->width;
+       font->space_width = pcm->width;
       else
       else
-       font->font.space_width = xfont->max_bounds.width;
-
-      font->font.average_width
-       = (XGetFontProperty (xfont, dpyinfo->Xatom_AVERAGE_WIDTH, &value)
-          ? (long) value / 10 : 0);
-      if (font->font.average_width < 0)
-       font->font.average_width = - font->font.average_width;
-      if (font->font.average_width == 0)
-       {
-         if (pcm)
-           {
-             int width = pcm->width;
-             for (char2b.byte2 = 33; char2b.byte2 <= 126; char2b.byte2++)
-               if ((pcm = xfont_get_pcm (xfont, &char2b)) != NULL)
-                 width += pcm->width;
-             font->font.average_width = width / 95;
-           }
-         else
-           font->font.average_width = xfont->max_bounds.width;
-       }
-    }
-  font->min_width = xfont->min_bounds.width;
-  if (font->min_width <= 0)
-    font->min_width = font->font.space_width;
-
-  BLOCK_INPUT;
-  /* Try to get the full name of FONT.  Put it in FULL_NAME.  */
-  if (XGetFontProperty (xfont, XA_FONT, &value))
-    {
-      char *full_name = NULL, *p0, *p;
-      int dashes = 0;
-
-      p0 = p = (char *) XGetAtomName (FRAME_X_DISPLAY (f), (Atom) value);;
-      /* Count the number of dashes in the "full name".
-        If it is too few, this isn't really the font's full name,
-        so don't use it.
-        In X11R4, the fonts did not come with their canonical names
-        stored in them.  */
-      while (*p)
+       font->space_width = 0;
+
+      val = Ffont_get (font_object, QCavgwidth);
+      if (INTEGERP (val))
+       font->average_width = XINT (val);
+      if (font->average_width < 0)
+       font->average_width = - font->average_width;
+      if (font->average_width == 0
+         && encoding->ascii_compatible_p)
        {
        {
-         if (*p == '-')
-           dashes++;
-         p++;
-       }
+         int width = font->space_width, n = pcm != NULL;
 
 
-      if (dashes >= 13)
-       {
-         full_name = (char *) malloc (p - p0 + 1);
-         if (full_name)
-           bcopy (p0, full_name, p - p0 + 1);
+         for (char2b.byte2 = 33; char2b.byte2 <= 126; char2b.byte2++)
+           if ((pcm = xfont_get_pcm (xfont, &char2b)) != NULL)
+             width += pcm->width, n++;
+         if (n > 0)
+           font->average_width = width / n;
        }
        }
-      XFree (p0);
-
-      if (full_name)
-       font->font.full_name = full_name;
-      else
-       font->font.full_name = font->font.name;
+      if (font->average_width == 0)
+       /* No easy way other than this to get a reasonable
+          average_width.  */
+       font->average_width
+         = (xfont->min_bounds.width + xfont->max_bounds.width) / 2;
     }
     }
-  font->file_name = NULL;
 
 
-  font->font.size = xfont->max_bounds.width;
-  font->font.height = xfont->ascent + xfont->descent;
-  font->font.baseline_offset
+  BLOCK_INPUT;
+  font->underline_thickness
+    = (XGetFontProperty (xfont, XA_UNDERLINE_THICKNESS, &value)
+       ? (long) value : 0);
+  font->underline_position
+    = (XGetFontProperty (xfont, XA_UNDERLINE_POSITION, &value)
+       ? (long) value : -1);
+  font->baseline_offset
     = (XGetFontProperty (xfont, dpyinfo->Xatom_MULE_BASELINE_OFFSET, &value)
        ? (long) value : 0);
     = (XGetFontProperty (xfont, dpyinfo->Xatom_MULE_BASELINE_OFFSET, &value)
        ? (long) value : 0);
-  font->font.relative_compose
+  font->relative_compose
     = (XGetFontProperty (xfont, dpyinfo->Xatom_MULE_RELATIVE_COMPOSE, &value)
        ? (long) value : 0);
     = (XGetFontProperty (xfont, dpyinfo->Xatom_MULE_RELATIVE_COMPOSE, &value)
        ? (long) value : 0);
-  font->font.default_ascent
+  font->default_ascent
     = (XGetFontProperty (xfont, dpyinfo->Xatom_MULE_DEFAULT_ASCENT, &value)
        ? (long) value : 0);
     = (XGetFontProperty (xfont, dpyinfo->Xatom_MULE_DEFAULT_ASCENT, &value)
        ? (long) value : 0);
-  font->font.vertical_centering
-    = (STRINGP (Vvertical_centering_font_regexp)
-       && (fast_c_string_match_ignore_case
-          (Vvertical_centering_font_regexp, font->font.full_name) >= 0));
-
   UNBLOCK_INPUT;
 
   UNBLOCK_INPUT;
 
-  dpyinfo->n_fonts++;
-
-  /* Set global flag fonts_changed_p to non-zero if the font loaded
-     has a character with a smaller width than any other character
-     before, or if the font loaded has a smaller height than any other
-     font loaded before.  If this happens, it will make a glyph matrix
-     reallocation necessary.  */
-  if (dpyinfo->n_fonts == 1)
-    {
-      dpyinfo->smallest_font_height = font->font.height;
-      dpyinfo->smallest_char_width = font->min_width;
-      fonts_changed_p = 1;
-    }
-  else
-    {
-      if (dpyinfo->smallest_font_height > font->font.height)
-       dpyinfo->smallest_font_height = font->font.height, fonts_changed_p |= 1;
-      if (dpyinfo->smallest_char_width > font->min_width)
-       dpyinfo->smallest_char_width = font->min_width, fonts_changed_p |= 1;
-    }
+  if (NILP (fullname))
+    fullname = AREF (font_object, FONT_NAME_INDEX);
+  font->vertical_centering
+    = (STRINGP (Vvertical_centering_font_regexp)
+       && (fast_string_match_ignore_case
+          (Vvertical_centering_font_regexp, fullname) >= 0));
 
 
-  return font;
+  return font_object;
 }
 
 static void
 }
 
 static void
@@ -652,14 +700,8 @@ xfont_close (f, font)
      struct font *font;
 {
   BLOCK_INPUT;
      struct font *font;
 {
   BLOCK_INPUT;
-  XFreeFont (FRAME_X_DISPLAY (f), font->font.font);
+  XFreeFont (FRAME_X_DISPLAY (f), ((struct xfont_info *) font)->xfont);
   UNBLOCK_INPUT;
   UNBLOCK_INPUT;
-
-  if (font->font.name != font->font.full_name)
-    free (font->font.full_name);
-  free (font->font.name);
-  free (font);
-  FRAME_X_DISPLAY_INFO (f)->n_fonts--;
 }
 
 static int
 }
 
 static int
@@ -668,28 +710,13 @@ xfont_prepare_face (f, face)
      struct face *face;
 {
   BLOCK_INPUT;
      struct face *face;
 {
   BLOCK_INPUT;
-  XSetFont (FRAME_X_DISPLAY (f), face->gc, face->font->fid);
+  XSetFont (FRAME_X_DISPLAY (f), face->gc,
+           ((struct xfont_info *) face->font)->xfont->fid);
   UNBLOCK_INPUT;
 
   return 0;
 }
 
   UNBLOCK_INPUT;
 
   return 0;
 }
 
-#if 0
-static void
-xfont_done_face (f, face)
-     FRAME_PTR f;
-     struct face *face;
-{
-  if (face->extra)
-    {
-      BLOCK_INPUT;
-      XFreeGC (FRAME_X_DISPLAY (f), (GC) face->extra);
-      UNBLOCK_INPUT;
-      face->extra = NULL;
-    }
-}
-#endif /* 0 */
-
 static int
 xfont_has_char (entity, c)
      Lisp_Object entity;
 static int
 xfont_has_char (entity, c)
      Lisp_Object entity;
@@ -710,6 +737,7 @@ xfont_encode_char (font, c)
      struct font *font;
      int c;
 {
      struct font *font;
      int c;
 {
+  XFontStruct *xfont = ((struct xfont_info *) font)->xfont;
   struct charset *charset;
   unsigned code;
   XChar2b char2b;
   struct charset *charset;
   unsigned code;
   XChar2b char2b;
@@ -717,16 +745,16 @@ xfont_encode_char (font, c)
   charset = CHARSET_FROM_ID (font->encoding_charset);
   code = ENCODE_CHAR (charset, c);
   if (code == CHARSET_INVALID_CODE (charset))
   charset = CHARSET_FROM_ID (font->encoding_charset);
   code = ENCODE_CHAR (charset, c);
   if (code == CHARSET_INVALID_CODE (charset))
-    return 0xFFFFFFFF;
+    return FONT_INVALID_CODE;
   if (font->repertory_charset >= 0)
     {
       charset = CHARSET_FROM_ID (font->repertory_charset);
       return (ENCODE_CHAR (charset, c) != CHARSET_INVALID_CODE (charset)
   if (font->repertory_charset >= 0)
     {
       charset = CHARSET_FROM_ID (font->repertory_charset);
       return (ENCODE_CHAR (charset, c) != CHARSET_INVALID_CODE (charset)
-             ? code : 0xFFFFFFFF);
+             ? code : FONT_INVALID_CODE);
     }
   char2b.byte1 = code >> 8;
   char2b.byte2 = code & 0xFF;
     }
   char2b.byte1 = code >> 8;
   char2b.byte2 = code & 0xFF;
-  return (xfont_get_pcm (font->font.font, &char2b) ? code : 0xFFFFFFFF);
+  return (xfont_get_pcm (xfont, &char2b) ? code : FONT_INVALID_CODE);
 }
 
 static int
 }
 
 static int
@@ -736,12 +764,13 @@ xfont_text_extents (font, code, nglyphs, metrics)
      int nglyphs;
      struct font_metrics *metrics;
 {
      int nglyphs;
      struct font_metrics *metrics;
 {
+  XFontStruct *xfont = ((struct xfont_info *) font)->xfont;
   int width = 0;
   int width = 0;
-  int i, x;
+  int i, first, x;
 
   if (metrics)
     bzero (metrics, sizeof (struct font_metrics));
 
   if (metrics)
     bzero (metrics, sizeof (struct font_metrics));
-  for (i = 0, x = 0; i < nglyphs; i++)
+  for (i = 0, x = 0, first = 1; i < nglyphs; i++)
     {
       XChar2b char2b;
       static XCharStruct *pcm;
     {
       XChar2b char2b;
       static XCharStruct *pcm;
@@ -749,17 +778,34 @@ xfont_text_extents (font, code, nglyphs, metrics)
       if (code[i] >= 0x10000)
        continue;
       char2b.byte1 = code[i] >> 8, char2b.byte2 = code[i] & 0xFF;
       if (code[i] >= 0x10000)
        continue;
       char2b.byte1 = code[i] >> 8, char2b.byte2 = code[i] & 0xFF;
-      pcm = xfont_get_pcm (font->font.font, &char2b);
+      pcm = xfont_get_pcm (xfont, &char2b);
       if (! pcm)
        continue;
       if (! pcm)
        continue;
-      if (metrics->lbearing > width + pcm->lbearing)
-       metrics->lbearing = width + pcm->lbearing;
-      if (metrics->rbearing < width + pcm->rbearing)
-       metrics->rbearing = width + pcm->rbearing;
-      if (metrics->ascent < pcm->ascent)
-       metrics->ascent = pcm->ascent;
-      if (metrics->descent < pcm->descent)
-       metrics->descent = pcm->descent;
+      if (first)
+       {
+         if (metrics)
+           {
+             metrics->lbearing = pcm->lbearing;
+             metrics->rbearing = pcm->rbearing;
+             metrics->ascent = pcm->ascent;
+             metrics->descent = pcm->descent;
+           }
+         first = 0;
+       }
+      else
+       {
+         if (metrics)
+           {
+             if (metrics->lbearing > width + pcm->lbearing)
+               metrics->lbearing = width + pcm->lbearing;
+             if (metrics->rbearing < width + pcm->rbearing)
+               metrics->rbearing = width + pcm->rbearing;
+             if (metrics->ascent < pcm->ascent)
+               metrics->ascent = pcm->ascent;
+             if (metrics->descent < pcm->descent)
+               metrics->descent = pcm->descent;
+           }
+       }
       width += pcm->width;
     }
   if (metrics)
       width += pcm->width;
     }
   if (metrics)
@@ -772,49 +818,87 @@ xfont_draw (s, from, to, x, y, with_background)
      struct glyph_string *s;
      int from, to, x, y, with_background;
 {
      struct glyph_string *s;
      int from, to, x, y, with_background;
 {
-  XFontStruct *xfont = s->face->font;
+  XFontStruct *xfont = ((struct xfont_info *) s->font)->xfont;
   int len = to - from;
   GC gc = s->gc;
   int len = to - from;
   GC gc = s->gc;
+  int i;
 
 
-  if (gc != s->face->gc)
+  if (s->gc != s->face->gc)
     {
     {
-      XGCValues xgcv;
-      Display_Info *dpyinfo = FRAME_X_DISPLAY_INFO (s->f);
-
-      XGetGCValues (s->display, gc, GCFont, &xgcv);
-      if (xgcv.font != xfont->fid)
-       XSetFont (s->display, gc, xfont->fid);
+      BLOCK_INPUT;
+      XSetFont (s->display, gc, xfont->fid);
+      UNBLOCK_INPUT;
     }
 
   if (xfont->min_byte1 == 0 && xfont->max_byte1 == 0)
     {
       char *str;
     }
 
   if (xfont->min_byte1 == 0 && xfont->max_byte1 == 0)
     {
       char *str;
-      int i;
       USE_SAFE_ALLOCA;
 
       SAFE_ALLOCA (str, char *, len);
       for (i = 0; i < len ; i++)
        str[i] = XCHAR2B_BYTE2 (s->char2b + from + i);
       USE_SAFE_ALLOCA;
 
       SAFE_ALLOCA (str, char *, len);
       for (i = 0; i < len ; i++)
        str[i] = XCHAR2B_BYTE2 (s->char2b + from + i);
+      BLOCK_INPUT;
       if (with_background > 0)
       if (with_background > 0)
-       XDrawImageString (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
-                         gc, x, y, str, len);
+       {
+         if (s->padding_p)
+           for (i = 0; i < len; i++)
+             XDrawImageString (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                               gc, x + i, y, str + i, 1);
+         else
+           XDrawImageString (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                             gc, x, y, str, len);
+       }
       else
       else
-       XDrawString (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
-                    gc, x, y, str, len);
+       {
+         if (s->padding_p)
+           for (i = 0; i < len; i++)
+             XDrawString (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                          gc, x + i, y, str + i, 1);
+         else
+           XDrawString (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                        gc, x, y, str, len);
+       }
+      UNBLOCK_INPUT;
       SAFE_FREE ();
       return s->nchars;
     }
 
       SAFE_FREE ();
       return s->nchars;
     }
 
+  BLOCK_INPUT;
   if (with_background > 0)
   if (with_background > 0)
-    XDrawImageString16 (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
-                       gc, x, y, s->char2b + from, len);
+    {
+      if (s->padding_p)
+       for (i = 0; i < len; i++)
+         XDrawImageString16 (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                             gc, x + i, y, s->char2b + from + i, 1);
+      else
+       XDrawImageString16 (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                           gc, x, y, s->char2b + from, len);
+    }
   else
   else
-    XDrawString16 (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
-                  gc, x, y, s->char2b + from, len);
+    {
+      if (s->padding_p)
+       for (i = 0; i < len; i++)
+         XDrawString16 (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                        gc, x + i, y, s->char2b + from + i, 1);
+      else
+       XDrawString16 (FRAME_X_DISPLAY (s->f), FRAME_X_WINDOW (s->f),
+                      gc, x, y, s->char2b + from, len);
+    }
+  UNBLOCK_INPUT;
 
   return len;
 }
 
 
   return len;
 }
 
+static int
+xfont_check (f, font)
+     FRAME_PTR f;
+     struct font *font;
+{
+  struct xfont_info *xfont = (struct xfont_info *) font;
+
+  return (FRAME_X_DISPLAY (f) == xfont->display ? 0 : -1);
+}
 
 \f
 void
 
 \f
 void