This commit was generated by cvs2svn to compensate for changes in r5670,
[chise/xemacs-chise.git.1] / src / glyphs.c
index 0e327f4..b481ce8 100644 (file)
@@ -3,7 +3,6 @@
    Copyright (C) 1995 Tinker Systems
    Copyright (C) 1995, 1996 Ben Wing
    Copyright (C) 1995 Sun Microsystems
-   Copyright (C) 1998, 1999 Andy Piper
 
 This file is part of XEmacs.
 
@@ -24,7 +23,7 @@ Boston, MA 02111-1307, USA.  */
 
 /* Synched up with: Not in FSF. */
 
-/* Written by Ben Wing and Chuck Thompson. */
+/* Written by Ben Wing and Chuck Thompson */
 
 #include <config.h>
 #include "lisp.h"
@@ -35,14 +34,10 @@ Boston, MA 02111-1307, USA.  */
 #include "faces.h"
 #include "frame.h"
 #include "insdel.h"
-#include "opaque.h"
+#include "glyphs.h"
 #include "objects.h"
 #include "redisplay.h"
 #include "window.h"
-#include "frame.h"
-#include "chartab.h"
-#include "rangetab.h"
-#include "blocktype.h"
 
 #ifdef HAVE_XPM
 #include <X11/xpm.h>
@@ -57,12 +52,11 @@ Lisp_Object Qmono_pixmap_image_instance_p;
 Lisp_Object Qcolor_pixmap_image_instance_p;
 Lisp_Object Qpointer_image_instance_p;
 Lisp_Object Qsubwindow_image_instance_p;
-Lisp_Object Qlayout_image_instance_p;
-Lisp_Object Qwidget_image_instance_p;
 Lisp_Object Qconst_glyph_variable;
 Lisp_Object Qmono_pixmap, Qcolor_pixmap, Qsubwindow;
-Lisp_Object Q_file, Q_data, Q_face, Q_pixel_width, Q_pixel_height;
+Lisp_Object Q_file, Q_data, Q_face;
 Lisp_Object Qformatted_string;
+
 Lisp_Object Vcurrent_display_table;
 Lisp_Object Vtruncation_glyph, Vcontinuation_glyph, Voctal_escape_glyph;
 Lisp_Object Vcontrol_arrow_glyph, Vinvisible_text_glyph, Vhscroll_glyph;
@@ -72,14 +66,10 @@ Lisp_Object Vimage_instantiator_format_list;
 Lisp_Object Vimage_instance_type_list;
 Lisp_Object Vglyph_type_list;
 
-int disable_animated_pixmaps;
-
 DEFINE_IMAGE_INSTANTIATOR_FORMAT (nothing);
 DEFINE_IMAGE_INSTANTIATOR_FORMAT (inherit);
 DEFINE_IMAGE_INSTANTIATOR_FORMAT (string);
 DEFINE_IMAGE_INSTANTIATOR_FORMAT (formatted_string);
-DEFINE_IMAGE_INSTANTIATOR_FORMAT (subwindow);
-DEFINE_IMAGE_INSTANTIATOR_FORMAT (text);
 
 #ifdef HAVE_WINDOW_SYSTEM
 DEFINE_IMAGE_INSTANTIATOR_FORMAT (xbm);
@@ -95,11 +85,6 @@ Lisp_Object Q_foreground, Q_background;
 #endif
 #endif
 
-#ifdef HAVE_XFACE
-DEFINE_IMAGE_INSTANTIATOR_FORMAT (xface);
-Lisp_Object Qxface;
-#endif
-
 #ifdef HAVE_XPM
 DEFINE_IMAGE_INSTANTIATOR_FORMAT (xpm);
 Lisp_Object Qxpm;
@@ -110,7 +95,6 @@ typedef struct image_instantiator_format_entry image_instantiator_format_entry;
 struct image_instantiator_format_entry
 {
   Lisp_Object symbol;
-  Lisp_Object device;
   struct image_instantiator_methods *meths;
 };
 
@@ -127,13 +111,6 @@ static void image_validate (Lisp_Object instantiator);
 static void glyph_property_was_changed (Lisp_Object glyph,
                                        Lisp_Object property,
                                        Lisp_Object locale);
-static void register_ignored_expose (struct frame* f, int x, int y, int width, int height);
-/* Unfortunately windows and X are different. In windows BeginPaint()
-   will prevent WM_PAINT messages being generated so it is unnecessary
-   to register exposures as they will not occur. Under X they will
-   always occur. */
-int hold_ignored_expose_registration;
-
 EXFUN (Fimage_instance_type, 1);
 EXFUN (Fglyph_type, 1);
 
@@ -142,9 +119,8 @@ EXFUN (Fglyph_type, 1);
  *                          Image Instantiators                             *
  ****************************************************************************/
 
-struct image_instantiator_methods *
-decode_device_ii_format (Lisp_Object device, Lisp_Object format,
-                        Error_behavior errb)
+static struct image_instantiator_methods *
+decode_image_instantiator_format (Lisp_Object format, Error_behavior errb)
 {
   int i;
 
@@ -158,19 +134,10 @@ decode_device_ii_format (Lisp_Object device, Lisp_Object format,
   for (i = 0; i < Dynarr_length (the_image_instantiator_format_entry_dynarr);
        i++)
     {
-      if ( EQ (format,
-              Dynarr_at (the_image_instantiator_format_entry_dynarr, i).
-              symbol) )
-       {
-         Lisp_Object d = Dynarr_at (the_image_instantiator_format_entry_dynarr, i).
-           device;
-         if ((NILP (d) && NILP (device))
-             ||
-             (!NILP (device) &&
-              EQ (CONSOLE_TYPE (XCONSOLE 
-                                (DEVICE_CONSOLE (XDEVICE (device)))), d)))
-           return Dynarr_at (the_image_instantiator_format_entry_dynarr, i).meths;
-       }
+      if (EQ (format,
+             Dynarr_at (the_image_instantiator_format_entry_dynarr, i).
+             symbol))
+       return Dynarr_at (the_image_instantiator_format_entry_dynarr, i).meths;
     }
 
   maybe_signal_simple_error ("Invalid image-instantiator format", format,
@@ -179,52 +146,22 @@ decode_device_ii_format (Lisp_Object device, Lisp_Object format,
   return 0;
 }
 
-struct image_instantiator_methods *
-decode_image_instantiator_format (Lisp_Object format, Error_behavior errb)
-{
-  return decode_device_ii_format (Qnil, format, errb);
-}
-
 static int
-valid_image_instantiator_format_p (Lisp_Object format, Lisp_Object locale)
+valid_image_instantiator_format_p (Lisp_Object format)
 {
-  int i;
-  struct image_instantiator_methods* meths =
-    decode_image_instantiator_format (format, ERROR_ME_NOT);
-  Lisp_Object contype = Qnil;
-  /* mess with the locale */
-  if (!NILP (locale) && SYMBOLP (locale))
-    contype = locale;
-  else
-    {
-      struct console* console = decode_console (locale);
-      contype = console ? CONSOLE_TYPE (console) : locale;
-    }
-  /* nothing is valid in all locales */
-  if (EQ (format, Qnothing))
-    return 1;
-  /* reject unknown formats */
-  else if (NILP (contype) || !meths)
-    return 0;
-
-  for (i = 0; i < Dynarr_length (meths->consoles); i++)
-    if (EQ (contype, Dynarr_at (meths->consoles, i).symbol))
-      return 1;
-  return 0;
+  return (decode_image_instantiator_format (format, ERROR_ME_NOT) != 0);
 }
 
 DEFUN ("valid-image-instantiator-format-p", Fvalid_image_instantiator_format_p,
-       1, 2, 0, /*
+       1, 1, 0, /*
 Given an IMAGE-INSTANTIATOR-FORMAT, return non-nil if it is valid.
-If LOCALE is non-nil then the format is checked in that domain.
-If LOCALE is nil the current console is used.
 Valid formats are some subset of 'nothing, 'string, 'formatted-string,
 'xpm, 'xbm, 'xface, 'gif, 'jpeg, 'png, 'tiff, 'cursor-font, 'font,
-'autodetect, 'widget and 'subwindow, depending on how XEmacs was compiled.
+'autodetect, and 'subwindow, depending on how XEmacs was compiled.
 */
-       (image_instantiator_format, locale))
+       (image_instantiator_format))
 {
-  return valid_image_instantiator_format_p (image_instantiator_format, locale) ?
+  return valid_image_instantiator_format_p (image_instantiator_format) ?
     Qt : Qnil;
 }
 
@@ -238,27 +175,19 @@ Return a list of valid image-instantiator formats.
 }
 
 void
-add_entry_to_device_ii_format_list (Lisp_Object device, Lisp_Object symbol,
-                                   struct image_instantiator_methods *meths)
+add_entry_to_image_instantiator_format_list (Lisp_Object symbol,
+                                            struct
+                                            image_instantiator_methods *meths)
 {
   struct image_instantiator_format_entry entry;
 
   entry.symbol = symbol;
-  entry.device = device;
   entry.meths = meths;
   Dynarr_add (the_image_instantiator_format_entry_dynarr, entry);
   Vimage_instantiator_format_list =
     Fcons (symbol, Vimage_instantiator_format_list);
 }
 
-void
-add_entry_to_image_instantiator_format_list (Lisp_Object symbol,
-                                            struct
-                                            image_instantiator_methods *meths)
-{
-  add_entry_to_device_ii_format_list (Qnil, symbol, meths);
-}
-
 static Lisp_Object *
 get_image_conversion_list (Lisp_Object console_type)
 {
@@ -426,13 +355,7 @@ check_valid_string (Lisp_Object data)
   CHECK_STRING (data);
 }
 
-void
-check_valid_vector (Lisp_Object data)
-{
-  CHECK_VECTOR (data);
-}
-
-void
+static void
 check_valid_face (Lisp_Object data)
 {
   Fget_face (data);
@@ -558,16 +481,12 @@ normalize_image_instantiator (Lisp_Object instantiator,
      longer exist (e.g. w3 pixmaps are almost always from temporary
      files). */
   {
-    struct gcpro gcpro1;
-    struct image_instantiator_methods *meths;
-
-    GCPRO1 (instantiator);
-    
-    meths = decode_image_instantiator_format (XVECTOR_DATA (instantiator)[0],
-                                             ERROR_ME);
-    RETURN_UNGCPRO (IIFORMAT_METH_OR_GIVEN (meths, normalize,
-                                           (instantiator, contype),
-                                           instantiator));
+    struct image_instantiator_methods * meths =
+      decode_image_instantiator_format (XVECTOR_DATA (instantiator)[0],
+                                       ERROR_ME);
+    return IIFORMAT_METH_OR_GIVEN (meths, normalize,
+                                  (instantiator, contype),
+                                  instantiator);
   }
 }
 
@@ -580,30 +499,16 @@ instantiate_image_instantiator (Lisp_Object device, Lisp_Object domain,
   Lisp_Object ii = allocate_image_instance (device);
   struct image_instantiator_methods *meths;
   struct gcpro gcpro1;
-  int  methp = 0;
 
   GCPRO1 (ii);
-  if (!valid_image_instantiator_format_p (XVECTOR_DATA (instantiator)[0], device))
-    signal_simple_error
-      ("Image instantiator format is invalid in this locale.",
-       instantiator);
-
   meths = decode_image_instantiator_format (XVECTOR_DATA (instantiator)[0],
                                            ERROR_ME);
-  methp = (int)HAS_IIFORMAT_METH_P (meths, instantiate);
-  MAYBE_IIFORMAT_METH (meths, instantiate, (ii, instantiator, pointer_fg,
-                                           pointer_bg, dest_mask, domain));
-  
-  /* now do device specific instantiation */
-  meths = decode_device_ii_format (device, XVECTOR_DATA (instantiator)[0],
-                                  ERROR_ME_NOT);
-
-  if (!methp && (!meths || !HAS_IIFORMAT_METH_P (meths, instantiate)))
+  if (!HAS_IIFORMAT_METH_P (meths, instantiate))
     signal_simple_error
       ("Don't know how to instantiate this image instantiator?",
        instantiator);
-  MAYBE_IIFORMAT_METH (meths, instantiate, (ii, instantiator, pointer_fg,
-                                           pointer_bg, dest_mask, domain));
+  IIFORMAT_METH (meths, instantiate, (ii, instantiator, pointer_fg,
+                                     pointer_bg, dest_mask, domain));
   UNGCPRO;
 
   return ii;
@@ -617,46 +522,33 @@ instantiate_image_instantiator (Lisp_Object device, Lisp_Object domain,
 Lisp_Object Qimage_instancep;
 
 static Lisp_Object
-mark_image_instance (Lisp_Object obj)
+mark_image_instance (Lisp_Object obj, void (*markobj) (Lisp_Object))
 {
   struct Lisp_Image_Instance *i = XIMAGE_INSTANCE (obj);
 
-  mark_object (i->name);
+  (markobj) (i->name);
   switch (IMAGE_INSTANCE_TYPE (i))
     {
     case IMAGE_TEXT:
-      mark_object (IMAGE_INSTANCE_TEXT_STRING (i));
+      (markobj) (IMAGE_INSTANCE_TEXT_STRING (i));
       break;
     case IMAGE_MONO_PIXMAP:
     case IMAGE_COLOR_PIXMAP:
-      mark_object (IMAGE_INSTANCE_PIXMAP_FILENAME (i));
-      mark_object (IMAGE_INSTANCE_PIXMAP_MASK_FILENAME (i));
-      mark_object (IMAGE_INSTANCE_PIXMAP_HOTSPOT_X (i));
-      mark_object (IMAGE_INSTANCE_PIXMAP_HOTSPOT_Y (i));
-      mark_object (IMAGE_INSTANCE_PIXMAP_FG (i));
-      mark_object (IMAGE_INSTANCE_PIXMAP_BG (i));
+      (markobj) (IMAGE_INSTANCE_PIXMAP_FILENAME (i));
+      (markobj) (IMAGE_INSTANCE_PIXMAP_MASK_FILENAME (i));
+      (markobj) (IMAGE_INSTANCE_PIXMAP_HOTSPOT_X (i));
+      (markobj) (IMAGE_INSTANCE_PIXMAP_HOTSPOT_Y (i));
+      (markobj) (IMAGE_INSTANCE_PIXMAP_FG (i));
+      (markobj) (IMAGE_INSTANCE_PIXMAP_BG (i));
       break;
-
-    case IMAGE_WIDGET:
-      mark_object (IMAGE_INSTANCE_WIDGET_TYPE (i));
-      mark_object (IMAGE_INSTANCE_WIDGET_PROPS (i));
-      mark_object (IMAGE_INSTANCE_WIDGET_FACE (i));
-      mark_object (IMAGE_INSTANCE_WIDGET_ITEMS (i));
     case IMAGE_SUBWINDOW:
-      mark_object (IMAGE_INSTANCE_SUBWINDOW_FRAME (i));
-      break;
-
-    case IMAGE_LAYOUT:
-      mark_object (IMAGE_INSTANCE_LAYOUT_CHILDREN (i));
-      mark_object (IMAGE_INSTANCE_LAYOUT_BORDER (i));
-      mark_object (IMAGE_INSTANCE_SUBWINDOW_FRAME (i));
+      /* #### implement me */
       break;
-
     default:
       break;
     }
 
-  MAYBE_DEVMETH (XDEVICE (i->device), mark_image_instance, (i));
+  MAYBE_DEVMETH (XDEVICE (i->device), mark_image_instance, (i, markobj));
 
   return i->device;
 }
@@ -753,51 +645,8 @@ print_image_instance (Lisp_Object obj, Lisp_Object printcharfun,
        }
       break;
 
-    case IMAGE_WIDGET:
-      /*
-      if (!NILP (IMAGE_INSTANCE_WIDGET_CALLBACK (ii)))
-       {
-         print_internal (IMAGE_INSTANCE_WIDGET_CALLBACK (ii), printcharfun, 0);
-         write_c_string (", ", printcharfun);
-       }
-      */
-      if (!NILP (IMAGE_INSTANCE_WIDGET_FACE (ii)))
-       {
-         write_c_string (" (", printcharfun);
-         print_internal
-           (IMAGE_INSTANCE_WIDGET_FACE (ii), printcharfun, 0);
-         write_c_string (")", printcharfun);
-       }
-
-      if (!NILP (IMAGE_INSTANCE_WIDGET_TEXT (ii)))
-       print_internal (IMAGE_INSTANCE_WIDGET_TEXT (ii), printcharfun, 0);
-
     case IMAGE_SUBWINDOW:
-    case IMAGE_LAYOUT:
-      sprintf (buf, " %dx%d", IMAGE_INSTANCE_SUBWINDOW_WIDTH (ii),
-              IMAGE_INSTANCE_SUBWINDOW_HEIGHT (ii));
-      write_c_string (buf, printcharfun);
-
-      /* This is stolen from frame.c.  Subwindows are strange in that they
-        are specific to a particular frame so we want to print in their
-        description what that frame is. */
-
-      write_c_string (" on #<", printcharfun);
-      {
-       struct frame* f  = XFRAME (IMAGE_INSTANCE_SUBWINDOW_FRAME (ii));
-       
-       if (!FRAME_LIVE_P (f))
-         write_c_string ("dead", printcharfun);
-       else 
-         write_c_string (DEVICE_TYPE_NAME (XDEVICE (FRAME_DEVICE (f))),
-                         printcharfun);
-
-       write_c_string ("-frame ", printcharfun);
-      }
-      write_c_string (">", printcharfun);
-      sprintf (buf, " 0x%p", IMAGE_INSTANCE_SUBWINDOW_ID (ii));
-      write_c_string (buf, printcharfun);
-      
+      /* #### implement me */
       break;
 
     default:
@@ -820,23 +669,14 @@ finalize_image_instance (void *header, int for_disksave)
     return;
   if (for_disksave) finalose (i);
 
-  /* do this so that the cachels get reset */
-  if (IMAGE_INSTANCE_TYPE (i) == IMAGE_WIDGET
-      ||
-      IMAGE_INSTANCE_TYPE (i) == IMAGE_SUBWINDOW)
-    {
-      MARK_FRAME_SUBWINDOWS_CHANGED 
-       (XFRAME (IMAGE_INSTANCE_SUBWINDOW_FRAME (i)));
-    }
-
   MAYBE_DEVMETH (XDEVICE (i->device), finalize_image_instance, (i));
 }
 
 static int
-image_instance_equal (Lisp_Object obj1, Lisp_Object obj2, int depth)
+image_instance_equal (Lisp_Object o1, Lisp_Object o2, int depth)
 {
-  struct Lisp_Image_Instance *i1 = XIMAGE_INSTANCE (obj1);
-  struct Lisp_Image_Instance *i2 = XIMAGE_INSTANCE (obj2);
+  struct Lisp_Image_Instance *i1 = XIMAGE_INSTANCE (o1);
+  struct Lisp_Image_Instance *i2 = XIMAGE_INSTANCE (o2);
   struct device *d1 = XDEVICE (i1->device);
   struct device *d2 = XDEVICE (i2->device);
 
@@ -869,8 +709,6 @@ image_instance_equal (Lisp_Object obj1, Lisp_Object obj2, int depth)
            IMAGE_INSTANCE_PIXMAP_HEIGHT (i2) &&
            IMAGE_INSTANCE_PIXMAP_DEPTH (i1) ==
            IMAGE_INSTANCE_PIXMAP_DEPTH (i2) &&
-           IMAGE_INSTANCE_PIXMAP_SLICE (i1) ==
-           IMAGE_INSTANCE_PIXMAP_SLICE (i2) &&
            EQ (IMAGE_INSTANCE_PIXMAP_HOTSPOT_X (i1),
                IMAGE_INSTANCE_PIXMAP_HOTSPOT_X (i2)) &&
            EQ (IMAGE_INSTANCE_PIXMAP_HOTSPOT_Y (i1),
@@ -884,35 +722,8 @@ image_instance_equal (Lisp_Object obj1, Lisp_Object obj2, int depth)
        return 0;
       break;
 
-    case IMAGE_WIDGET:
-      if (!(EQ (IMAGE_INSTANCE_WIDGET_TYPE (i1),
-               IMAGE_INSTANCE_WIDGET_TYPE (i2))
-           && internal_equal (IMAGE_INSTANCE_WIDGET_ITEMS (i1),
-                              IMAGE_INSTANCE_WIDGET_ITEMS (i2),
-                              depth + 1)
-           && internal_equal (IMAGE_INSTANCE_WIDGET_PROPS (i1),
-                              IMAGE_INSTANCE_WIDGET_PROPS (i2),
-                              depth + 1)
-           ))
-       return 0;
-    case IMAGE_LAYOUT:
-      if (IMAGE_INSTANCE_TYPE (i1) == IMAGE_LAYOUT
-         &&
-         !(EQ (IMAGE_INSTANCE_LAYOUT_BORDER (i1),
-               IMAGE_INSTANCE_LAYOUT_BORDER (i2))
-           &&
-           internal_equal (IMAGE_INSTANCE_LAYOUT_CHILDREN (i1),
-                           IMAGE_INSTANCE_LAYOUT_CHILDREN (i2),
-                           depth + 1)))
-       return 0;
     case IMAGE_SUBWINDOW:
-      if (!(IMAGE_INSTANCE_SUBWINDOW_WIDTH (i1) ==
-           IMAGE_INSTANCE_SUBWINDOW_WIDTH (i2) &&
-           IMAGE_INSTANCE_SUBWINDOW_HEIGHT (i1) ==
-           IMAGE_INSTANCE_SUBWINDOW_HEIGHT (i2) &&
-           IMAGE_INSTANCE_SUBWINDOW_ID (i1) ==
-           IMAGE_INSTANCE_SUBWINDOW_ID (i2)))
-       return 0;
+      /* #### implement me */
       break;
 
     default:
@@ -942,29 +753,15 @@ image_instance_hash (Lisp_Object obj, int depth)
     case IMAGE_MONO_PIXMAP:
     case IMAGE_COLOR_PIXMAP:
     case IMAGE_POINTER:
-      hash = HASH6 (hash, IMAGE_INSTANCE_PIXMAP_WIDTH (i),
+      hash = HASH5 (hash, IMAGE_INSTANCE_PIXMAP_WIDTH (i),
                    IMAGE_INSTANCE_PIXMAP_HEIGHT (i),
                    IMAGE_INSTANCE_PIXMAP_DEPTH (i),
-                   IMAGE_INSTANCE_PIXMAP_SLICE (i),
                    internal_hash (IMAGE_INSTANCE_PIXMAP_FILENAME (i),
                                   depth + 1));
       break;
 
-    case IMAGE_WIDGET:
-      hash = HASH4 (hash, 
-                   internal_hash (IMAGE_INSTANCE_WIDGET_TYPE (i), depth + 1),
-                   internal_hash (IMAGE_INSTANCE_WIDGET_PROPS (i), depth + 1),
-                   internal_hash (IMAGE_INSTANCE_WIDGET_ITEMS (i), depth + 1));
-    case IMAGE_LAYOUT:
-      if (IMAGE_INSTANCE_TYPE (i) == IMAGE_LAYOUT)
-       hash = HASH3 (hash,
-                     internal_hash (IMAGE_INSTANCE_LAYOUT_BORDER (i), depth + 1),
-                     internal_hash (IMAGE_INSTANCE_LAYOUT_CHILDREN (i),
-                                    depth + 1));
     case IMAGE_SUBWINDOW:
-      hash = HASH4 (hash, IMAGE_INSTANCE_SUBWINDOW_WIDTH (i),
-                   IMAGE_INSTANCE_SUBWINDOW_HEIGHT (i),
-                   (int) IMAGE_INSTANCE_SUBWINDOW_ID (i));
+      /* #### implement me */
       break;
 
     default:
@@ -978,22 +775,20 @@ image_instance_hash (Lisp_Object obj, int depth)
 DEFINE_LRECORD_IMPLEMENTATION ("image-instance", image_instance,
                               mark_image_instance, print_image_instance,
                               finalize_image_instance, image_instance_equal,
-                              image_instance_hash, 0,
+                              image_instance_hash,
                               struct Lisp_Image_Instance);
 
 static Lisp_Object
 allocate_image_instance (Lisp_Object device)
 {
   struct Lisp_Image_Instance *lp =
-    alloc_lcrecord_type (struct Lisp_Image_Instance, &lrecord_image_instance);
+    alloc_lcrecord_type (struct Lisp_Image_Instance, lrecord_image_instance);
   Lisp_Object val;
 
   zero_lcrecord (lp);
   lp->device = device;
   lp->type = IMAGE_NOTHING;
   lp->name = Qnil;
-  lp->x_offset = 0;
-  lp->y_offset = 0;
   XSETIMAGE_INSTANCE (val, lp);
   return val;
 }
@@ -1010,8 +805,6 @@ decode_image_instance_type (Lisp_Object type, Error_behavior errb)
   if (EQ (type, Qcolor_pixmap)) return IMAGE_COLOR_PIXMAP;
   if (EQ (type, Qpointer))      return IMAGE_POINTER;
   if (EQ (type, Qsubwindow))    return IMAGE_SUBWINDOW;
-  if (EQ (type, Qwidget))    return IMAGE_WIDGET;
-  if (EQ (type, Qlayout))    return IMAGE_LAYOUT;
 
   maybe_signal_simple_error ("Invalid image-instance type", type,
                             Qimage, errb);
@@ -1030,8 +823,6 @@ encode_image_instance_type (enum image_instance_type type)
     case IMAGE_COLOR_PIXMAP: return Qcolor_pixmap;
     case IMAGE_POINTER:      return Qpointer;
     case IMAGE_SUBWINDOW:    return Qsubwindow;
-    case IMAGE_WIDGET:    return Qwidget;
-    case IMAGE_LAYOUT:    return Qlayout;
     default:
       abort ();
     }
@@ -1044,7 +835,7 @@ image_instance_type_to_mask (enum image_instance_type type)
 {
   /* This depends on the fact that enums are assigned consecutive
      integers starting at 0. (Remember that IMAGE_UNKNOWN is the
-     first enum.) I'm fairly sure this behavior is ANSI-mandated,
+     first enum.) I'm fairly sure this behavior in ANSI-mandated,
      so there should be no portability problems here. */
   return (1 << ((int) (type) - 1));
 }
@@ -1167,14 +958,14 @@ make_image_instance_1 (Lisp_Object data, Lisp_Object device,
   /* instantiate_image_instantiator() will abort if given an
      image instance ... */
   if (IMAGE_INSTANCEP (data))
-    signal_simple_error ("Image instances not allowed here", data);
+    signal_simple_error ("image instances not allowed here", data);
   image_validate (data);
   dest_mask = decode_image_instance_type_list (dest_types);
   data = normalize_image_instantiator (data, DEVICE_TYPE (XDEVICE (device)),
                                       make_int (dest_mask));
   GCPRO1 (data);
   if (VECTORP (data) && EQ (XVECTOR_DATA (data)[0], Qinherit))
-    signal_simple_error ("Inheritance not allowed here", data);
+    signal_simple_error ("inheritance not allowed here", data);
   ii = instantiate_image_instantiator (device, device, data,
                                       Qnil, Qnil, dest_mask);
   RETURN_UNGCPRO (ii);
@@ -1215,9 +1006,7 @@ be generated.  The recognized image instance types are
 'subwindow
   A child window that is treated as an image.  This allows (e.g.)
   another program to be responsible for drawing into the window.
-'widget
-  A child window that contains a window-system widget, e.g. a push
-  button.
+  Not currently implemented.
 
 The DEST-TYPES list is unordered.  If multiple destination types
 are possible for a given instantiator, the "most natural" type
@@ -1239,7 +1028,7 @@ If DEST-TYPES is omitted, all possible types are allowed.
 NO-ERROR controls what happens when the image cannot be generated.
 If nil, an error message is generated.  If t, no messages are
 generated and this function returns nil.  If anything else, a warning
-message is generated and this function returns nil.  
+message is generated and this function returns nil.
 */
        (data, device, dest_types, no_error))
 {
@@ -1280,106 +1069,17 @@ Return the name of the given image instance.
 
 DEFUN ("image-instance-string", Fimage_instance_string, 1, 1, 0, /*
 Return the string of the given image instance.
-This will only be non-nil for text image instances and widgets.
+This will only be non-nil for text image instances.
 */
        (image_instance))
 {
   CHECK_IMAGE_INSTANCE (image_instance);
   if (XIMAGE_INSTANCE_TYPE (image_instance) == IMAGE_TEXT)
     return XIMAGE_INSTANCE_TEXT_STRING (image_instance);
-  else if (XIMAGE_INSTANCE_TYPE (image_instance) == IMAGE_WIDGET)
-    return XIMAGE_INSTANCE_WIDGET_TEXT (image_instance);
   else
     return Qnil;
 }
 
-DEFUN ("image-instance-property", Fimage_instance_property, 2, 2, 0, /*
-Return the given property of the given image instance.  
-Returns nil if the property or the property method do not exist for
-the image instance in the domain.  
-*/
-       (image_instance, prop))
-{
-  struct Lisp_Image_Instance* ii;
-  Lisp_Object type, ret;
-  struct image_instantiator_methods* meths;
-
-  CHECK_IMAGE_INSTANCE (image_instance);
-  CHECK_SYMBOL (prop);
-  ii = XIMAGE_INSTANCE (image_instance);
-
-  /* ... then try device specific methods ... */
-  type = encode_image_instance_type (IMAGE_INSTANCE_TYPE (ii));
-  meths = decode_device_ii_format (IMAGE_INSTANCE_DEVICE (ii), 
-                                  type, ERROR_ME_NOT);
-  if (meths && HAS_IIFORMAT_METH_P (meths, property)
-      && 
-      !UNBOUNDP (ret =  IIFORMAT_METH (meths, property, (image_instance, prop))))
-    {
-      return ret;
-    }
-  /* ... then format specific methods ... */
-  meths = decode_device_ii_format (Qnil, type, ERROR_ME_NOT);
-  if (meths && HAS_IIFORMAT_METH_P (meths, property)
-      &&
-      !UNBOUNDP (ret = IIFORMAT_METH (meths, property, (image_instance, prop))))
-    {
-      return ret;
-    }
-  /* ... then fail */
-  return Qnil;
-}
-
-DEFUN ("set-image-instance-property", Fset_image_instance_property, 3, 3, 0, /*
-Set the given property of the given image instance.  
-Does nothing if the property or the property method do not exist for
-the image instance in the domain.
-*/
-       (image_instance, prop, val))
-{
-  struct Lisp_Image_Instance* ii;
-  Lisp_Object type, ret;
-  struct image_instantiator_methods* meths;
-
-  CHECK_IMAGE_INSTANCE (image_instance);
-  CHECK_SYMBOL (prop);
-  ii = XIMAGE_INSTANCE (image_instance);
-  type = encode_image_instance_type (IMAGE_INSTANCE_TYPE (ii));
-  /* try device specific methods first ... */
-  meths = decode_device_ii_format (IMAGE_INSTANCE_DEVICE (ii), 
-                                  type, ERROR_ME_NOT);
-  if (meths && HAS_IIFORMAT_METH_P (meths, set_property)
-      &&
-      !UNBOUNDP (ret = 
-                IIFORMAT_METH (meths, set_property, (image_instance, prop, val))))
-    {
-      val = ret;
-    }
-  else
-    {
-      /* ... then format specific methods ... */
-      meths = decode_device_ii_format (Qnil, type, ERROR_ME_NOT);
-      if (meths && HAS_IIFORMAT_METH_P (meths, set_property)
-         &&
-         !UNBOUNDP (ret = 
-                    IIFORMAT_METH (meths, set_property, (image_instance, prop, val))))
-       {
-         val = ret;
-       }
-      else
-       {
-         val = Qnil;
-       }
-    }
-
-  /* Make sure the image instance gets redisplayed. */
-  MARK_IMAGE_INSTANCE_CHANGED (ii);
-  MARK_SUBWINDOWS_STATE_CHANGED;
-  MARK_GLYPHS_CHANGED;
-
-  return val;
-}
-
 DEFUN ("image-instance-file-name", Fimage_instance_file_name, 1, 1, 0, /*
 Return the file name from which IMAGE-INSTANCE was read, if known.
 */
@@ -1452,11 +1152,6 @@ Return the height of the image instance, in pixels.
     case IMAGE_POINTER:
       return make_int (XIMAGE_INSTANCE_PIXMAP_HEIGHT (image_instance));
 
-    case IMAGE_SUBWINDOW:
-    case IMAGE_WIDGET:
-    case IMAGE_LAYOUT:
-      return make_int (XIMAGE_INSTANCE_SUBWINDOW_HEIGHT (image_instance));
-
     default:
       return Qnil;
     }
@@ -1476,11 +1171,6 @@ Return the width of the image instance, in pixels.
     case IMAGE_POINTER:
       return make_int (XIMAGE_INSTANCE_PIXMAP_WIDTH (image_instance));
 
-    case IMAGE_SUBWINDOW:
-    case IMAGE_WIDGET:
-    case IMAGE_LAYOUT:
-      return make_int (XIMAGE_INSTANCE_SUBWINDOW_WIDTH (image_instance));
-
     default:
       return Qnil;
     }
@@ -1550,12 +1240,6 @@ colorized mono pixmaps and for pointers.)
     case IMAGE_POINTER:
       return XIMAGE_INSTANCE_PIXMAP_FG (image_instance);
 
-    case IMAGE_WIDGET:
-      return FACE_FOREGROUND (
-                             XIMAGE_INSTANCE_WIDGET_FACE (image_instance),
-                             XIMAGE_INSTANCE_SUBWINDOW_FRAME 
-                             (image_instance));
-
     default:
       return Qnil;
     }
@@ -1577,12 +1261,6 @@ colorized mono pixmaps and for pointers.)
     case IMAGE_POINTER:
       return XIMAGE_INSTANCE_PIXMAP_BG (image_instance);
 
-    case IMAGE_WIDGET:
-      return FACE_BACKGROUND (
-                             XIMAGE_INSTANCE_WIDGET_FACE (image_instance),
-                             XIMAGE_INSTANCE_SUBWINDOW_FRAME 
-                             (image_instance));
-
     default:
       return Qnil;
     }
@@ -1609,8 +1287,6 @@ instance is a mono pixmap; otherwise, the same image instance is returned.
   if (!HAS_DEVMETH_P (XDEVICE (device), colorize_image_instance))
     return image_instance;
 
-  /* #### There should be a copy_image_instance(), which calls a
-     device-specific method to copy the window-system subobject. */
   new = allocate_image_instance (device);
   copy_lcrecord (XIMAGE_INSTANCE (new), XIMAGE_INSTANCE (image_instance));
   /* note that if this method returns non-zero, this method MUST
@@ -1737,23 +1413,6 @@ string_instantiate (Lisp_Object image_instance, Lisp_Object instantiator,
     incompatible_image_types (instantiator, dest_mask, IMAGE_TEXT_MASK);
 }
 
-/* set the properties of a string */
-static Lisp_Object
-text_set_property (Lisp_Object image_instance, Lisp_Object prop,
-                  Lisp_Object val)
-{
-  struct Lisp_Image_Instance *ii = XIMAGE_INSTANCE (image_instance);
-
-  if (EQ (prop, Q_data))
-    {
-      CHECK_STRING (val);
-      IMAGE_INSTANCE_TEXT_STRING (ii) = val;
-
-      return Qt;
-    }
-  return Qunbound;
-}
-
 \f
 /****************************************************************************
  *                             formatted-string                             *
@@ -1824,7 +1483,7 @@ potential_pixmap_file_instantiator (Lisp_Object instantiator,
   if (!NILP (file) && NILP (data))
     {
       Lisp_Object retval = MAYBE_LISP_CONTYPE_METH
-       (decode_console_type(console_type, ERROR_ME),
+       (decode_console_type(console_type, ERROR_ME), 
         locate_pixmap_file, (file));
 
       if (!NILP (retval))
@@ -1832,7 +1491,7 @@ potential_pixmap_file_instantiator (Lisp_Object instantiator,
       else
        return Fcons (file, Qnil); /* should have been file */
     }
-
+  
   return Qnil;
 }
 
@@ -1855,7 +1514,7 @@ simple_image_type_normalize (Lisp_Object inst, Lisp_Object console_type,
      Note that if we cannot generate any regular inline data, we
      skip out. */
 
-  file = potential_pixmap_file_instantiator (inst, Q_file, Q_data,
+  file = potential_pixmap_file_instantiator (inst, Q_file, Q_data, 
                                             console_type);
 
   if (CONSP (file)) /* failure locating filename */
@@ -1941,9 +1600,7 @@ xbm_validate (Lisp_Object instantiator)
    -- maybe return an error, or return Qnil.
  */
 
-#ifdef HAVE_X_WINDOWS
-#include <X11/Xlib.h>
-#else
+#ifndef HAVE_X_WINDOWS
 #define XFree(data) free(data)
 #endif
 
@@ -1957,7 +1614,7 @@ bitmap_to_lisp_data (Lisp_Object name, int *xhot, int *yhot,
   CONST char *filename_ext;
 
   GET_C_STRING_FILENAME_DATA_ALLOCA (name, filename_ext);
-  result = read_bitmap_data_from_file (filename_ext, &w, &h,
+  result = read_bitmap_data_from_file (filename_ext, &w, &h, 
                                       &data, xhot, yhot);
 
   if (result == BitmapSuccess)
@@ -2018,11 +1675,11 @@ xbm_mask_file_munging (Lisp_Object alist, Lisp_Object file,
       && !NILP (file))
     {
       mask_file = MAYBE_LISP_CONTYPE_METH
-       (decode_console_type(console_type, ERROR_ME),
+       (decode_console_type(console_type, ERROR_ME), 
         locate_pixmap_file, (concat2 (file, build_string ("Mask"))));
       if (NILP (mask_file))
        mask_file = MAYBE_LISP_CONTYPE_METH
-         (decode_console_type(console_type, ERROR_ME),
+         (decode_console_type(console_type, ERROR_ME), 
           locate_pixmap_file, (concat2 (file, build_string ("msk"))));
     }
 
@@ -2110,81 +1767,20 @@ xbm_possible_dest_types (void)
     IMAGE_POINTER_MASK;
 }
 
-#endif
-
-\f
-#ifdef HAVE_XFACE
-/**********************************************************************
- *                             X-Face                                 *
- **********************************************************************/
-
 static void
-xface_validate (Lisp_Object instantiator)
-{
-  file_or_data_must_be_present (instantiator);
-}
-
-static Lisp_Object
-xface_normalize (Lisp_Object inst, Lisp_Object console_type)
+xbm_instantiate (Lisp_Object image_instance, Lisp_Object instantiator,
+                Lisp_Object pointer_fg, Lisp_Object pointer_bg,
+                int dest_mask, Lisp_Object domain)
 {
-  /* This function can call lisp */
-  Lisp_Object file = Qnil, mask_file = Qnil;
-  struct gcpro gcpro1, gcpro2, gcpro3;
-  Lisp_Object alist = Qnil;
-
-  GCPRO3 (file, mask_file, alist);
-
-  /* Now, convert any file data into inline data for both the regular
-     data and the mask data.  At the end of this, `data' will contain
-     the inline data (if any) or Qnil, and `file' will contain
-     the name this data was derived from (if known) or Qnil.
-     Likewise for `mask_file' and `mask_data'.
-
-     Note that if we cannot generate any regular inline data, we
-     skip out. */
-
-  file = potential_pixmap_file_instantiator (inst, Q_file, Q_data,
-                                            console_type);
-  mask_file = potential_pixmap_file_instantiator (inst, Q_mask_file,
-                                                 Q_mask_data, console_type);
-
-  if (CONSP (file)) /* failure locating filename */
-    signal_double_file_error ("Opening bitmap file",
-                             "no such file or directory",
-                             Fcar (file));
-
-  if (NILP (file) && NILP (mask_file)) /* no conversion necessary */
-    RETURN_UNGCPRO (inst);
-
-  alist = tagged_vector_to_alist (inst);
-
-  {
-    Lisp_Object data = make_string_from_file (file);
-    alist = remassq_no_quit (Q_file, alist);
-    /* there can't be a :data at this point. */
-    alist = Fcons (Fcons (Q_file, file),
-                  Fcons (Fcons (Q_data, data), alist));
-  }
-
-  alist = xbm_mask_file_munging (alist, file, mask_file, console_type);
+  Lisp_Object device= IMAGE_INSTANCE_DEVICE (XIMAGE_INSTANCE (image_instance));
 
-  {
-    Lisp_Object result = alist_to_tagged_vector (Qxface, alist);
-    free_alist (alist);
-    RETURN_UNGCPRO (result);
-  }
-}
-
-static int
-xface_possible_dest_types (void)
-{
-  return
-    IMAGE_MONO_PIXMAP_MASK  |
-    IMAGE_COLOR_PIXMAP_MASK |
-    IMAGE_POINTER_MASK;
+  MAYBE_DEVMETH (XDEVICE (device), 
+                xbm_instantiate,
+                (image_instance, instantiator, pointer_fg, 
+                 pointer_bg, dest_mask, domain));
 }
 
-#endif /* HAVE_XFACE */
+#endif
 
 \f
 #ifdef HAVE_XPM
@@ -2198,10 +1794,8 @@ pixmap_to_lisp_data (Lisp_Object name, int ok_if_data_invalid)
 {
   char **data;
   int result;
-  char *fname = 0;
-  
-  GET_C_STRING_FILENAME_DATA_ALLOCA (name, fname);
-  result = XpmReadFileToData (fname, &data);
+
+  result = XpmReadFileToData ((char *) XSTRING_DATA (name), &data);
 
   if (result == XpmSuccess)
     {
@@ -2349,7 +1943,7 @@ xpm_normalize (Lisp_Object inst, Lisp_Object console_type)
      Note that if we cannot generate any regular inline data, we
      skip out. */
 
-  file = potential_pixmap_file_instantiator (inst, Q_file, Q_data,
+  file = potential_pixmap_file_instantiator (inst, Q_file, Q_data, 
                                             console_type);
 
   if (CONSP (file)) /* failure locating filename */
@@ -2363,7 +1957,7 @@ xpm_normalize (Lisp_Object inst, Lisp_Object console_type)
   if (NILP (file) && !UNBOUNDP (color_symbols))
     /* no conversion necessary */
     RETURN_UNGCPRO (inst);
-
+  
   alist = tagged_vector_to_alist (inst);
 
   if (!NILP (file))
@@ -2374,7 +1968,7 @@ xpm_normalize (Lisp_Object inst, Lisp_Object console_type)
       alist = Fcons (Fcons (Q_file, file),
                     Fcons (Fcons (Q_data, data), alist));
     }
-
+  
   if (UNBOUNDP (color_symbols))
     {
       color_symbols = evaluate_xpm_color_symbols ();
@@ -2398,6 +1992,19 @@ xpm_possible_dest_types (void)
     IMAGE_POINTER_MASK;
 }
 
+static void
+xpm_instantiate (Lisp_Object image_instance, Lisp_Object instantiator,
+                Lisp_Object pointer_fg, Lisp_Object pointer_bg,
+                int dest_mask, Lisp_Object domain)
+{
+  Lisp_Object device= IMAGE_INSTANCE_DEVICE (XIMAGE_INSTANCE (image_instance));
+
+  MAYBE_DEVMETH (XDEVICE (device), 
+                xpm_instantiate,
+                (image_instance, instantiator, pointer_fg, 
+                 pointer_bg, dest_mask, domain));
+}
+
 #endif /* HAVE_XPM */
 
 \f
@@ -2418,12 +2025,12 @@ image_create (Lisp_Object obj)
 }
 
 static void
-image_mark (Lisp_Object obj)
+image_mark (Lisp_Object obj, void (*markobj) (Lisp_Object))
 {
   struct Lisp_Specifier *image = XIMAGE_SPECIFIER (obj);
 
-  mark_object (IMAGE_SPECIFIER_ATTACHEE (image));
-  mark_object (IMAGE_SPECIFIER_ATTACHEE_PROPERTY (image));
+  ((markobj) (IMAGE_SPECIFIER_ATTACHEE (image)));
+  ((markobj) (IMAGE_SPECIFIER_ATTACHEE_PROPERTY (image)));
 }
 
 static Lisp_Object
@@ -2500,7 +2107,7 @@ image_instantiate (Lisp_Object specifier, Lisp_Object matchspec,
          /* For the image instance cache, we do comparisons with EQ rather
             than with EQUAL, as we do for color and font names.
             The reasons are:
-            
+
             1) pixmap data can be very long, and thus the hashing and
             comparing will take awhile.
             2) It's not so likely that we'll run into things that are EQUAL
@@ -2514,38 +2121,18 @@ image_instantiate (Lisp_Object specifier, Lisp_Object matchspec,
             pointer face.
           */
 
-         subtable = make_lisp_hash_table (20,
-                                          pointerp ? HASH_TABLE_KEY_CAR_WEAK
-                                          : HASH_TABLE_KEY_WEAK,
-                                          pointerp ? HASH_TABLE_EQUAL
-                                          : HASH_TABLE_EQ);
+         subtable = make_lisp_hashtable (20,
+                                         pointerp ? HASHTABLE_KEY_CAR_WEAK
+                                         : HASHTABLE_KEY_WEAK,
+                                         pointerp ? HASHTABLE_EQUAL
+                                         : HASHTABLE_EQ);
          Fputhash (make_int (dest_mask), subtable,
                    d->image_instance_cache);
          instance = Qunbound;
        }
       else
-       {
-         instance = Fgethash (pointerp ? ls3 : instantiator,
-                              subtable, Qunbound);
-         /* subwindows have a per-window cache and have to be treated
-            differently.  dest_mask can be a bitwise OR of all image
-            types so we will only catch someone possibly trying to
-            instantiate a subwindow type thing. Unfortunately, this
-            will occur most of the time so this probably slows things
-            down. But with the current design I don't see anyway
-            round it. */
-         if (UNBOUNDP (instance)
-             &&
-             dest_mask & (IMAGE_SUBWINDOW_MASK | IMAGE_WIDGET_MASK))
-           {
-             if (!WINDOWP (domain))
-               signal_simple_error ("Can't instantiate subwindow outside a window",
-                                    instantiator);
-             instance = Fgethash (instantiator, 
-                                  XWINDOW (domain)->subwindow_instance_cache, 
-                                  Qunbound);
-           }
-       }
+       instance = Fgethash (pointerp ? ls3 : instantiator,
+                            subtable, Qunbound);
 
       if (UNBOUNDP (instance))
        {
@@ -2554,7 +2141,7 @@ image_instantiate (Lisp_Object specifier, Lisp_Object matchspec,
                          noseeum_cons (pointerp ? ls3 : instantiator,
                                        subtable));
          int speccount = specpdl_depth ();
-         
+
          /* make sure we cache the failures, too.
             Use an unwind-protect to catch such errors.
             If we fail, the unwind-protect records nil in
@@ -2568,21 +2155,7 @@ image_instantiate (Lisp_Object specifier, Lisp_Object matchspec,
                                                     instantiator,
                                                     pointer_fg, pointer_bg,
                                                     dest_mask);
-         
          Fsetcar (locative, instance);
-         /* only after the image has been instantiated do we know
-             whether we need to put it in the per-window image instance
-             cache. */
-         if (image_instance_type_to_mask (XIMAGE_INSTANCE_TYPE (instance))
-             &
-             (IMAGE_SUBWINDOW_MASK | IMAGE_WIDGET_MASK))
-           {
-             if (!WINDOWP (domain))
-               signal_simple_error ("Can't instantiate subwindow outside a window",
-                                    instantiator);
-             
-             Fsetcdr (XCDR (locative), XWINDOW (domain)->subwindow_instance_cache );
-           }
          unbind_to (speccount, Qnil);
        }
       else
@@ -2782,7 +2355,7 @@ pairs.  FORMAT should be one of
   (Display this image as a text string, with replaceable fields;
   not currently implemented.)
 'xbm
-  (An X bitmap; only if X or Windows support was compiled into this XEmacs.
+  (An X bitmap; only if X support was compiled into this XEmacs.
    Can be instanced as `mono-pixmap', `color-pixmap', or `pointer'.)
 'xpm
   (An XPM pixmap; only if XPM support was compiled into this XEmacs.
@@ -2793,17 +2366,15 @@ pairs.  FORMAT should be one of
   instanced as `mono-pixmap', `color-pixmap', or `pointer'.)
 'gif
   (A GIF87 or GIF89 image; only if GIF support was compiled into this
-   XEmacs.  NOTE: only the first frame of animated gifs will be displayed.
-   Can be instanced as `color-pixmap'.)
+   XEmacs.  Can be instanced as `color-pixmap'.)
 'jpeg
   (A JPEG image; only if JPEG support was compiled into this XEmacs.
    Can be instanced as `color-pixmap'.)
 'png
-  (A PNG image; only if PNG support was compiled into this XEmacs.
+  (A PNG/GIF24 image; only if PNG support was compiled into this XEmacs.
    Can be instanced as `color-pixmap'.)
 'tiff
-  (A TIFF image; only if TIFF support was compiled into this XEmacs.
-   Can be instanced as `color-pixmap'.)
+  (A TIFF image; not currently implemented.)
 'cursor-font
   (One of the standard cursor-font names, such as "watch" or
    "right_ptr" under X.  Under X, this is, more specifically, any
@@ -2817,21 +2388,7 @@ pairs.  FORMAT should be one of
    Currently can only be instanced as `pointer', although this should
    probably be fixed.)
 'subwindow
-  (An embedded windowing system window.)
-'edit-field
-  (A text editing widget glyph.)
-'button
-  (A button widget glyph; either a push button, radio button or toggle button.)
-'tab-control
-  (A tab widget glyph; a series of user selectable tabs.)
-'progress-gauge
-  (A sliding widget glyph, for showing progress.)
-'combo-box
-  (A drop list of selectable items in a widget glyph, for editing text.)
-'label
-  (A static, text-only, widget glyph; for displaying text.)
-'tree-view
-  (A folding widget glyph.)
+  (An embedded X window; not currently implemented.)
 'autodetect
   (XEmacs tries to guess what format the data is in.  If X support
   exists, the data string will be checked to see if it names a filename.
@@ -2862,7 +2419,7 @@ The valid keywords are:
   `cursor-font', `font', `autodetect', and `inherit'.)
 :foreground
 :background
-  (For `xbm', `xface', `cursor-font', `widget' and `font'.  These keywords
+  (For `xbm', `xface', `cursor-font', and `font'.  These keywords
   allow you to explicitly specify foreground and background colors.
   The argument should be anything acceptable to `make-color-instance'.
   This will cause what would be a `mono-pixmap' to instead be colorized
@@ -2893,14 +2450,7 @@ The valid keywords are:
   object).  If this is not specified, the contents of `xpm-color-symbols'
   are used to generate the alist.)
 :face
-  (Only for `inherit'.  This specifies the face to inherit from.
-  For widget glyphs this also specifies the face to use for
-  display. It defaults to gui-element-face.)
-
-Keywords accepted as menu item specs are also accepted by widget
-glyphs. These are `:selected', `:active', `:suffix', `:keys',
-`:style', `:filter', `:config', `:included', `:key-sequence',
-`:accelerator', `:label' and `:callback'.
+  (Only for `inherit'.  This specifies the face to inherit from.)
 
 If instead of a vector, the instantiator is a string, it will be
 converted into a vector by looking it up according to the specs in the
@@ -2928,14 +2478,14 @@ file).
  ****************************************************************************/
 
 static Lisp_Object
-mark_glyph (Lisp_Object obj)
+mark_glyph (Lisp_Object obj, void (*markobj) (Lisp_Object))
 {
   struct Lisp_Glyph *glyph = XGLYPH (obj);
 
-  mark_object (glyph->image);
-  mark_object (glyph->contrib_p);
-  mark_object (glyph->baseline);
-  mark_object (glyph->face);
+  ((markobj) (glyph->image));
+  ((markobj) (glyph->contrib_p));
+  ((markobj) (glyph->baseline));
+  ((markobj) (glyph->face));
 
   return glyph->plist;
 }
@@ -2964,10 +2514,10 @@ print_glyph (Lisp_Object obj, Lisp_Object printcharfun, int escapeflag)
    This isn't concerned with "unspecified" attributes, that's what
    #'glyph-differs-from-default-p is for. */
 static int
-glyph_equal (Lisp_Object obj1, Lisp_Object obj2, int depth)
+glyph_equal (Lisp_Object o1, Lisp_Object o2, int depth)
 {
-  struct Lisp_Glyph *g1 = XGLYPH (obj1);
-  struct Lisp_Glyph *g2 = XGLYPH (obj2);
+  struct Lisp_Glyph *g1 = XGLYPH (o1);
+  struct Lisp_Glyph *g2 = XGLYPH (o2);
 
   depth++;
 
@@ -3005,9 +2555,9 @@ glyph_getprop (Lisp_Object obj, Lisp_Object prop)
 static int
 glyph_putprop (Lisp_Object obj, Lisp_Object prop, Lisp_Object value)
 {
-  if (EQ (prop, Qimage)     ||
-      EQ (prop, Qcontrib_p) ||
-      EQ (prop, Qbaseline))
+  if ((EQ (prop, Qimage))     ||
+      (EQ (prop, Qcontrib_p)) ||
+      (EQ (prop, Qbaseline)))
     return 0;
 
   if (EQ (prop, Qface))
@@ -3023,9 +2573,9 @@ glyph_putprop (Lisp_Object obj, Lisp_Object prop, Lisp_Object value)
 static int
 glyph_remprop (Lisp_Object obj, Lisp_Object prop)
 {
-  if (EQ (prop, Qimage)     ||
-      EQ (prop, Qcontrib_p) ||
-      EQ (prop, Qbaseline))
+  if ((EQ (prop, Qimage))     ||
+      (EQ (prop, Qcontrib_p)) ||
+      (EQ (prop, Qbaseline)))
     return -1;
 
   if (EQ (prop, Qface))
@@ -3051,14 +2601,9 @@ glyph_plist (Lisp_Object obj)
   return result;
 }
 
-static const struct lrecord_description glyph_description[] = {
-  { XD_LISP_OBJECT, offsetof(struct Lisp_Glyph, image), 5 },
-  { XD_END }
-};
-
 DEFINE_LRECORD_IMPLEMENTATION_WITH_PROPS ("glyph", glyph,
                                          mark_glyph, print_glyph, 0,
-                                         glyph_equal, glyph_hash, glyph_description,
+                                         glyph_equal, glyph_hash,
                                          glyph_getprop, glyph_putprop,
                                          glyph_remprop, glyph_plist,
                                          struct Lisp_Glyph);
@@ -3071,19 +2616,16 @@ allocate_glyph (enum glyph_type type,
   /* This function can GC */
   Lisp_Object obj = Qnil;
   struct Lisp_Glyph *g =
-    alloc_lcrecord_type (struct Lisp_Glyph, &lrecord_glyph);
+    alloc_lcrecord_type (struct Lisp_Glyph, lrecord_glyph);
 
   g->type = type;
   g->image = Fmake_specifier (Qimage); /* This function can GC */
-  g->dirty = 0;
   switch (g->type)
     {
     case GLYPH_BUFFER:
       XIMAGE_SPECIFIER_ALLOWED (g->image) =
-       IMAGE_NOTHING_MASK | IMAGE_TEXT_MASK 
-       | IMAGE_MONO_PIXMAP_MASK | IMAGE_COLOR_PIXMAP_MASK 
-       | IMAGE_SUBWINDOW_MASK | IMAGE_WIDGET_MASK
-       | IMAGE_LAYOUT_MASK;
+       IMAGE_NOTHING_MASK | IMAGE_TEXT_MASK | IMAGE_MONO_PIXMAP_MASK |
+         IMAGE_COLOR_PIXMAP_MASK | IMAGE_SUBWINDOW_MASK;
       break;
     case GLYPH_POINTER:
       XIMAGE_SPECIFIER_ALLOWED (g->image) =
@@ -3217,10 +2759,12 @@ The return value will be one of 'buffer, 'pointer, or 'icon.
   CHECK_GLYPH (glyph);
   switch (XGLYPH_TYPE (glyph))
     {
-    default: abort ();
     case GLYPH_BUFFER:  return Qbuffer;
     case GLYPH_POINTER: return Qpointer;
     case GLYPH_ICON:    return Qicon;
+    default:
+      abort ();
+      return Qnil; /* not reached */
     }
 }
 
@@ -3232,16 +2776,18 @@ The return value will be one of 'buffer, 'pointer, or 'icon.
  the given FACE, unless a face is defined by the glyph itself.
  ****************************************************************************/
 unsigned short
-glyph_width (Lisp_Object glyph_or_image, Lisp_Object frame_face,
+glyph_width (Lisp_Object glyph, Lisp_Object frame_face,
             face_index window_findex, Lisp_Object window)
 {
-  Lisp_Object instance = glyph_or_image;
+  Lisp_Object instance;
   Lisp_Object frame = XWINDOW (window)->frame;
 
   /* #### We somehow need to distinguish between the user causing this
      error condition and a bug causing it. */
-  if (GLYPHP (glyph_or_image))
-    instance = glyph_image_instance (glyph_or_image, window, ERROR_ME_NOT, 1);
+  if (!GLYPHP (glyph))
+    return 0;
+  else
+    instance = glyph_image_instance (glyph, window, ERROR_ME_NOT, 1);
 
   if (!IMAGE_INSTANCEP (instance))
     return 0;
@@ -3251,10 +2797,7 @@ glyph_width (Lisp_Object glyph_or_image, Lisp_Object frame_face,
     case IMAGE_TEXT:
       {
        Lisp_Object str = XIMAGE_INSTANCE_TEXT_STRING (instance);
-       Lisp_Object private_face = Qnil;
-
-       if (GLYPHP (glyph_or_image))
-         private_face = XGLYPH_FACE(glyph_or_image);
+        Lisp_Object private_face = XGLYPH_FACE(glyph);
 
        if (!NILP (private_face))
          return redisplay_frame_text_width_string (XFRAME (frame),
@@ -3280,9 +2823,8 @@ glyph_width (Lisp_Object glyph_or_image, Lisp_Object frame_face,
       return 0;
 
     case IMAGE_SUBWINDOW:
-    case IMAGE_WIDGET:
-    case IMAGE_LAYOUT:
-      return XIMAGE_INSTANCE_SUBWINDOW_WIDTH (instance);
+      /* #### implement me */
+      return 0;
 
     default:
       abort ();
@@ -3320,15 +2862,17 @@ glyph_image_instance (Lisp_Object glyph, Lisp_Object domain,
 }
 
 static unsigned short
-glyph_height_internal (Lisp_Object glyph_or_image, Lisp_Object frame_face,
+glyph_height_internal (Lisp_Object glyph, Lisp_Object frame_face,
                       face_index window_findex, Lisp_Object window,
                       int function)
 {
-  Lisp_Object instance = glyph_or_image;
+  Lisp_Object instance;
   Lisp_Object frame = XWINDOW (window)->frame;
 
-  if (GLYPHP (glyph_or_image))
-    instance = glyph_image_instance (glyph_or_image, window, ERROR_ME_NOT, 1);
+  if (!GLYPHP (glyph))
+    return 0;
+  else
+    instance = glyph_image_instance (glyph, window, ERROR_ME_NOT, 1);
 
   if (!IMAGE_INSTANCEP (instance))
     return 0;
@@ -3383,13 +2927,8 @@ glyph_height_internal (Lisp_Object glyph_or_image, Lisp_Object frame_face,
       return 0;
 
     case IMAGE_SUBWINDOW:
-    case IMAGE_WIDGET:
-    case IMAGE_LAYOUT:
-      /* #### Ugh ugh ugh -- temporary crap */
-      if (function == RETURN_ASCENT || function == RETURN_HEIGHT)
-       return XIMAGE_INSTANCE_SUBWINDOW_HEIGHT (instance);
-      else
-       return 0;
+      /* #### implement me */
+      return 0;
 
     default:
       abort ();
@@ -3466,35 +3005,6 @@ that redisplay will.
 #undef RETURN_DESCENT
 #undef RETURN_HEIGHT
 
-static unsigned int
-glyph_dirty_p (Lisp_Object glyph_or_image, Lisp_Object window)
-{
-  Lisp_Object instance = glyph_or_image;
-
-  if (GLYPHP (glyph_or_image))
-    instance = glyph_image_instance (glyph_or_image, window, ERROR_ME_NOT, 1);
-
-  return XIMAGE_INSTANCE_DIRTYP (instance);
-}
-
-static void
-set_glyph_dirty_p (Lisp_Object glyph_or_image, Lisp_Object window, int dirty)
-{
-  Lisp_Object instance = glyph_or_image;
-
-  if (!NILP (glyph_or_image))
-    {
-      if (GLYPHP (glyph_or_image))
-       {
-         instance = glyph_image_instance (glyph_or_image, window,
-                                          ERROR_ME_NOT, 1);
-         XGLYPH_DIRTYP (glyph_or_image) = dirty;
-       }
-
-      XIMAGE_INSTANCE_DIRTYP (instance) = dirty;
-    }
-}
-
 /* #### do we need to cache this info to speed things up? */
 
 Lisp_Object
@@ -3557,10 +3067,12 @@ glyph_property_was_changed (Lisp_Object glyph, Lisp_Object property,
 /*
  #### All of this is 95% copied from face cachels.
       Consider consolidating.
+ #### We need to add a dirty flag to the glyphs.
  */
 
 void
-mark_glyph_cachels (glyph_cachel_dynarr *elements)
+mark_glyph_cachels (glyph_cachel_dynarr *elements,
+                   void (*markobj) (Lisp_Object))
 {
   int elt;
 
@@ -3570,7 +3082,7 @@ mark_glyph_cachels (glyph_cachel_dynarr *elements)
   for (elt = 0; elt < Dynarr_length (elements); elt++)
     {
       struct glyph_cachel *cachel = Dynarr_atp (elements, elt);
-      mark_object (cachel->glyph);
+      ((markobj) (cachel->glyph));
     }
 }
 
@@ -3578,21 +3090,19 @@ static void
 update_glyph_cachel_data (struct window *w, Lisp_Object glyph,
                          struct glyph_cachel *cachel)
 {
-  if (!cachel->updated || NILP (cachel->glyph) || !EQ (cachel->glyph, glyph)
-      || XGLYPH_DIRTYP (cachel->glyph))
+  /* #### This should be || !cachel->updated */
+  if (NILP (cachel->glyph) || !EQ (cachel->glyph, glyph))
     {
-      Lisp_Object window, instance;
+      Lisp_Object window;
 
       XSETWINDOW (window, w);
 
+    /* #### This could be sped up if we redid things to grab the glyph
+       instantiation and passed it to the size functions. */
       cachel->glyph   = glyph;
-    /* Speed things up slightly by grabbing the glyph instantiation
-       and passing it to the size functions. */
-      instance = glyph_image_instance (glyph, window, ERROR_ME_NOT, 1);
-      cachel->dirty = XGLYPH_DIRTYP (glyph) = glyph_dirty_p (glyph, window);
-      cachel->width   = glyph_width   (instance, Qnil, DEFAULT_INDEX, window);
-      cachel->ascent  = glyph_ascent  (instance, Qnil, DEFAULT_INDEX, window);
-      cachel->descent = glyph_descent (instance, Qnil, DEFAULT_INDEX, window);
+      cachel->width   = glyph_width   (glyph, Qnil, DEFAULT_INDEX, window);
+      cachel->ascent  = glyph_ascent  (glyph, Qnil, DEFAULT_INDEX, window);
+      cachel->descent = glyph_descent (glyph, Qnil, DEFAULT_INDEX, window);
     }
 
   cachel->updated = 1;
@@ -3610,7 +3120,7 @@ add_glyph_cachel (struct window *w, Lisp_Object glyph)
   Dynarr_add (w->glyph_cachels, new_cachel);
 }
 
-glyph_index
+static glyph_index
 get_glyph_cachel_index (struct window *w, Lisp_Object glyph)
 {
   int elt;
@@ -3625,7 +3135,8 @@ get_glyph_cachel_index (struct window *w, Lisp_Object glyph)
 
       if (EQ (cachel->glyph, glyph) && !NILP (glyph))
        {
-         update_glyph_cachel_data (w, glyph, cachel);
+         if (!cachel->updated)
+           update_glyph_cachel_data (w, glyph, cachel);
          return elt;
        }
     }
@@ -3668,24 +3179,7 @@ mark_glyph_cachels_as_not_updated (struct window *w)
 #undef FROB
 
   for (elt = 0; elt < Dynarr_length (w->glyph_cachels); elt++)
-    {
-      Dynarr_atp (w->glyph_cachels, elt)->updated = 0;
-    }
-}
-
-/* Unset the dirty bit on all the glyph cachels that have it. */
-void 
-mark_glyph_cachels_as_clean (struct window* w)
-{
-  int elt;
-  Lisp_Object window;
-  XSETWINDOW (window, w);
-  for (elt = 0; elt < Dynarr_length (w->glyph_cachels); elt++)
-    {
-      struct glyph_cachel *cachel = Dynarr_atp (w->glyph_cachels, elt);
-      cachel->dirty = 0;
-      set_glyph_dirty_p (cachel->glyph, window, 0);
-    }
+    Dynarr_atp (w->glyph_cachels, elt)->updated = 0;
 }
 
 #ifdef MEMORY_USAGE_STATS
@@ -3704,635 +3198,34 @@ compute_glyph_cachel_usage (glyph_cachel_dynarr *glyph_cachels,
 
 #endif /* MEMORY_USAGE_STATS */
 
-
 \f
 /*****************************************************************************
- *                     subwindow cachel functions                                   *
+ *                              display tables                               *
  *****************************************************************************/
-/* subwindows are curious in that you have to physically unmap them to
-   not display them. It is problematic deciding what to do in
-   redisplay. We have two caches - a per-window instance cache that
-   keeps track of subwindows on a window, these are linked to their
-   instantiator in the hashtable and when the instantiator goes away
-   we want the instance to go away also. However we also have a
-   per-frame instance cache that we use to determine if a subwindow is
-   obscuring an area that we want to clear. We need to be able to flip
-   through this quickly so a hashtable is not suitable hence the
-   subwindow_cachels. The question is should we just not mark
-   instances in the subwindow_cachels or should we try and invalidate
-   the cache at suitable points in redisplay? If we don't invalidate
-   the cache it will fill up with crud that will only get removed when
-   the frame is deleted. So invalidation is good, the question is when
-   and whether we mark as well. Go for the simple option - don't mark,
-   MARK_SUBWINDOWS_CHANGED when a subwindow gets deleted. */
 
-void
-mark_subwindow_cachels (subwindow_cachel_dynarr *elements)
-{
-  int elt;
+/* Get the display table for use currently on window W with face FACE.
+   Precedence:
 
-  if (!elements)
-    return;
+   -- FACE's display table
+   -- W's display table (comes from specifier `current-display-table')
 
-  for (elt = 0; elt < Dynarr_length (elements); elt++)
-    {
-      struct subwindow_cachel *cachel = Dynarr_atp (elements, elt);
-      mark_object (cachel->subwindow);
-    }
-}
+   Ignore the specified tables if they are not valid;
+   if no valid table is specified, return 0.  */
 
-static void
-update_subwindow_cachel_data (struct frame *f, Lisp_Object subwindow,
-                         struct subwindow_cachel *cachel)
-{
-  cachel->subwindow   = subwindow;
-  cachel->width   = XIMAGE_INSTANCE_SUBWINDOW_WIDTH (subwindow);
-  cachel->height   = XIMAGE_INSTANCE_SUBWINDOW_HEIGHT (subwindow);
-  cachel->updated = 1;
-}
-
-static void
-add_subwindow_cachel (struct frame *f, Lisp_Object subwindow)
-{
-  struct subwindow_cachel new_cachel;
-
-  xzero (new_cachel);
-  new_cachel.subwindow = Qnil;
-  new_cachel.x=0;
-  new_cachel.y=0;
-  new_cachel.being_displayed=0;
-
-  update_subwindow_cachel_data (f, subwindow, &new_cachel);
-  Dynarr_add (f->subwindow_cachels, new_cachel);
-}
-
-static int
-get_subwindow_cachel_index (struct frame *f, Lisp_Object subwindow)
-{
-  int elt;
-
-  if (noninteractive)
-    return 0;
-
-  for (elt = 0; elt < Dynarr_length (f->subwindow_cachels); elt++)
-    {
-      struct subwindow_cachel *cachel =
-       Dynarr_atp (f->subwindow_cachels, elt);
-
-      if (EQ (cachel->subwindow, subwindow) && !NILP (subwindow))
-       {
-         if (!cachel->updated)
-           update_subwindow_cachel_data (f, subwindow, cachel);
-         return elt;
-       }
-    }
-
-  /* If we didn't find the glyph, add it and then return its index. */
-  add_subwindow_cachel (f, subwindow);
-  return elt;
-}
-
-static void
-update_subwindow_cachel (Lisp_Object subwindow)
-{
-  struct frame* f;
-  int elt;
-
-  if (NILP (subwindow))
-    return;
-
-  f = XFRAME ( XIMAGE_INSTANCE_SUBWINDOW_FRAME (subwindow));
-
-  for (elt = 0; elt < Dynarr_length (f->subwindow_cachels); elt++)
-    {
-      struct subwindow_cachel *cachel =
-       Dynarr_atp (f->subwindow_cachels, elt);
-      
-      if (EQ (cachel->subwindow, subwindow) && !NILP (subwindow))
-       {
-         update_subwindow_cachel_data (f, subwindow, cachel);
-       }
-    }
-}
-
-/* redisplay in general assumes that drawing something will erase
-   what was there before. unfortunately this does not apply to
-   subwindows that need to be specifically unmapped in order to
-   disappear. we take a brute force approach - on the basis that its
-   cheap - and unmap all subwindows in a display line */
-void
-reset_subwindow_cachels (struct frame *f)
-{
-  int elt;
-  for (elt = 0; elt < Dynarr_length (f->subwindow_cachels); elt++)
-    {
-      struct subwindow_cachel *cachel =
-       Dynarr_atp (f->subwindow_cachels, elt);
-
-      if (!NILP (cachel->subwindow) && cachel->being_displayed)
-       {
-         cachel->updated = 1;
-         /* #### This is not optimal as update_subwindow will search
-             the cachels for ourselves as well. We could easily optimize. */
-         unmap_subwindow (cachel->subwindow);
-       }
-    }
-  Dynarr_reset (f->subwindow_cachels);
-}
-
-void
-mark_subwindow_cachels_as_not_updated (struct frame *f)
-{
-  int elt;
-
-  for (elt = 0; elt < Dynarr_length (f->subwindow_cachels); elt++)
-    Dynarr_atp (f->subwindow_cachels, elt)->updated = 0;
-}
-
-
-
-/*****************************************************************************
- *                              subwindow exposure ignorance                    *
- *****************************************************************************/
-/* when we unmap subwindows the associated window system will generate
-   expose events. This we do not want as redisplay already copes with
-   the repainting necessary. Worse, we can get in an endless cycle of
-   redisplay if we are not careful. Thus we keep a per-frame list of
-   expose events that are going to come and ignore them as
-   required. */
-
-struct expose_ignore_blocktype
-{
-  Blocktype_declare (struct expose_ignore);
-} *the_expose_ignore_blocktype;
-
-int
-check_for_ignored_expose (struct frame* f, int x, int y, int width, int height)
-{
-  struct expose_ignore *ei, *prev;
-  /* the ignore list is FIFO so we should generally get a match with
-     the first element in the list */
-  for (ei = f->subwindow_exposures, prev = 0; ei; ei = ei->next)
-    {
-      /* Checking for exact matches just isn't good enough as we
-        mighte get exposures for partially obscure subwindows, thus
-        we have to check for overlaps. Being conservative we will
-        check for exposures wholly contained by the subwindow, this
-        might give us what we want.*/
-      if (ei->x <= x && ei->y <= y 
-         && ei->x + ei->width >= x + width
-         && ei->y + ei->height >= y + height)
-       {
-#ifdef DEBUG_WIDGETS
-         stderr_out ("ignored %d+%d, %dx%d for exposure %d+%d, %dx%d\n",
-                     x, y, width, height, ei->x, ei->y, ei->width, ei->height);
-#endif
-         if (!prev)
-           f->subwindow_exposures = ei->next;
-         else
-           prev->next = ei->next;
-         
-         if (ei == f->subwindow_exposures_tail)
-           f->subwindow_exposures_tail = prev;
-
-         Blocktype_free (the_expose_ignore_blocktype, ei);
-         return 1;
-       }
-      prev = ei;
-    }
-  return 0;
-}
-
-static void
-register_ignored_expose (struct frame* f, int x, int y, int width, int height)
-{
-  if (!hold_ignored_expose_registration)
-    {
-      struct expose_ignore *ei;
-      
-      ei = Blocktype_alloc (the_expose_ignore_blocktype);
-      
-      ei->next = NULL;
-      ei->x = x;
-      ei->y = y;
-      ei->width = width;
-      ei->height = height;
-      
-      /* we have to add the exposure to the end of the list, since we
-        want to check the oldest events first. for speed we keep a record
-        of the end so that we can add right to it. */
-      if (f->subwindow_exposures_tail)
-       {
-         f->subwindow_exposures_tail->next = ei;
-       }
-      if (!f->subwindow_exposures)
-       {
-         f->subwindow_exposures = ei;
-       }
-      f->subwindow_exposures_tail = ei;
-    }
-}
-
-/****************************************************************************
- find_matching_subwindow
-
- See if there is a subwindow that completely encloses the requested
- area.
- ****************************************************************************/
-int find_matching_subwindow (struct frame* f, int x, int y, int width, int height)
-{
-  int elt;
-
-  for (elt = 0; elt < Dynarr_length (f->subwindow_cachels); elt++)
-    {
-      struct subwindow_cachel *cachel =
-       Dynarr_atp (f->subwindow_cachels, elt);
-
-      if (cachel->being_displayed
-         &&
-         cachel->x <= x && cachel->y <= y
-         && 
-         cachel->x + cachel->width >= x + width
-         &&
-         cachel->y + cachel->height >= y + height)
-       {
-         return 1;
-       }
-    }
-  return 0;
-}
-
-\f
-/*****************************************************************************
- *                              subwindow functions                          *
- *****************************************************************************/
-
-/* update the displayed characteristics of a subwindow */
-static void
-update_subwindow (Lisp_Object subwindow)
-{
-  struct Lisp_Image_Instance* ii = XIMAGE_INSTANCE (subwindow);
-
-  if (!IMAGE_INSTANCE_TYPE (ii) == IMAGE_WIDGET
-      ||
-      NILP (IMAGE_INSTANCE_SUBWINDOW_FRAME (ii)))
-    return;
-
-  MAYBE_DEVMETH (XDEVICE (ii->device), update_subwindow, (ii));
-}
-
-void
-update_frame_subwindows (struct frame *f)
-{
-  int elt;
-
-  if (f->subwindows_changed || f->subwindows_state_changed || f->faces_changed)
-    for (elt = 0; elt < Dynarr_length (f->subwindow_cachels); elt++)
-      {
-       struct subwindow_cachel *cachel =
-         Dynarr_atp (f->subwindow_cachels, elt);
-       
-       if (cachel->being_displayed)
-         {
-           update_subwindow (cachel->subwindow);
-         }
-      }
-}
-
-/* remove a subwindow from its frame */
-void unmap_subwindow (Lisp_Object subwindow)
-{
-  struct Lisp_Image_Instance* ii = XIMAGE_INSTANCE (subwindow);
-  int elt;
-  struct subwindow_cachel* cachel;
-  struct frame* f;
-
-  if (!(IMAGE_INSTANCE_TYPE (ii) == IMAGE_WIDGET
-       ||
-       IMAGE_INSTANCE_TYPE (ii) == IMAGE_SUBWINDOW)
-      ||
-      NILP (IMAGE_INSTANCE_SUBWINDOW_FRAME (ii)))
-    return;
-#ifdef DEBUG_WIDGETS
-  stderr_out ("unmapping subwindow %d\n", IMAGE_INSTANCE_SUBWINDOW_ID (ii));
-#endif
-  f = XFRAME (IMAGE_INSTANCE_SUBWINDOW_FRAME (ii));
-  elt = get_subwindow_cachel_index (f, subwindow);
-  cachel = Dynarr_atp (f->subwindow_cachels, elt);
-
-  /* make sure we don't get expose events */
-  register_ignored_expose (f, cachel->x, cachel->y, cachel->width, cachel->height);
-  cachel->x = -1;
-  cachel->y = -1;
-  cachel->being_displayed = 0;
-  IMAGE_INSTANCE_SUBWINDOW_DISPLAYEDP (ii) = 0;
-
-  MAYBE_DEVMETH (XDEVICE (ii->device), unmap_subwindow, (ii));
-}
-
-/* show a subwindow in its frame */
-void map_subwindow (Lisp_Object subwindow, int x, int y,
-                   struct display_glyph_area *dga)
-{
-  struct Lisp_Image_Instance* ii = XIMAGE_INSTANCE (subwindow);
-  int elt; 
-  struct subwindow_cachel* cachel;
-  struct frame* f;
-
-  if (!(IMAGE_INSTANCE_TYPE (ii) == IMAGE_WIDGET
-       ||
-       IMAGE_INSTANCE_TYPE (ii) == IMAGE_SUBWINDOW)
-      ||
-      NILP (IMAGE_INSTANCE_SUBWINDOW_FRAME (ii)))
-    return;
-
-#ifdef DEBUG_WIDGETS
-  stderr_out ("mapping subwindow %d, %dx%d@%d+%d\n",
-             IMAGE_INSTANCE_SUBWINDOW_ID (ii),
-             dga->width, dga->height, x, y);
-#endif
-  f = XFRAME (IMAGE_INSTANCE_SUBWINDOW_FRAME (ii));
-  IMAGE_INSTANCE_SUBWINDOW_DISPLAYEDP (ii) = 1;
-  elt = get_subwindow_cachel_index (f, subwindow);
-  cachel = Dynarr_atp (f->subwindow_cachels, elt);
-  cachel->x = x;
-  cachel->y = y;
-  cachel->width = dga->width;
-  cachel->height = dga->height;
-  cachel->being_displayed = 1;
-
-  MAYBE_DEVMETH (XDEVICE (ii->device), map_subwindow, (ii, x, y, dga));
-}
-
-static int
-subwindow_possible_dest_types (void)
-{
-  return IMAGE_SUBWINDOW_MASK;
-}
-
-/* Partially instantiate a subwindow. */
-void
-subwindow_instantiate (Lisp_Object image_instance, Lisp_Object instantiator,
-                      Lisp_Object pointer_fg, Lisp_Object pointer_bg,
-                      int dest_mask, Lisp_Object domain)
-{
-  struct Lisp_Image_Instance *ii = XIMAGE_INSTANCE (image_instance);
-  Lisp_Object device = IMAGE_INSTANCE_DEVICE (ii);
-  Lisp_Object frame = FW_FRAME (domain);
-  Lisp_Object width = find_keyword_in_vector (instantiator, Q_pixel_width);
-  Lisp_Object height = find_keyword_in_vector (instantiator, Q_pixel_height);
-
-  if (NILP (frame))
-    signal_simple_error ("No selected frame", device);
-  
-  if (!(dest_mask & IMAGE_SUBWINDOW_MASK))
-    incompatible_image_types (instantiator, dest_mask, IMAGE_SUBWINDOW_MASK);
-
-  ii->data = 0;
-  IMAGE_INSTANCE_SUBWINDOW_ID (ii) = 0;
-  IMAGE_INSTANCE_SUBWINDOW_DISPLAYEDP (ii) = 0;
-  IMAGE_INSTANCE_SUBWINDOW_FRAME (ii) = frame;
-
-  /* this stuff may get overidden by the widget code */
-  if (NILP (width))
-    IMAGE_INSTANCE_SUBWINDOW_WIDTH (ii) = 20;
-  else
-    {
-      int w = 1;
-      CHECK_INT (width);
-      if (XINT (width) > 1)
-       w = XINT (width);
-      IMAGE_INSTANCE_SUBWINDOW_WIDTH (ii) = w;
-    }
-  if (NILP (height))
-    IMAGE_INSTANCE_SUBWINDOW_HEIGHT (ii) = 20;
-  else
-    {
-      int h = 1;
-      CHECK_INT (height);
-      if (XINT (height) > 1)
-       h = XINT (height);
-      IMAGE_INSTANCE_SUBWINDOW_HEIGHT (ii) = h;
-    }
-}
-
-DEFUN ("subwindowp", Fsubwindowp, 1, 1, 0, /*
-Return non-nil if OBJECT is a subwindow.
-*/
-       (object))
-{
-  CHECK_IMAGE_INSTANCE (object);
-  return (XIMAGE_INSTANCE_TYPE (object) == IMAGE_SUBWINDOW) ? Qt : Qnil;
-}
-
-DEFUN ("image-instance-subwindow-id", Fimage_instance_subwindow_id, 1, 1, 0, /*
-Return the window id of SUBWINDOW as a number.
-*/
-       (subwindow))
-{
-  CHECK_SUBWINDOW_IMAGE_INSTANCE (subwindow);
-  return make_int ((int) XIMAGE_INSTANCE_SUBWINDOW_ID (subwindow));
-}
-
-DEFUN ("resize-subwindow", Fresize_subwindow, 1, 3, 0, /*
-Resize SUBWINDOW to WIDTH x HEIGHT.
-If a value is nil that parameter is not changed.
-*/
-       (subwindow, width, height))
-{
-  int neww, newh;
-
-  CHECK_SUBWINDOW_IMAGE_INSTANCE (subwindow);
-
-  if (NILP (width))
-    neww = XIMAGE_INSTANCE_SUBWINDOW_WIDTH (subwindow);
-  else
-    neww = XINT (width);
-
-  if (NILP (height))
-    newh = XIMAGE_INSTANCE_SUBWINDOW_HEIGHT (subwindow);
-  else
-    newh = XINT (height);
-
-  
-  MAYBE_DEVMETH (XDEVICE (XIMAGE_INSTANCE_DEVICE (subwindow)), 
-                resize_subwindow, (XIMAGE_INSTANCE (subwindow), neww, newh));
-
-  XIMAGE_INSTANCE_SUBWINDOW_HEIGHT (subwindow) = newh;
-  XIMAGE_INSTANCE_SUBWINDOW_WIDTH (subwindow) = neww;
-
-  /* need to update the cachels as redisplay will not do this */
-  update_subwindow_cachel (subwindow);
-
-  return subwindow;
-}
-
-DEFUN ("force-subwindow-map", Fforce_subwindow_map, 1, 1, 0, /*
-Generate a Map event for SUBWINDOW.
-*/
-       (subwindow))
-{
-  CHECK_SUBWINDOW_IMAGE_INSTANCE (subwindow);
-#if 0
-  map_subwindow (subwindow, 0, 0);
-#endif
-  return subwindow;
-}
-
-\f
-/*****************************************************************************
- *                              display tables                               *
- *****************************************************************************/
-
-/* Get the display tables for use currently on window W with face
-   FACE.  #### This will have to be redone.  */
-
-void
-get_display_tables (struct window *w, face_index findex,
-                   Lisp_Object *face_table, Lisp_Object *window_table)
+struct Lisp_Vector *
+get_display_table (struct window *w, face_index findex)
 {
   Lisp_Object tem;
-  tem = WINDOW_FACE_CACHEL_DISPLAY_TABLE (w, findex);
-  if (UNBOUNDP (tem))
-    tem = Qnil;
-  if (!LISTP (tem))
-    tem = noseeum_cons (tem, Qnil);
-  *face_table = tem;
-  tem = w->display_table;
-  if (UNBOUNDP (tem))
-    tem = Qnil;
-  if (!LISTP (tem))
-    tem = noseeum_cons (tem, Qnil);
-  *window_table = tem;
-}
-
-Lisp_Object
-display_table_entry (Emchar ch, Lisp_Object face_table,
-                    Lisp_Object window_table)
-{
-  Lisp_Object tail;
 
-  /* Loop over FACE_TABLE, and then over WINDOW_TABLE. */
-  for (tail = face_table; 1; tail = XCDR (tail))
-    {
-      Lisp_Object table;
-      if (NILP (tail))
-       {
-         if (!NILP (window_table))
-           {
-             tail = window_table;
-             window_table = Qnil;
-           }
-         else
-           return Qnil;
-       }
-      table = XCAR (tail);
-
-      if (VECTORP (table))
-       {
-         if (ch < XVECTOR_LENGTH (table) && !NILP (XVECTOR_DATA (table)[ch]))
-           return XVECTOR_DATA (table)[ch];
-         else
-           continue;
-       }
-      else if (CHAR_TABLEP (table)
-              && XCHAR_TABLE_TYPE (table) == CHAR_TABLE_TYPE_CHAR)
-       {
-         return get_char_table (ch, XCHAR_TABLE (table));
-       }
-      else if (CHAR_TABLEP (table)
-              && XCHAR_TABLE_TYPE (table) == CHAR_TABLE_TYPE_GENERIC)
-       {
-         Lisp_Object gotit = get_char_table (ch, XCHAR_TABLE (table));
-         if (!NILP (gotit))
-           return gotit;
-         else
-           continue;
-       }
-      else if (RANGE_TABLEP (table))
-       {
-         Lisp_Object gotit = Fget_range_table (make_char (ch), table, Qnil);
-         if (!NILP (gotit))
-           return gotit;
-         else
-           continue;
-       }
-      else
-       abort ();
-    }
-}
-
-/*****************************************************************************
- *                              timeouts for animated glyphs                      *
- *****************************************************************************/
-static Lisp_Object Qglyph_animated_timeout_handler;
-
-DEFUN ("glyph-animated-timeout-handler", Fglyph_animated_timeout_handler, 1, 1, 0, /*
-Callback function for updating animated images.
-Don't use this.
-*/
-       (arg))
-{
-  CHECK_WEAK_LIST (arg);
-
-  if (!NILP (XWEAK_LIST_LIST (arg)) && !NILP (XCAR (XWEAK_LIST_LIST (arg))))
-    {
-      Lisp_Object value = XCAR (XWEAK_LIST_LIST (arg));
-      
-      if (IMAGE_INSTANCEP (value))
-       {
-         struct Lisp_Image_Instance* ii = XIMAGE_INSTANCE (value);
-
-         if (COLOR_PIXMAP_IMAGE_INSTANCEP (value)
-             &&
-             IMAGE_INSTANCE_PIXMAP_MAXSLICE (ii) > 1
-             &&
-             !disable_animated_pixmaps)
-           {
-             /* Increment the index of the image slice we are currently
-                viewing. */
-             IMAGE_INSTANCE_PIXMAP_SLICE (ii) =
-               (IMAGE_INSTANCE_PIXMAP_SLICE (ii) + 1)
-               % IMAGE_INSTANCE_PIXMAP_MAXSLICE (ii);
-             /* We might need to kick redisplay at this point - but we
-                also might not. */
-             MARK_DEVICE_FRAMES_GLYPHS_CHANGED 
-               (XDEVICE (IMAGE_INSTANCE_DEVICE (ii)));
-             IMAGE_INSTANCE_DIRTYP (ii) = 1;
-           }
-       }
-    }
-  return Qnil;
-}
-
-Lisp_Object add_glyph_animated_timeout (EMACS_INT tickms, Lisp_Object image)
-{
-  Lisp_Object ret = Qnil;
-
-  if (tickms > 0 && IMAGE_INSTANCEP (image))
-    {
-      double ms = ((double)tickms) / 1000.0;
-      struct gcpro gcpro1;
-      Lisp_Object holder = make_weak_list (WEAK_LIST_SIMPLE);
-
-      GCPRO1 (holder);
-      XWEAK_LIST_LIST (holder) = Fcons (image, Qnil);
-
-      ret = Fadd_timeout (make_float (ms),
-                         Qglyph_animated_timeout_handler,
-                         holder, make_float (ms));
-
-      UNGCPRO;
-    }
-  return ret;
-}
+  tem = WINDOW_FACE_CACHEL_DISPLAY_TABLE (w, findex);
+  if (VECTORP (tem) && XVECTOR_LENGTH (tem) == DISP_TABLE_SIZE)
+    return XVECTOR (tem);
 
-void disable_glyph_animated_timeout (int i)
-{
-  Lisp_Object id;
-  XSETINT (id, i);
+  tem = w->display_table;
+  if (VECTORP (tem) && XVECTOR_LENGTH (tem) == DISP_TABLE_SIZE)
+    return XVECTOR (tem);
 
-  Fdisable_timeout (id);
+  return 0;
 }
 
 \f
@@ -4353,8 +3246,6 @@ syms_of_glyphs (void)
   defkeyword (&Q_file, ":file");
   defkeyword (&Q_data, ":data");
   defkeyword (&Q_face, ":face");
-  defkeyword (&Q_pixel_height, ":pixel-height");
-  defkeyword (&Q_pixel_width, ":pixel-width");
 
 #ifdef HAVE_XPM
   defkeyword (&Q_color_symbols, ":color-symbols");
@@ -4381,9 +3272,7 @@ syms_of_glyphs (void)
   defsymbol (&Qmono_pixmap_image_instance_p, "mono-pixmap-image-instance-p");
   defsymbol (&Qcolor_pixmap_image_instance_p, "color-pixmap-image-instance-p");
   defsymbol (&Qpointer_image_instance_p, "pointer-image-instance-p");
-  defsymbol (&Qwidget_image_instance_p, "widget-image-instance-p");
   defsymbol (&Qsubwindow_image_instance_p, "subwindow-image-instance-p");
-  defsymbol (&Qlayout_image_instance_p, "layout-image-instance-p");
 
   DEFSUBR (Fmake_image_instance);
   DEFSUBR (Fimage_instance_p);
@@ -4401,14 +3290,7 @@ syms_of_glyphs (void)
   DEFSUBR (Fimage_instance_hotspot_y);
   DEFSUBR (Fimage_instance_foreground);
   DEFSUBR (Fimage_instance_background);
-  DEFSUBR (Fimage_instance_property);
-  DEFSUBR (Fset_image_instance_property);
   DEFSUBR (Fcolorize_image_instance);
-  /* subwindows */
-  DEFSUBR (Fsubwindowp);
-  DEFSUBR (Fimage_instance_subwindow_id);
-  DEFSUBR (Fresize_subwindow);
-  DEFSUBR (Fforce_subwindow_map);
 
   /* Qnothing defined as part of the "nothing" image-instantiator
      type. */
@@ -4416,6 +3298,7 @@ syms_of_glyphs (void)
   defsymbol (&Qmono_pixmap, "mono-pixmap");
   defsymbol (&Qcolor_pixmap, "color-pixmap");
   /* Qpointer defined in general.c */
+  defsymbol (&Qsubwindow, "subwindow");
 
   /* glyphs */
 
@@ -4442,12 +3325,6 @@ syms_of_glyphs (void)
   /* Qbuffer defined in general.c. */
   /* Qpointer defined above */
 
-  /* Unfortunately, timeout handlers must be lisp functions. This is
-     for animated glyphs. */
-  defsymbol (&Qglyph_animated_timeout_handler,
-             "glyph-animated-timeout-handler");
-  DEFSUBR (Fglyph_animated_timeout_handler);
-
   /* Errors */
   deferror (&Qimage_conversion_error,
            "image-conversion-error",
@@ -4455,11 +3332,6 @@ syms_of_glyphs (void)
 
 }
 
-static const struct lrecord_description image_specifier_description[] = {
-  { XD_LISP_OBJECT, specifier_data_offset + offsetof(struct image_specifier, attachee), 2 },
-  { XD_END }
-};
-
 void
 specifier_type_create_image (void)
 {
@@ -4476,66 +3348,6 @@ specifier_type_create_image (void)
 }
 
 void
-reinit_specifier_type_create_image (void)
-{
-  REINITIALIZE_SPECIFIER_TYPE (image);
-}
-
-
-static const struct lrecord_description iike_description_1[] = {
-  { XD_LISP_OBJECT, offsetof(ii_keyword_entry, keyword), 1 },
-  { XD_END }
-};
-
-static const struct struct_description iike_description = {
-  sizeof(ii_keyword_entry),
-  iike_description_1
-};
-
-static const struct lrecord_description iiked_description_1[] = {
-  XD_DYNARR_DESC(ii_keyword_entry_dynarr, &iike_description),
-  { XD_END }
-};
-
-static const struct struct_description iiked_description = {
-  sizeof(ii_keyword_entry_dynarr),
-  iiked_description_1
-};
-
-static const struct lrecord_description iife_description_1[] = {
-  { XD_LISP_OBJECT, offsetof(image_instantiator_format_entry, symbol), 2 },
-  { XD_STRUCT_PTR,  offsetof(image_instantiator_format_entry, meths),  1, &iim_description },
-  { XD_END }
-};
-
-static const struct struct_description iife_description = {
-  sizeof(image_instantiator_format_entry),
-  iife_description_1
-};
-
-static const struct lrecord_description iifed_description_1[] = {
-  XD_DYNARR_DESC(image_instantiator_format_entry_dynarr, &iife_description),
-  { XD_END }
-};
-
-static const struct struct_description iifed_description = {
-  sizeof(image_instantiator_format_entry_dynarr),
-  iifed_description_1
-};
-
-static const struct lrecord_description iim_description_1[] = {
-  { XD_LISP_OBJECT, offsetof(struct image_instantiator_methods, symbol), 2 },
-  { XD_STRUCT_PTR,  offsetof(struct image_instantiator_methods, keywords), 1, &iiked_description },
-  { XD_STRUCT_PTR,  offsetof(struct image_instantiator_methods, consoles), 1, &cted_description },
-  { XD_END }
-};
-
-const struct struct_description iim_description = {
-  sizeof(struct image_instantiator_methods),
-  iim_description_1
-};
-
-void
 image_instantiator_format_create (void)
 {
   /* image instantiators */
@@ -4546,8 +3358,6 @@ image_instantiator_format_create (void)
   Vimage_instantiator_format_list = Qnil;
   staticpro (&Vimage_instantiator_format_list);
 
-  dumpstruct (&the_image_instantiator_format_entry_dynarr, &iifed_description);
-
   INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (nothing, "nothing");
 
   IIFORMAT_HAS_METHOD (nothing, possible_dest_types);
@@ -4569,23 +3379,14 @@ image_instantiator_format_create (void)
   IIFORMAT_HAS_METHOD (string, instantiate);
 
   IIFORMAT_VALID_KEYWORD (string, Q_data, check_valid_string);
-  /* Do this so we can set strings. */
-  INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (text, "text");
-  IIFORMAT_HAS_METHOD (text, set_property);
 
   INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (formatted_string, "formatted-string");
 
   IIFORMAT_HAS_METHOD (formatted_string, validate);
   IIFORMAT_HAS_METHOD (formatted_string, possible_dest_types);
   IIFORMAT_HAS_METHOD (formatted_string, instantiate);
-  IIFORMAT_VALID_KEYWORD (formatted_string, Q_data, check_valid_string);
 
-  /* subwindows */
-  INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (subwindow, "subwindow");
-  IIFORMAT_HAS_METHOD (subwindow, possible_dest_types);
-  IIFORMAT_HAS_METHOD (subwindow, instantiate);
-  IIFORMAT_VALID_KEYWORD (subwindow, Q_pixel_width, check_valid_int);
-  IIFORMAT_VALID_KEYWORD (subwindow, Q_pixel_height, check_valid_int);
+  IIFORMAT_VALID_KEYWORD (formatted_string, Q_data, check_valid_string);
 
 #ifdef HAVE_WINDOW_SYSTEM
   INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (xbm, "xbm");
@@ -4593,6 +3394,7 @@ image_instantiator_format_create (void)
   IIFORMAT_HAS_METHOD (xbm, validate);
   IIFORMAT_HAS_METHOD (xbm, normalize);
   IIFORMAT_HAS_METHOD (xbm, possible_dest_types);
+  IIFORMAT_HAS_METHOD (xbm, instantiate);
 
   IIFORMAT_VALID_KEYWORD (xbm, Q_data, check_valid_xbm_inline);
   IIFORMAT_VALID_KEYWORD (xbm, Q_file, check_valid_string);
@@ -4604,27 +3406,13 @@ image_instantiator_format_create (void)
   IIFORMAT_VALID_KEYWORD (xbm, Q_background, check_valid_string);
 #endif /* HAVE_WINDOW_SYSTEM */
 
-#ifdef HAVE_XFACE
-  INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (xface, "xface");
-
-  IIFORMAT_HAS_METHOD (xface, validate);
-  IIFORMAT_HAS_METHOD (xface, normalize);
-  IIFORMAT_HAS_METHOD (xface, possible_dest_types);
-
-  IIFORMAT_VALID_KEYWORD (xface, Q_data, check_valid_string);
-  IIFORMAT_VALID_KEYWORD (xface, Q_file, check_valid_string);
-  IIFORMAT_VALID_KEYWORD (xface, Q_hotspot_x, check_valid_int);
-  IIFORMAT_VALID_KEYWORD (xface, Q_hotspot_y, check_valid_int);
-  IIFORMAT_VALID_KEYWORD (xface, Q_foreground, check_valid_string);
-  IIFORMAT_VALID_KEYWORD (xface, Q_background, check_valid_string);
-#endif
-
 #ifdef HAVE_XPM
   INITIALIZE_IMAGE_INSTANTIATOR_FORMAT (xpm, "xpm");
 
   IIFORMAT_HAS_METHOD (xpm, validate);
   IIFORMAT_HAS_METHOD (xpm, normalize);
   IIFORMAT_HAS_METHOD (xpm, possible_dest_types);
+  IIFORMAT_HAS_METHOD (xpm, instantiate);
 
   IIFORMAT_VALID_KEYWORD (xpm, Q_data, check_valid_string);
   IIFORMAT_VALID_KEYWORD (xpm, Q_file, check_valid_string);
@@ -4633,28 +3421,15 @@ image_instantiator_format_create (void)
 }
 
 void
-reinit_vars_of_glyphs (void)
-{
-  the_expose_ignore_blocktype =
-    Blocktype_new (struct expose_ignore_blocktype);
-
-  hold_ignored_expose_registration = 0;
-}
-
-
-void
 vars_of_glyphs (void)
 {
-  reinit_vars_of_glyphs ();
-
   Vthe_nothing_vector = vector1 (Qnothing);
   staticpro (&Vthe_nothing_vector);
 
   /* image instances */
 
-  Vimage_instance_type_list = Fcons (Qnothing, 
-                                    list6 (Qtext, Qmono_pixmap, Qcolor_pixmap, 
-                                           Qpointer, Qsubwindow, Qwidget));
+  Vimage_instance_type_list = list6 (Qnothing, Qtext, Qmono_pixmap,
+                                    Qcolor_pixmap, Qpointer, Qsubwindow);
   staticpro (&Vimage_instance_type_list);
 
   /* glyphs */
@@ -4690,9 +3465,7 @@ Normally this is three dots ("...").
 What to display at the beginning of horizontally scrolled lines.
 */);
   Vhscroll_glyph = allocate_glyph (GLYPH_BUFFER, redisplay_glyph_changed);
-#ifdef HAVE_WINDOW_SYSTEM
-  Fprovide (Qxbm);
-#endif
+
 #ifdef HAVE_XPM
   Fprovide (Qxpm);
 
@@ -4709,15 +3482,6 @@ The default value of this variable defines the logical color names
 */ );
   Vxpm_color_symbols = Qnil; /* initialized in x-faces.el */
 #endif /* HAVE_XPM */
-#ifdef HAVE_XFACE
-  Fprovide (Qxface);
-#endif
-
-  DEFVAR_BOOL ("disable-animated-pixmaps", &disable_animated_pixmaps /*
-Whether animated pixmaps should be animated.
-Default is t.
-*/);
-  disable_animated_pixmaps = 0;
 }
 
 void