This commit was generated by cvs2svn to compensate for changes in r4969,
[chise/xemacs-chise.git.1] / src / faces.c
index e87d934..750055c 100644 (file)
@@ -36,6 +36,7 @@ Boston, MA 02111-1307, USA.  */
 #include "faces.h"
 #include "frame.h"
 #include "glyphs.h"
+#include "hash.h"
 #include "objects.h"
 #include "specifier.h"
 #include "window.h"
@@ -55,10 +56,11 @@ Lisp_Object Qinit_global_faces;
    calling Ffind_face. */
 Lisp_Object Vdefault_face, Vmodeline_face, Vgui_element_face;
 Lisp_Object Vleft_margin_face, Vright_margin_face, Vtext_cursor_face;
-Lisp_Object Vpointer_face, Vvertical_divider_face, Vtoolbar_face, Vwidget_face;
+Lisp_Object Vpointer_face, Vvertical_divider_face, Vtoolbar_face;
 
-/* Qdefault, Qhighlight, Qleft_margin, Qright_margin defined in general.c */
-Lisp_Object Qmodeline, Qgui_element, Qtext_cursor, Qvertical_divider;
+/* Qdefault, Qhighlight defined in general.c */
+Lisp_Object Qmodeline, Qgui_element, Qleft_margin, Qright_margin, Qtext_cursor;
+Lisp_Object Qvertical_divider;
 
 /* In the old implementation Vface_list was a list of the face names,
    not the faces themselves.  We now distinguish between permanent and
@@ -72,26 +74,26 @@ Lisp_Object Vbuilt_in_face_specifiers;
 \f
 
 static Lisp_Object
-mark_face (Lisp_Object obj)
+mark_face (Lisp_Object obj, void (*markobj) (Lisp_Object))
 {
-  Lisp_Face *face =  XFACE (obj);
+  struct Lisp_Face *face =  XFACE (obj);
 
-  mark_object (face->name);
-  mark_object (face->doc_string);
+  ((markobj) (face->name));
+  ((markobj) (face->doc_string));
 
-  mark_object (face->foreground);
-  mark_object (face->background);
-  mark_object (face->font);
-  mark_object (face->display_table);
-  mark_object (face->background_pixmap);
-  mark_object (face->underline);
-  mark_object (face->strikethru);
-  mark_object (face->highlight);
-  mark_object (face->dim);
-  mark_object (face->blinking);
-  mark_object (face->reverse);
+  ((markobj) (face->foreground));
+  ((markobj) (face->background));
+  ((markobj) (face->font));
+  ((markobj) (face->display_table));
+  ((markobj) (face->background_pixmap));
+  ((markobj) (face->underline));
+  ((markobj) (face->strikethru));
+  ((markobj) (face->highlight));
+  ((markobj) (face->dim));
+  ((markobj) (face->blinking));
+  ((markobj) (face->reverse));
 
-  mark_object (face->charsets_warned_about);
+  ((markobj) (face->charsets_warned_about));
 
   return face->plist;
 }
@@ -99,7 +101,7 @@ mark_face (Lisp_Object obj)
 static void
 print_face (Lisp_Object obj, Lisp_Object printcharfun, int escapeflag)
 {
-  Lisp_Face *face = XFACE (obj);
+  struct Lisp_Face *face = XFACE (obj);
 
   if (print_readably)
     {
@@ -127,10 +129,10 @@ print_face (Lisp_Object obj, Lisp_Object printcharfun, int escapeflag)
    This isn't concerned with "unspecified" attributes, that's what
    #'face-differs-from-default-p is for. */
 static int
-face_equal (Lisp_Object obj1, Lisp_Object obj2, int depth)
+face_equal (Lisp_Object o1, Lisp_Object o2, int depth)
 {
-  Lisp_Face *f1 = XFACE (obj1);
-  Lisp_Face *f2 = XFACE (obj2);
+  struct Lisp_Face *f1 = XFACE (o1);
+  struct Lisp_Face *f2 = XFACE (o2);
 
   depth++;
 
@@ -153,7 +155,7 @@ face_equal (Lisp_Object obj1, Lisp_Object obj2, int depth)
 static unsigned long
 face_hash (Lisp_Object obj, int depth)
 {
-  Lisp_Face *f = XFACE (obj);
+  struct Lisp_Face *f = XFACE (obj);
 
   depth++;
 
@@ -167,28 +169,28 @@ face_hash (Lisp_Object obj, int depth)
 static Lisp_Object
 face_getprop (Lisp_Object obj, Lisp_Object prop)
 {
-  Lisp_Face *f = XFACE (obj);
+  struct Lisp_Face *f = XFACE (obj);
 
   return
-    (EQ (prop, Qforeground)       ? f->foreground        :
-     EQ (prop, Qbackground)       ? f->background        :
-     EQ (prop, Qfont)             ? f->font              :
-     EQ (prop, Qdisplay_table)    ? f->display_table     :
-     EQ (prop, Qbackground_pixmap) ? f->background_pixmap :
-     EQ (prop, Qunderline)        ? f->underline         :
-     EQ (prop, Qstrikethru)       ? f->strikethru        :
-     EQ (prop, Qhighlight)        ? f->highlight         :
-     EQ (prop, Qdim)              ? f->dim               :
-     EQ (prop, Qblinking)         ? f->blinking          :
-     EQ (prop, Qreverse)          ? f->reverse           :
-     EQ (prop, Qdoc_string)       ? f->doc_string        :
+    ((EQ (prop, Qforeground))       ? f->foreground        :
+     (EQ (prop, Qbackground))       ? f->background        :
+     (EQ (prop, Qfont))                     ? f->font              :
+     (EQ (prop, Qdisplay_table))     ? f->display_table            :
+     (EQ (prop, Qbackground_pixmap)) ? f->background_pixmap :
+     (EQ (prop, Qunderline))        ? f->underline         :
+     (EQ (prop, Qstrikethru))       ? f->strikethru        :
+     (EQ (prop, Qhighlight))        ? f->highlight         :
+     (EQ (prop, Qdim))              ? f->dim               :
+     (EQ (prop, Qblinking))         ? f->blinking          :
+     (EQ (prop, Qreverse))          ? f->reverse           :
+     (EQ (prop, Qdoc_string))       ? f->doc_string        :
      external_plist_get (&f->plist, prop, 0, ERROR_ME));
 }
 
 static int
 face_putprop (Lisp_Object obj, Lisp_Object prop, Lisp_Object value)
 {
-  Lisp_Face *f = XFACE (obj);
+  struct Lisp_Face *f = XFACE (obj);
 
   if (EQ (prop, Qforeground)        ||
       EQ (prop, Qbackground)        ||
@@ -218,7 +220,7 @@ face_putprop (Lisp_Object obj, Lisp_Object prop, Lisp_Object value)
 static int
 face_remprop (Lisp_Object obj, Lisp_Object prop)
 {
-  Lisp_Face *f = XFACE (obj);
+  struct Lisp_Face *f = XFACE (obj);
 
   if (EQ (prop, Qforeground)        ||
       EQ (prop, Qbackground)        ||
@@ -245,7 +247,7 @@ face_remprop (Lisp_Object obj, Lisp_Object prop)
 static Lisp_Object
 face_plist (Lisp_Object obj)
 {
-  Lisp_Face *face = XFACE (obj);
+  struct Lisp_Face *face = XFACE (obj);
   Lisp_Object result = face->plist;
 
   result = cons3 (Qreverse,          face->reverse,           result);
@@ -263,30 +265,11 @@ face_plist (Lisp_Object obj)
   return result;
 }
 
-static const struct lrecord_description face_description[] = {
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, name) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, doc_string) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, foreground) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, background) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, font) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, display_table) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, background_pixmap) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, underline) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, strikethru) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, highlight) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, dim) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, blinking) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, reverse) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, plist) },
-  { XD_LISP_OBJECT, offsetof (Lisp_Face, charsets_warned_about) },
-  { XD_END }
-};
-
 DEFINE_LRECORD_IMPLEMENTATION_WITH_PROPS ("face", face,
                                          mark_face, print_face, 0, face_equal,
-                                         face_hash, face_description, face_getprop,
+                                         face_hash, face_getprop,
                                          face_putprop, face_remprop,
-                                         face_plist, Lisp_Face);
+                                         face_plist, struct Lisp_Face);
 \f
 /************************************************************************/
 /*                             face read syntax                         */
@@ -352,7 +335,7 @@ face_instantiate (Lisp_Object data)
  ****************************************************************************/
 
 static void
-reset_face (Lisp_Face *f)
+reset_face (struct Lisp_Face *f)
 {
   f->name = Qnil;
   f->doc_string = Qnil;
@@ -372,10 +355,11 @@ reset_face (Lisp_Face *f)
   f->charsets_warned_about = Qnil;
 }
 
-static Lisp_Face *
+static struct Lisp_Face *
 allocate_face (void)
 {
-  Lisp_Face *result = alloc_lcrecord_type (Lisp_Face, &lrecord_face);
+  struct Lisp_Face *result =
+    alloc_lcrecord_type (struct Lisp_Face, lrecord_face);
 
   reset_face (result);
   return result;
@@ -391,14 +375,19 @@ struct face_list_closure
 };
 
 static int
-add_face_to_list_mapper (Lisp_Object key, Lisp_Object value,
+add_face_to_list_mapper (CONST void *hash_key, void *hash_contents,
                         void *face_list_closure)
 {
   /* This function can GC */
+  Lisp_Object key, contents;
+  Lisp_Object *face_list;
   struct face_list_closure *fcl =
     (struct face_list_closure *) face_list_closure;
+  CVOID_TO_LISP (key, hash_key);
+  VOID_TO_LISP (contents, hash_contents);
+  face_list = fcl->face_list;
 
-  *(fcl->face_list) = Fcons (XFACE (value)->name, (*fcl->face_list));
+  *face_list = Fcons (XFACE (contents)->name, *face_list);
   return 0;
 }
 
@@ -431,12 +420,15 @@ temporary_faces_list (void)
 
 \f
 static int
-mark_face_as_clean_mapper (Lisp_Object key, Lisp_Object value,
+mark_face_as_clean_mapper (CONST void *hash_key, void *hash_contents,
                           void *flag_closure)
 {
   /* This function can GC */
+  Lisp_Object key, contents;
   int *flag = (int *) flag_closure;
-  XFACE (value)->dirty = *flag;
+  CVOID_TO_LISP (key, hash_key);
+  VOID_TO_LISP (contents, hash_contents);
+  XFACE (contents)->dirty = *flag;
   return 0;
 }
 
@@ -496,7 +488,7 @@ update_inheritance_mapper_internal (Lisp_Object cur_face,
 }
 
 static int
-update_face_inheritance_mapper (const void *hash_key, void *hash_contents,
+update_face_inheritance_mapper (CONST void *hash_key, void *hash_contents,
                                void *face_inheritance_closure)
 {
   Lisp_Object key, contents;
@@ -596,7 +588,7 @@ face_property_matching_instance (Lisp_Object face, Lisp_Object property,
 
 \f
 DEFUN ("facep", Ffacep, 1, 1, 0, /*
-Return t if OBJECT is a face.
+Return non-nil if OBJECT is a face.
 */
        (object))
 {
@@ -762,15 +754,15 @@ other non-nil value both permanent and temporary are included.
 }
 
 DEFUN ("make-face", Fmake_face, 1, 3, 0, /*
-Define a new face with name NAME (a symbol), described by DOC-STRING.
-You can modify the font, color, etc. of a face with the set-face-* functions.
+Define and return a new FACE described by DOC-STRING.
+You can modify the font, color, etc of a face with the set-face-* functions.
 If the face already exists, it is unmodified.
 If TEMPORARY is non-nil, this face will cease to exist if not in use.
 */
        (name, doc_string, temporary))
 {
   /* This function can GC if initialized is non-zero */
-  Lisp_Face *f;
+  struct Lisp_Face *f;
   Lisp_Object face;
 
   CHECK_SYMBOL (name);
@@ -998,7 +990,8 @@ Here's an approach that should keep things clean and unconfused:
 /* mark for GC a dynarr of face cachels. */
 
 void
-mark_face_cachels (face_cachel_dynarr *elements)
+mark_face_cachels (face_cachel_dynarr *elements,
+                  void (*markobj) (Lisp_Object))
 {
   int elt;
 
@@ -1014,13 +1007,13 @@ mark_face_cachels (face_cachel_dynarr *elements)
 
        for (i = 0; i < NUM_LEADING_BYTES; i++)
          if (!NILP (cachel->font[i]) && !UNBOUNDP (cachel->font[i]))
-           mark_object (cachel->font[i]);
+           ((markobj) (cachel->font[i]));
       }
-      mark_object (cachel->face);
-      mark_object (cachel->foreground);
-      mark_object (cachel->background);
-      mark_object (cachel->display_table);
-      mark_object (cachel->background_pixmap);
+      ((markobj) (cachel->face));
+      ((markobj) (cachel->foreground));
+      ((markobj) (cachel->background));
+      ((markobj) (cachel->display_table));
+      ((markobj) (cachel->background_pixmap));
     }
 }
 
@@ -1138,7 +1131,7 @@ face_cachel_charset_font_metric_info (struct face_cachel *cachel,
        {
          Lisp_Object charset = CHARSET_BY_LEADING_BYTE (i + MIN_LEADING_BYTE);
          Lisp_Object font_instance = FACE_CACHEL_FONT (cachel, charset);
-         Lisp_Font_Instance *fi = XFONT_INSTANCE (font_instance);
+         struct Lisp_Font_Instance *fi = XFONT_INSTANCE (font_instance);
 
          assert (CHARSETP (charset));
          assert (FONT_INSTANCEP (font_instance));
@@ -1154,6 +1147,22 @@ face_cachel_charset_font_metric_info (struct face_cachel *cachel,
     }
 }
 
+/* Called when the updated flag has been cleared on a cachel. */
+
+void
+update_face_cachel_data (struct face_cachel *cachel,
+                        Lisp_Object domain,
+                        Lisp_Object face)
+{
+  if (XFACE (face)->dirty || UNBOUNDP (cachel->face))
+    {
+      int default_face = EQ (face, Vdefault_face);
+      cachel->face = face;
+
+      /* We normally only set the _specified flags if the value was
+         actually bound.  The exception is for the default face where
+         we always set it since it is the ultimate fallback. */
+
 #define FROB(field)                                                         \
   do {                                                                      \
     Lisp_Object new_val =                                                   \
@@ -1172,125 +1181,31 @@ face_cachel_charset_font_metric_info (struct face_cachel *cachel,
     cachel->field##_specified = (bound || default_face);                    \
   } while (0)
 
-/*
- * A face's background pixmap will override the face's
- * background color.  But the background pixmap of the
- * default face should not override the background color of
- * a face if the background color has been specified or
- * inherited.
- *
- * To accomplish this we remove the background pixmap of the
- * cachel and mark it as having been specified so that cachel
- * merging won't override it later.
- */
-#define MAYBE_UNFROB_BACKGROUND_PIXMAP          \
-do                                              \
-{                                               \
-  if (! default_face                            \
-      && cachel->background_specified           \
-      && ! cachel->background_pixmap_specified) \
-    {                                           \
-      cachel->background_pixmap = Qunbound;     \
-      cachel->background_pixmap_specified = 1;  \
-    }                                           \
-} while (0)
-
-
-/* Add a cachel for the given face to the given window's cache. */
-
-static void
-add_face_cachel (struct window *w, Lisp_Object face)
-{
-  int must_finish_frobbing = ! WINDOW_FACE_CACHEL (w, DEFAULT_INDEX);
-  struct face_cachel new_cachel;
-  Lisp_Object domain;
-
-  reset_face_cachel (&new_cachel);
-  XSETWINDOW (domain, w);
-  update_face_cachel_data (&new_cachel, domain, face);
-  Dynarr_add (w->face_cachels, new_cachel);
-
-  /* The face's background pixmap have not yet been frobbed (see comment
-     int update_face_cachel_data), so we have to do it now */
-  if (must_finish_frobbing)
-    {
-      int default_face = EQ (face, Vdefault_face);
-      struct face_cachel *cachel
-       = Dynarr_atp (w->face_cachels, Dynarr_length (w->face_cachels) - 1);
-
-      FROB (background_pixmap);
-      MAYBE_UNFROB_BACKGROUND_PIXMAP;
-    }
-}
-
-/* Called when the updated flag has been cleared on a cachel.
-   This function returns 1 if the caller must finish the update (see comment
-   below), 0 otherwise.
-*/
-
-void
-update_face_cachel_data (struct face_cachel *cachel,
-                        Lisp_Object domain,
-                        Lisp_Object face)
-{
-  if (XFACE (face)->dirty || UNBOUNDP (cachel->face))
-    {
-      int default_face = EQ (face, Vdefault_face);
-      cachel->face = face;
-
-      /* We normally only set the _specified flags if the value was
-         actually bound.  The exception is for the default face where
-         we always set it since it is the ultimate fallback. */
-
       FROB (foreground);
       FROB (background);
       FROB (display_table);
+      FROB (background_pixmap);
 
-      /* #### WARNING: the background pixmap property of faces is currently
-        the only one dealing with images. The problem we have here is that
-        frobbing the background pixmap might lead to image instantiation
-        which in turn might require that the cache we're building be up to
-        date, hence a crash. Here's a typical scenario of this:
-
-        - a new window is created and it's face cache elements are
-        initialized through a call to reset_face_cachels[1]. At that point,
-        the cache for the default and modeline faces (normaly taken care of
-        by redisplay itself) are null.
-        - the default face has a background pixmap which needs to be
-        instantiated right here, as a consequence of cache initialization.
-        - the background pixmap image happens to be instantiated as a string
-        (this happens on tty's for instance).
-        - In order to do this, we need to compute the string geometry.
-        - In order to do this, we might have to access the window's default
-        face cache. But this is the cache we're building right now, it is
-        null.
-        - BARF !!!!!
-
-        To sum up, this means that it is in general unsafe to instantiate
-        images before face cache updating is complete (appart from image
-        related face attributes). The solution we use below is to actually
-        detect whether we're building the window's face_cachels for the first
-        time, and simply NOT frob the background pixmap in that case. If
-        other image-related face attributes are ever implemented, they should
-        be protected the same way right here.
-
-        One note:
-        * See comment in `default_face_font_info' in face.c. Who wrote it ?
-        Maybe we have the begining of an answer here ?
-
-        Footnotes:
-        [1] See comment at the top of `allocate_window' in window.c.
-
-        -- didier
-      */
-      if (! WINDOWP (domain)
-         || WINDOW_FACE_CACHEL (DOMAIN_XWINDOW (domain), DEFAULT_INDEX))
+      /*
+       * A face's background pixmap will override the face's
+       * background color.  But the background pixmap of the
+       * default face should not override the background color of
+       * a face if the background color has been specified or
+       * inherited.
+       *
+       * To accomplish this we remove the background pixmap of the
+       * cachel and mark it as having been specified so that cachel
+       * merging won't override it later.
+       */
+      if (! default_face
+         && cachel->background_specified
+         && ! cachel->background_pixmap_specified)
        {
-         FROB (background_pixmap);
-         MAYBE_UNFROB_BACKGROUND_PIXMAP;
+         cachel->background_pixmap = Qunbound;
+         cachel->background_pixmap_specified = 1;
        }
+
 #undef FROB
-#undef MAYBE_UNFROB_BACKGROUND_PIXMAP
 
       ensure_face_cachel_contains_charset (cachel, domain, Vcharset_ascii);
 
@@ -1395,6 +1310,20 @@ reset_face_cachel (struct face_cachel *cachel)
   cachel->background_pixmap = Qunbound;
 }
 
+/* Add a cachel for the given face to the given window's cache. */
+
+static void
+add_face_cachel (struct window *w, Lisp_Object face)
+{
+  struct face_cachel new_cachel;
+  Lisp_Object window;
+
+  reset_face_cachel (&new_cachel);
+  XSETWINDOW (window, w);
+  update_face_cachel_data (&new_cachel, window, face);
+  Dynarr_add (w->face_cachels, new_cachel);
+}
+
 /* Retrieve the index to a cachel for window W that corresponds to
    the specified face.  If necessary, add a new element to the
    cache. */
@@ -1639,16 +1568,7 @@ get_extent_fragment_face_cache_index (struct window *w,
       findex = get_builtin_face_cache_index (w, Vdefault_face);
       merge_face_cachel_data (w, findex, &cachel);
 
-      findex = get_merged_face_cache_index (w, &cachel);
-      if (cachel.merged_faces &&
-         /* merged_faces did not get stored and available via return value */
-         Dynarr_at (w->face_cachels, findex).merged_faces !=
-         cachel.merged_faces)
-       {
-         Dynarr_free (cachel.merged_faces);
-         cachel.merged_faces = 0;
-       }
-      return findex;
+      return get_merged_face_cache_index (w, &cachel);
     }
 }
 
@@ -1718,19 +1638,23 @@ face_property_was_changed (Lisp_Object face, Lisp_Object property,
 
   if (WINDOWP (locale))
     {
-      MARK_FRAME_FACES_CHANGED (XFRAME (XWINDOW (locale)->frame));
+      struct frame *f = XFRAME (XWINDOW (locale)->frame);
+      MARK_FRAME_FACES_CHANGED (f);
     }
   else if (FRAMEP (locale))
     {
-      MARK_FRAME_FACES_CHANGED (XFRAME (locale));
+      struct frame *f = XFRAME (locale);
+      MARK_FRAME_FACES_CHANGED (f);
     }
   else if (DEVICEP (locale))
     {
-      MARK_DEVICE_FRAMES_FACES_CHANGED (XDEVICE (locale));
+      struct device *d = XDEVICE (locale);
+      MARK_DEVICE_FRAMES_FACES_CHANGED (d);
     }
   else
     {
       Lisp_Object devcons, concons;
+
       DEVICE_LOOP_NO_BREAK (devcons, concons)
        MARK_DEVICE_FRAMES_FACES_CHANGED (XDEVICE (XCAR (devcons)));
     }
@@ -1763,7 +1687,7 @@ LOCALE, TAG-SET, EXACT-P, and HOW-TO-ADD are as in `copy-specifier'.
 */
        (old_face, new_name, locale, tag_set, exact_p, how_to_add))
 {
-  Lisp_Face *fold, *fnew;
+  struct Lisp_Face *fold, *fnew;
   Lisp_Object new_face = Qnil;
   struct gcpro gcpro1, gcpro2, gcpro3, gcpro4;
 
@@ -1825,11 +1749,11 @@ LOCALE, TAG-SET, EXACT-P, and HOW-TO-ADD are as in `copy-specifier'.
 void
 syms_of_faces (void)
 {
-  INIT_LRECORD_IMPLEMENTATION (face);
-
-  /* Qdefault, Qwidget, Qleft_margin, Qright_margin defined in general.c */
+  /* Qdefault defined in general.c */
   defsymbol (&Qmodeline, "modeline");
   defsymbol (&Qgui_element, "gui-element");
+  defsymbol (&Qleft_margin, "left-margin");
+  defsymbol (&Qright_margin, "right-margin");
   defsymbol (&Qtext_cursor, "text-cursor");
   defsymbol (&Qvertical_divider, "vertical-divider");
 
@@ -1882,8 +1806,6 @@ vars_of_faces (void)
   Vdefault_face = Qnil;
   staticpro (&Vgui_element_face);
   Vgui_element_face = Qnil;
-  staticpro (&Vwidget_face);
-  Vwidget_face = Qnil;
   staticpro (&Vmodeline_face);
   Vmodeline_face = Qnil;
   staticpro (&Vtoolbar_face);
@@ -1916,7 +1838,7 @@ vars_of_faces (void)
     syms[n++] = Qblinking;
     syms[n++] = Qreverse;
 
-    Vbuilt_in_face_specifiers = Flist (n, syms);
+    Vbuilt_in_face_specifiers = pure_list (n, syms);
     staticpro (&Vbuilt_in_face_specifiers);
   }
 }
@@ -1924,10 +1846,10 @@ vars_of_faces (void)
 void
 complex_vars_of_faces (void)
 {
-  Vpermanent_faces_cache =
-    make_lisp_hash_table (10, HASH_TABLE_NON_WEAK, HASH_TABLE_EQ);
-  Vtemporary_faces_cache =
-    make_lisp_hash_table (0, HASH_TABLE_WEAK, HASH_TABLE_EQ);
+  Vpermanent_faces_cache = make_lisp_hashtable (10, HASHTABLE_NONWEAK,
+                                               HASHTABLE_EQ);
+  Vtemporary_faces_cache = make_lisp_hashtable (0, HASHTABLE_WEAK,
+                                               HASHTABLE_EQ);
 
   /* Create the default face now so we know what it is immediately. */
 
@@ -1951,8 +1873,6 @@ complex_vars_of_faces (void)
     bg_fb = acons (list1 (Qtty), Fvector (0, 0), bg_fb);
 #endif
 #ifdef HAVE_MS_WINDOWS
-    fg_fb = acons (list1 (Qmsprinter), build_string ("black"), fg_fb);
-    bg_fb = acons (list1 (Qmsprinter), build_string ("white"), bg_fb);
     fg_fb = acons (list1 (Qmswindows), build_string ("black"), fg_fb);
     bg_fb = acons (list1 (Qmswindows), build_string ("white"), bg_fb);
 #endif
@@ -1969,7 +1889,7 @@ complex_vars_of_faces (void)
        (#### Perhaps we should remove the stuff from x-faces.el
        and only depend on this stuff here?  That should work.)
      */
-    const char *fonts[] =
+    CONST char *fonts[] =
     {
       "-*-courier-medium-r-*-*-*-120-*-*-*-*-iso8859-*",
       "-*-courier-medium-r-*-*-*-120-*-*-*-*-iso8859-*",
@@ -1987,7 +1907,7 @@ complex_vars_of_faces (void)
       "-*-*-*-*-*-*-*-120-*-*-*-*-*-*",
       "*"
     };
-    const char **fontptr;
+    CONST char **fontptr;
 
     for (fontptr = fonts + countof(fonts) - 1; fontptr >= fonts; fontptr--)
       inst_list = Fcons (Fcons (list1 (Qx), build_string (*fontptr)),
@@ -1999,18 +1919,8 @@ complex_vars_of_faces (void)
                       inst_list);
 #endif /* HAVE_TTY */
 #ifdef HAVE_MS_WINDOWS
-    /* Fixedsys does not exist for printers */
-    inst_list = Fcons (Fcons (list1 (Qmsprinter),
-                      build_string ("Courier:Regular:10::Western")), inst_list);
-    inst_list = Fcons (Fcons (list1 (Qmsprinter),
-                      build_string ("Courier New:Regular:10::Western")), inst_list);
-
-    inst_list = Fcons (Fcons (list1 (Qmswindows),
-                      build_string ("Fixedsys:Regular:9::Western")), inst_list);
-    inst_list = Fcons (Fcons (list1 (Qmswindows),
-                      build_string ("Courier:Regular:10::Western")), inst_list);
-    inst_list = Fcons (Fcons (list1 (Qmswindows),
-                      build_string ("Courier New:Regular:10::Western")), inst_list);
+    inst_list = Fcons (Fcons (list1 (Qmswindows), build_string ("Courier New")),
+                      inst_list);
 #endif /* HAVE_MS_WINDOWS */
     set_specifier_fallback (Fget (Vdefault_face, Qfont, Qnil), inst_list);
   }
@@ -2027,7 +1937,7 @@ complex_vars_of_faces (void)
                         list1 (Fcons (Qnil, Qnil)));
   set_specifier_fallback (Fget (Vdefault_face, Qreverse, Qnil),
                         list1 (Fcons (Qnil, Qnil)));
-
+  
   /* gui-element is the parent face of all gui elements such as
      modeline, vertical divider and toolbar. */
   Vgui_element_face = Fmake_face (Qgui_element,
@@ -2048,8 +1958,6 @@ complex_vars_of_faces (void)
     bg_fb = acons (list1 (Qtty), Fvector (0, 0), bg_fb);
 #endif
 #ifdef HAVE_MS_WINDOWS
-    fg_fb = acons (list1 (Qmsprinter), build_string ("black"), fg_fb);
-    bg_fb = acons (list1 (Qmsprinter), build_string ("white"), bg_fb);
     fg_fb = acons (list1 (Qmswindows), build_string ("black"), fg_fb);
     bg_fb = acons (list1 (Qmswindows), build_string ("Gray75"), bg_fb);
 #endif
@@ -2072,7 +1980,7 @@ complex_vars_of_faces (void)
   set_specifier_fallback (Fget (Vmodeline_face, Qbackground_pixmap, Qnil),
                          Fget (Vgui_element_face, Qbackground_pixmap,
                                Qunbound));
-
+  
   /* toolbar is another gui element */
   Vtoolbar_face = Fmake_face (Qtoolbar,
                              build_string ("toolbar face"),
@@ -2099,18 +2007,6 @@ complex_vars_of_faces (void)
                          Fget (Vgui_element_face, Qbackground_pixmap,
                                Qunbound));
 
-  /* widget is another gui element */
-  Vwidget_face = Fmake_face (Qwidget,
-                            build_string ("widget face"),
-                            Qnil);
-  set_specifier_fallback (Fget (Vwidget_face, Qfont, Qunbound),
-                         Fget (Vgui_element_face, Qfont, Qunbound));
-  set_specifier_fallback (Fget (Vwidget_face, Qforeground, Qunbound),
-                         Fget (Vgui_element_face, Qforeground, Qunbound));
-  set_specifier_fallback (Fget (Vwidget_face, Qbackground, Qunbound),
-                         Fget (Vgui_element_face, Qbackground, Qunbound));
-  /* We don't want widgets to have a default background pixmap. */
-
   Vleft_margin_face = Fmake_face (Qleft_margin,
                                  build_string ("left margin face"),
                                  Qnil);