Merge r21-4-11-chise-0_20-=ucs.
[chise/xemacs-chise.git.1] / src / gutter.c
index 64d0df3..55dc0f8 100644 (file)
@@ -1,5 +1,5 @@
 /* Gutter implementation.
-   Copyright (C) 1999 Andy Piper.
+   Copyright (C) 1999, 2000 Andy Piper.
 
 This file is part of XEmacs.
 
@@ -20,7 +20,8 @@ Boston, MA 02111-1307, USA.  */
 
 /* Synched up with: Not in FSF. */
 
-/* Specifers ripped-off from toolbar.c */
+/* written by Andy Piper <andy@xemacs.org> with specifiers partially
+   ripped-off from toolbar.c */
 
 #include <config.h>
 #include "lisp.h"
@@ -45,25 +46,32 @@ Lisp_Object Vdefault_gutter_border_width;
 
 Lisp_Object Vdefault_gutter_position;
 
-#define SET_GUTTER_WAS_VISIBLE_FLAG(frame, pos, flag)                  \
-  do {                                                                 \
-    switch (pos)                                                       \
-      {                                                                        \
-      case TOP_GUTTER:                                                 \
-       (frame)->top_gutter_was_visible = flag;                 \
-       break;                                                          \
-      case BOTTOM_GUTTER:                                              \
-       (frame)->bottom_gutter_was_visible = flag;                      \
-       break;                                                          \
-      case LEFT_GUTTER:                                                \
-       (frame)->left_gutter_was_visible = flag;                        \
-       break;                                                          \
-      case RIGHT_GUTTER:                                               \
-       (frame)->right_gutter_was_visible = flag;                       \
-       break;                                                          \
-      default:                                                         \
-       abort ();                                                       \
-      }                                                                        \
+Lisp_Object Qgutter_size;
+Lisp_Object Qgutter_visible;
+Lisp_Object Qdefault_gutter_position_changed_hook;
+
+static void
+update_gutter_geometry (struct frame *f, enum gutter_pos pos);
+
+#define SET_GUTTER_WAS_VISIBLE_FLAG(frame, pos, flag)  \
+  do {                                                 \
+    switch (pos)                                       \
+      {                                                        \
+      case TOP_GUTTER:                                 \
+       (frame)->top_gutter_was_visible = flag;         \
+       break;                                          \
+      case BOTTOM_GUTTER:                              \
+       (frame)->bottom_gutter_was_visible = flag;      \
+       break;                                          \
+      case LEFT_GUTTER:                                        \
+       (frame)->left_gutter_was_visible = flag;        \
+       break;                                          \
+      case RIGHT_GUTTER:                               \
+       (frame)->right_gutter_was_visible = flag;       \
+       break;                                          \
+      default:                                         \
+       abort ();                                       \
+      }                                                        \
   } while (0)
 
 static int gutter_was_visible (struct frame* frame, enum gutter_pos pos)
@@ -71,18 +79,20 @@ static int gutter_was_visible (struct frame* frame, enum gutter_pos pos)
   switch (pos)
     {
     case TOP_GUTTER:
-      return (frame)->top_gutter_was_visible;
+      return frame->top_gutter_was_visible;
     case BOTTOM_GUTTER:
-      return (frame)->bottom_gutter_was_visible;
+      return frame->bottom_gutter_was_visible;
     case LEFT_GUTTER:
-      return (frame)->left_gutter_was_visible;
+      return frame->left_gutter_was_visible;
     case RIGHT_GUTTER:
-      return (frame)->right_gutter_was_visible;
+      return frame->right_gutter_was_visible;
     default:
       abort ();
+       return 0;       /* To keep the compiler happy */
     }
 }
 
+#if 0
 static Lisp_Object
 frame_topmost_window (struct frame *f)
 {
@@ -97,6 +107,7 @@ frame_topmost_window (struct frame *f)
 
   return w;
 }
+#endif
 
 static Lisp_Object
 frame_bottommost_window (struct frame *f)
@@ -159,12 +170,11 @@ frame_rightmost_window (struct frame *f)
    use this for calculating the gutter positions we run into trouble
    if it is not the window nearest the gutter. Instead we predetermine
    the nearest window and then use that.*/
-void
+static void
 get_gutter_coords (struct frame *f, enum gutter_pos pos, int *x, int *y,
                   int *width, int *height)
 {
   struct window
-    * top = XWINDOW (frame_topmost_window (f)),
     * bot = XWINDOW (frame_bottommost_window (f));
   /* The top and bottom gutters take precedence over the left and
      right. */
@@ -173,37 +183,34 @@ get_gutter_coords (struct frame *f, enum gutter_pos pos, int *x, int *y,
     case TOP_GUTTER:
       *x = FRAME_LEFT_BORDER_END (f);
       *y = FRAME_TOP_BORDER_END (f);
-      *width = FRAME_RIGHT_BORDER_START (f) 
+      *width = FRAME_RIGHT_BORDER_START (f)
        - FRAME_LEFT_BORDER_END (f);
       *height = FRAME_TOP_GUTTER_BOUNDS (f);
       break;
 
     case BOTTOM_GUTTER:
       *x = FRAME_LEFT_BORDER_END (f);
-      *y = WINDOW_BOTTOM (bot)
-       - FRAME_BOTTOM_GUTTER_BOUNDS (f);
-      *width = FRAME_RIGHT_BORDER_START (f) 
+      *y = WINDOW_BOTTOM (bot);
+      *width = FRAME_RIGHT_BORDER_START (f)
        - FRAME_LEFT_BORDER_END (f);
       *height = FRAME_BOTTOM_GUTTER_BOUNDS (f);
       break;
 
     case LEFT_GUTTER:
       *x = FRAME_LEFT_BORDER_END (f);
-      *y = WINDOW_TEXT_TOP (top);
+      *y = FRAME_TOP_BORDER_END (f) + FRAME_TOP_GUTTER_BOUNDS (f);
       *width = FRAME_LEFT_GUTTER_BOUNDS (f);
       *height = WINDOW_BOTTOM (bot)
-       - (WINDOW_TEXT_TOP (top)
-          + FRAME_BOTTOM_GUTTER_BOUNDS (f));
+       - (FRAME_TOP_BORDER_END (f) + FRAME_TOP_GUTTER_BOUNDS (f));
       break;
-      
+
     case RIGHT_GUTTER:
       *x = FRAME_RIGHT_BORDER_START (f)
        - FRAME_RIGHT_GUTTER_BOUNDS (f);
-      *y = WINDOW_TEXT_TOP (top);
+      *y = FRAME_TOP_BORDER_END (f) + FRAME_TOP_GUTTER_BOUNDS (f);
       *width = FRAME_RIGHT_GUTTER_BOUNDS (f);
       *height = WINDOW_BOTTOM (bot)
-       - (WINDOW_TEXT_TOP (top) 
-          + FRAME_BOTTOM_GUTTER_BOUNDS (f));
+       - (FRAME_TOP_BORDER_END (f) + FRAME_TOP_GUTTER_BOUNDS (f));
       break;
 
     default:
@@ -211,99 +218,293 @@ get_gutter_coords (struct frame *f, enum gutter_pos pos, int *x, int *y,
     }
 }
 
-static void
-output_gutter (struct frame *f, enum gutter_pos pos)
+/*
+ display_boxes_in_gutter_p
+
+ Determine whether the required display_glyph_area is completely
+ inside the gutter. -1 means the display_box is not in the gutter. 1
+ means the display_box and the display_glyph_area are in the
+ window. 0 means the display_box is in the gutter but the
+ display_glyph_area is not. */
+int display_boxes_in_gutter_p (struct frame *f, struct display_box* db,
+                              struct display_glyph_area* dga)
 {
-  Lisp_Object frame;
-  Lisp_Object window = FRAME_LAST_NONMINIBUF_WINDOW (f);
-  struct device *d = XDEVICE (f->device);
-  struct window* w = XWINDOW (window);
-  int x, y, width, height, ypos;
-  int line;
-  int border_width = FRAME_GUTTER_BORDER_WIDTH (f, pos);
-  face_index findex = get_builtin_face_cache_index (w, Vgui_element_face);
-  display_line_dynarr* ddla, *cdla;
-  struct display_line *dl;
+  enum gutter_pos pos;
+  GUTTER_POS_LOOP (pos)
+    {
+      if (FRAME_GUTTER_VISIBLE (f, pos))
+       {
+         int x, y, width, height;
+         get_gutter_coords (f, pos, &x, &y, &width, &height);
+         if (db->xpos + dga->xoffset >= x
+             &&
+             db->ypos + dga->yoffset >= y
+             &&
+             db->xpos + dga->xoffset + dga->width <= x + width
+             &&
+             db->ypos + dga->yoffset + dga->height <= y + height)
+           return 1;
+         else if (db->xpos >= x && db->ypos >= y
+                  && db->xpos + db->width <= x + width
+                  && db->ypos + db->height <= y + height)
+           return 0;
+       }
+    }
+  return -1;
+}
 
-  if (!f->current_display_lines)
-    f->current_display_lines = Dynarr_new (display_line);
-  if (!f->desired_display_lines)
-    f->desired_display_lines = Dynarr_new (display_line);
-  
-  ddla = f->desired_display_lines;
-  cdla = f->current_display_lines;
+/* Convert the gutter specifier into something we can actually
+   display. */
+static Lisp_Object construct_window_gutter_spec (struct window* w,
+                                                enum gutter_pos pos)
+{
+  Lisp_Object rest, *args;
+  int nargs = 0;
+  Lisp_Object gutter = RAW_WINDOW_GUTTER (w, pos);
 
-  XSETFRAME (frame, f);
+  if (STRINGP (gutter) || NILP (gutter))
+    return gutter;
 
-  get_gutter_coords (f, pos, &x, &y, &width, &height);
-  /* clear out what we want to cover */
-  /* generate some display lines */
-  generate_displayable_area (w, WINDOW_GUTTER (w, pos),
-                            x + border_width, y + border_width,
-                            width - 2 * border_width, 
-                            height - 2 * border_width, ddla, 0, findex);
-  /* Output each line. */
-  for (line = 0; line < Dynarr_length (ddla); line++)
+  GET_LIST_LENGTH (gutter, nargs);
+  args = alloca_array (Lisp_Object, nargs >> 1);
+  nargs = 0;
+
+  for (rest = gutter; !NILP (rest); rest = XCDR (XCDR (rest)))
     {
-      output_display_line (w, cdla, ddla, line, -1, -1);
+      /* We only put things in the real gutter that are declared to be
+         visible. */
+      if (!CONSP (WINDOW_GUTTER_VISIBLE (w, pos))
+         ||
+         !NILP (Fmemq (XCAR (rest), WINDOW_GUTTER_VISIBLE (w, pos))))
+       {
+         args [nargs++] = XCAR (XCDR (rest));
+       }
     }
 
-  /* grab coordinates of last line and blank after it. */
-  dl = Dynarr_atp (ddla, Dynarr_length (ddla) - 1);
-  ypos = dl->ypos + dl->descent - dl->clip;
-  redisplay_clear_region (window, findex, x + border_width , ypos,
-                         width - 2 * border_width, height - (ypos - y));
+  return Fconcat (nargs, args);
+}
 
-  /* bevel the gutter area if so desired */
-  if (border_width != 0)
+/* Sizing gutters is a pain so we try and help the user by determining
+   what height will accommodate all lines. This is useless on left and
+   right gutters as we always have a maximal number of lines. */
+static int
+calculate_gutter_size_from_display_lines (enum gutter_pos pos,
+                                         display_line_dynarr* ddla)
+{
+  int size = 0;
+  struct display_line *dl;
+
+  /* For top and bottom the calculation is easy. */
+  if (pos == TOP_GUTTER || pos == BOTTOM_GUTTER)
+    {
+      /* grab coordinates of last line  */
+      if (Dynarr_length (ddla))
+       {
+         dl = Dynarr_atp (ddla, Dynarr_length (ddla) - 1);
+         size = (dl->ypos + dl->descent - dl->clip) 
+           - (Dynarr_atp (ddla, 0)->ypos - Dynarr_atp (ddla, 0)->ascent);
+       }
+    }
+  /* For left and right we have to do some maths. */
+  else
     {
-      MAYBE_DEVMETH (d, bevel_area, 
-                    (w, findex, x, y, width, height, border_width));
+      int start_pos = 0, end_pos = 0, line;
+      for (line = 0; line < Dynarr_length (ddla); line++)
+       {
+         int block;
+         dl = Dynarr_atp (ddla, line);
+
+         for (block = 0; block < Dynarr_largest (dl->display_blocks); block++)
+           {
+             struct display_block *db = Dynarr_atp (dl->display_blocks, block);
+
+             if (db->type == TEXT)
+               {
+                 start_pos = min (db->start_pos, start_pos);
+                 end_pos = max (db->end_pos, end_pos);
+               }
+           }
+       }
+      size = end_pos - start_pos;
     }
+
+  return size;
 }
 
-/* sizing gutters is a pain so we try and help the user by detemining
-   what height will accomodate all lines. This is useless on left and
-   right gutters as we always have a maximal number of lines. */
 static Lisp_Object
 calculate_gutter_size (struct window *w, enum gutter_pos pos)
 {
   struct frame* f = XFRAME (WINDOW_FRAME (w));
-  int ypos;
+  int count;
   display_line_dynarr* ddla;
-  struct display_line *dl;
+  Lisp_Object ret = Qnil;
 
-  /* we cannot autodetect gutter sizes for the left and right as there
-     is no reasonable metric to use */
-  assert (pos == TOP_GUTTER || pos == BOTTOM_GUTTER);
   /* degenerate case */
-  if (NILP (WINDOW_GUTTER (w, pos))
+  if (NILP (RAW_WINDOW_GUTTER (w, pos))
       ||
-      !FRAME_VISIBLE_P (f))
+      !FRAME_VISIBLE_P (f)
+      ||
+      NILP (w->buffer))
     return Qnil;
 
+  /* Redisplay code that we use relies on GC not happening. Make it
+     so. */
+  count = specpdl_depth ();
+  record_unwind_protect (restore_gc_inhibit,
+                        make_int (gc_currently_forbidden));
+  gc_currently_forbidden = 1;
+
   ddla = Dynarr_new (display_line);
   /* generate some display lines */
   generate_displayable_area (w, WINDOW_GUTTER (w, pos),
                             FRAME_LEFT_BORDER_END (f),
-                            0,
+                            FRAME_TOP_BORDER_END (f),
                             FRAME_RIGHT_BORDER_START (f)
                             - FRAME_LEFT_BORDER_END (f),
-                            200,
+                            FRAME_BOTTOM_BORDER_START (f)
+                            - FRAME_TOP_BORDER_END (f),
                             ddla, 0, 0);
-  /* grab coordinates of last line  */
-  if (Dynarr_length (ddla))
+
+  /* Let GC happen again. */
+  unbind_to (count, Qnil);
+
+  ret = make_int (calculate_gutter_size_from_display_lines (pos, ddla));
+  free_display_lines (ddla);
+
+  return ret;
+}
+
+static void
+output_gutter (struct frame *f, enum gutter_pos pos, int force)
+{
+  Lisp_Object frame;
+  Lisp_Object window = FRAME_LAST_NONMINIBUF_WINDOW (f);
+  struct device *d = XDEVICE (f->device);
+  struct window* w = XWINDOW (window);
+  int x, y, width, height, ypos;
+  int line, border_width;
+  face_index findex;
+  display_line_dynarr* ddla, *cdla;
+  struct display_line *dl = 0;
+  int cdla_len;
+
+  if (!WINDOW_LIVE_P (w))
+    return;
+
+  border_width = FRAME_GUTTER_BORDER_WIDTH (f, pos);
+  findex = get_builtin_face_cache_index (w, Vwidget_face);
+
+  if (!f->current_display_lines[pos])
+    f->current_display_lines[pos] = Dynarr_new (display_line);
+  if (!f->desired_display_lines[pos])
+    f->desired_display_lines[pos] = Dynarr_new (display_line);
+
+  ddla = f->desired_display_lines[pos];
+  cdla = f->current_display_lines[pos];
+  cdla_len = Dynarr_length (cdla);
+
+  XSETFRAME (frame, f);
+
+  get_gutter_coords (f, pos, &x, &y, &width, &height);
+  /* generate some display lines */
+  generate_displayable_area (w, WINDOW_GUTTER (w, pos),
+                            x + border_width, y + border_width,
+                            width - 2 * border_width,
+                            height - 2 * border_width, ddla, 0, findex);
+
+  /* We only output the gutter if we think something of significance
+     has changed.  This is, for example, because redisplay can cause
+     new face cache elements to get added causing compare_runes to
+     fail because the findex for a particular face has changed.  */
+  if (force || f->faces_changed || f->frame_changed ||
+      f->gutter_changed || f->glyphs_changed ||
+      f->size_changed || f->subwindows_changed ||
+      w->windows_changed || f->windows_structure_changed ||
+      cdla_len != Dynarr_length (ddla) ||
+      (f->extents_changed && w->gutter_extent_modiff[pos]))
     {
-      dl = Dynarr_atp (ddla, Dynarr_length (ddla) - 1);
-      ypos = dl->ypos + dl->descent - dl->clip;
-      Dynarr_free (ddla);
-      return make_int (ypos);
+#ifdef DEBUG_GUTTERS
+      printf ("gutter redisplay [%dx%d@%d+%d] triggered by %s,\n", 
+             width, height, x, y, force ? "force" :
+             f->faces_changed ? "f->faces_changed" :
+             f->frame_changed ? "f->frame_changed" :
+             f->gutter_changed ? "f->gutter_changed" :
+             f->glyphs_changed ? "f->glyphs_changed" :
+             f->size_changed ? "f->size_changed" :
+             f->subwindows_changed ? "f->subwindows_changed" :
+             w->windows_changed ? "w->windows_changed" :
+             f->windows_structure_changed ? "f->windows_structure_changed" :
+             cdla_len != Dynarr_length (ddla) ? "different display structures" :
+             f->extents_changed && w->gutter_extent_modiff[pos] ?
+             "f->extents_changed && w->gutter_extent_modiff[pos]" : "<null>");
+#endif
+      /* Output each line. */
+      for (line = 0; line < Dynarr_length (ddla); line++)
+       {
+         output_display_line (w, cdla, ddla, line, -1, -1);
+       }
+
+      /* If the number of display lines has shrunk, adjust. */
+      if (cdla_len > Dynarr_length (ddla))
+       {
+         Dynarr_length (cdla) = Dynarr_length (ddla);
+       }
+
+      /* grab coordinates of last line and blank after it. */
+      if (Dynarr_length (ddla) > 0)
+       {
+         dl = Dynarr_atp (ddla, Dynarr_length (ddla) - 1);
+         ypos = dl->ypos + dl->descent - dl->clip;
+       }
+      else
+       ypos = y;
+
+      redisplay_clear_region (window, findex, x + border_width , ypos,
+                             width - 2 * border_width, height - (ypos - y) - border_width);
+      /* If, for some reason, we have more to display than we have
+         room for, and we are allowed to resize the gutter, then make
+         sure this happens before the next time we try and
+         output. This can happen when face font sizes change. */
+      if (dl && EQ (w->gutter_size[pos], Qautodetect) 
+         && (dl->clip > 0 ||
+             calculate_gutter_size_from_display_lines (pos, ddla) > 
+             WINDOW_GUTTER_SIZE_INTERNAL (w, pos)))
+       {
+         /* #### Ideally we would just mark the specifier as dirty
+         and everything else would "just work". Unfortunately we have
+         two problems with this. One is that the specifier cache
+         won't be recalculated unless the specifier code thinks the
+         cached value has actually changed, even though we have
+         marked the specifier as dirty. Additionally, although doing
+         this results in a gutter size change, we never seem to get
+         back into redisplay so that the frame size can be updated. I
+         think this is because we are already in redisplay and later
+         on the frame will be marked as clean. Thus we also have to
+         force a pending recalculation of the frame size.  */
+         w->gutter_size[pos] = Qnil;
+         Fset_specifier_dirty_flag (Vgutter_size[pos]);
+         update_gutter_geometry (f, pos);
+       }
+
+      /* bevel the gutter area if so desired */
+      if (border_width != 0)
+       {
+         MAYBE_DEVMETH (d, bevel_area,
+                        (w, findex, x, y, width, height, border_width,
+                         EDGE_ALL, EDGE_BEVEL_OUT));
+       }
     }
   else
     {
-      Dynarr_free (ddla);
-      return Qnil;
+      /* Nothing of significance happened so sync the display line
+         structs. */
+      for (line = 0; line < Dynarr_length (ddla); line++)
+       {
+         sync_display_line_structs (w, line, 1, cdla, ddla);
+       }
     }
+
+  w->gutter_extent_modiff [pos] = 0;
 }
 
 static void
@@ -312,7 +513,7 @@ clear_gutter (struct frame *f, enum gutter_pos pos)
   int x, y, width, height;
   Lisp_Object window = FRAME_LAST_NONMINIBUF_WINDOW (f);
   face_index findex = get_builtin_face_cache_index (XWINDOW (window),
-                                                   Vgui_element_face);
+                                                   Vwidget_face);
   get_gutter_coords (f, pos, &x, &y, &width, &height);
 
   SET_GUTTER_WAS_VISIBLE_FLAG (f, pos, 0);
@@ -320,24 +521,140 @@ clear_gutter (struct frame *f, enum gutter_pos pos)
   redisplay_clear_region (window, findex, x, y, width, height);
 }
 
+/* #### I don't currently believe that redisplay needs to mark the
+   glyphs in its structures since these will always be referenced from
+   somewhere else. However, I'm not sure enough to stake my life on it
+   at this point, so we do the safe thing. */
+
+/* See the comment in image_instantiate_cache_result as to why marking
+   the glyph will also mark the image_instance. */
+void
+mark_gutters (struct frame* f)
+{
+  enum gutter_pos pos;
+  GUTTER_POS_LOOP (pos)
+    {
+      if (f->current_display_lines[pos])
+       mark_redisplay_structs (f->current_display_lines[pos]);
+      /* #### Do we really need to mark the desired lines? */
+      if (f->desired_display_lines[pos])
+       mark_redisplay_structs (f->desired_display_lines[pos]);
+    }
+}
+
+/* This is called by extent_changed_for_redisplay, so that redisplay
+   knows exactly what extents have changed. */
+void
+gutter_extent_signal_changed_region_maybe (Lisp_Object obj,
+                                          Bufpos start, Bufpos end)
+{
+  /* #### Start and end are currently ignored but could be used by a
+     more optimal gutter redisplay. We currently loop over all frames
+     here, this could be optimized. */
+  Lisp_Object frmcons, devcons, concons;
+
+  FRAME_LOOP_NO_BREAK (frmcons, devcons, concons)
+    {
+      struct frame *f = XFRAME (XCAR (frmcons));
+      enum gutter_pos pos;
+      Lisp_Object window = FRAME_LAST_NONMINIBUF_WINDOW (f);
+      struct window* w = XWINDOW (window);
+
+      GUTTER_POS_LOOP (pos)
+       {
+         if (EQ (WINDOW_GUTTER (w, pos), obj))
+           {
+             w->gutter_extent_modiff[pos]++;
+           }
+       }
+    }
+}
+
+/* We have to change the gutter geometry separately to the gutter
+   update since it needs to occur outside of redisplay proper. */
+static void
+update_gutter_geometry (struct frame *f, enum gutter_pos pos)
+{
+  /* If the gutter geometry has changed then re-layout the
+     frame. If we are in display there is almost no point in doing
+     anything else since the frame size changes will be delayed
+     until we are out of redisplay proper. */
+  if (FRAME_GUTTER_BOUNDS (f, pos) != f->current_gutter_bounds[pos])
+    {
+      int width, height;
+      pixel_to_char_size (f, FRAME_PIXWIDTH (f), FRAME_PIXHEIGHT (f),
+                         &width, &height);
+      change_frame_size (f, height, width, 0);
+      MARK_FRAME_LAYOUT_CHANGED (f);
+    }
+
+  /* Mark sizes as up-to-date. */
+  f->current_gutter_bounds[pos] = FRAME_GUTTER_BOUNDS (f, pos);
+}
+
+void
+update_frame_gutter_geometry (struct frame *f)
+{
+  if (f->gutter_changed 
+      || f->frame_layout_changed 
+      || f->windows_structure_changed)
+    {
+      enum gutter_pos pos;
+
+      /* If the gutter geometry has changed then re-layout the
+         frame. If we are in display there is almost no point in doing
+         anything else since the frame size changes will be delayed
+         until we are out of redisplay proper. */
+      GUTTER_POS_LOOP (pos)
+       {
+         update_gutter_geometry (f, pos);
+       }
+    }
+}
+
 void
 update_frame_gutters (struct frame *f)
 {
-  if (f->gutter_changed || f->frame_changed || f->clear)
+  if (f->faces_changed || f->frame_changed ||
+      f->gutter_changed || f->glyphs_changed ||
+      f->size_changed || f->subwindows_changed ||
+      f->windows_changed || f->windows_structure_changed ||
+      f->extents_changed || f->frame_layout_changed)
     {
-      int pos;
-      /* and output */
+      enum gutter_pos pos;
 
-      for (pos = 0; pos < 4; pos++)
+      /* We don't actually care about these when outputting the gutter
+         so locally disable them. */
+      int local_clip_changed = f->clip_changed;
+      int local_buffers_changed = f->buffers_changed;
+      f->clip_changed = 0;
+      f->buffers_changed = 0;
+
+      /* and output */
+      GUTTER_POS_LOOP (pos)
        {
          if (FRAME_GUTTER_VISIBLE (f, pos))
-           output_gutter (f, pos);
+             output_gutter (f, pos, 0);
+
          else if (gutter_was_visible (f, pos))
-           clear_gutter (f, pos);
+             clear_gutter (f, pos);
        }
 
+      f->clip_changed = local_clip_changed;
+      f->buffers_changed = local_buffers_changed;
+      f->gutter_changed = 0;
+    }
+}
+
+void
+reset_gutter_display_lines (struct frame* f)
+{
+  enum gutter_pos pos;
+  GUTTER_POS_LOOP (pos)
+    {
+      if (f->current_display_lines[pos])
+       Dynarr_reset (f->current_display_lines[pos]);
     }
-  f->gutter_changed = 0;
 }
 
 static void
@@ -345,54 +662,63 @@ redraw_exposed_gutter (struct frame *f, enum gutter_pos pos, int x, int y,
                       int width, int height)
 {
   int g_x, g_y, g_width, g_height;
-  int newx, newy;
 
   get_gutter_coords (f, pos, &g_x, &g_y, &g_width, &g_height);
 
-  if (((y + height) < g_y) || (y > (g_y + g_height)))
+  if (((y + height) < g_y) || (y > (g_y + g_height)) || !height || !width || !g_height || !g_width)
     return;
   if (((x + width) < g_x) || (x > (g_x + g_width)))
     return;
 
+#ifdef DEBUG_WIDGETS
+  printf ("redrawing gutter after expose %d+%d, %dx%d\n",
+         x, y, width, height);
+#endif
   /* #### optimize this - redrawing the whole gutter for every expose
      is very expensive. We reset the current display lines because if
      they're being exposed they are no longer current. */
-  if (f->current_display_lines)
-    Dynarr_reset (f->current_display_lines);
-  /* we have to do this in-case there were subwindows where we are
-     redrawing, unfortunately sometimes this also generates expose
-     events resulting in an endless cycle of redsplay. */
-  newx = max (x, g_x);
-  newy = max (y, g_y);
-  width = min (x + width - newx, g_x + g_width - newx);
-  height = min (y + height - newy, g_y + g_height - newy);
-  redisplay_unmap_subwindows_maybe (f, newx, newy, width, height);
+  reset_gutter_display_lines (f);
 
   /* Even if none of the gutter is in the area, the blank region at
      the very least must be because the first thing we did is verify
      that some portion of the gutter is in the exposed region. */
-  output_gutter (f, pos);
+  output_gutter (f, pos, 1);
 }
 
 void
 redraw_exposed_gutters (struct frame *f, int x, int y, int width,
                        int height)
 {
-      int pos;
-      for (pos = 0; pos < 4; pos++)
-       {
-         if (FRAME_GUTTER_VISIBLE (f, pos))
-           redraw_exposed_gutter (f, pos, x, y, width, height);
-       }
+  enum gutter_pos pos;
+
+  /* We have to be "in display" when we output the gutter - make it
+     so. */
+  hold_frame_size_changes ();
+  GUTTER_POS_LOOP (pos)
+    {
+      if (FRAME_GUTTER_VISIBLE (f, pos))
+       redraw_exposed_gutter (f, pos, x, y, width, height);
+    }
+  unhold_one_frame_size_changes (f);
 }
 
 void
 free_frame_gutters (struct frame *f)
 {
-  if (f->current_display_lines)
-    Dynarr_free (f->current_display_lines);
-  if (f->desired_display_lines)
-    Dynarr_free (f->desired_display_lines);
+  enum gutter_pos pos;
+  GUTTER_POS_LOOP (pos)
+    {
+      if (f->current_display_lines[pos])
+       {
+         free_display_lines (f->current_display_lines[pos]);
+         f->current_display_lines[pos] = 0;
+       }
+      if (f->desired_display_lines[pos])
+       {
+         free_display_lines (f->desired_display_lines[pos]);
+         f->desired_display_lines[pos] = 0;
+       }
+    }
 }
 
 static enum gutter_pos
@@ -435,14 +761,15 @@ See `default-gutter-position'.
                              list1 (Fcons (Qnil, Qzero)));
       set_specifier_fallback (Vgutter_border_width[new],
                              Vdefault_gutter_border_width);
-      set_specifier_fallback (Vgutter_visible_p[cur],
-                             list1 (Fcons (Qnil, Qt)));
-      set_specifier_fallback (Vgutter_visible_p[new],
-                             Vdefault_gutter_visible_p);
+      set_specifier_fallback (Vgutter_visible_p[cur], list1 (Fcons (Qnil, Qt)));
+      set_specifier_fallback (Vgutter_visible_p[new], Vdefault_gutter_visible_p);
+
       Vdefault_gutter_position = position;
       unhold_frame_size_changes ();
     }
 
+  run_hook (Qdefault_gutter_position_changed_hook);
+
   return position;
 }
 
@@ -512,20 +839,28 @@ gutter_validate (Lisp_Object instantiator)
   if (NILP (instantiator))
     return;
 
+  /* Must be a string or a plist. */
+  if (!STRINGP (instantiator) && NILP (Fvalid_plist_p (instantiator)))
+      signal_simple_error ("Gutter spec must be string, plist or nil", instantiator);
+
   if (!STRINGP (instantiator))
-    signal_simple_error ("Gutter spec must be string or nil", instantiator);
+    {
+      Lisp_Object rest;
+
+      for (rest = instantiator; !NILP (rest); rest = XCDR (XCDR (rest)))
+       {
+         if (!SYMBOLP (XCAR (rest))
+             || !STRINGP (XCAR (XCDR (rest))))
+           signal_simple_error ("Gutter plist spec must contain strings", instantiator);
+       }
+    }
 }
 
 DEFUN ("gutter-specifier-p", Fgutter_specifier_p, 1, 1, 0, /*
 Return non-nil if OBJECT is a gutter specifier.
-Gutter specifiers are used to specify the format of a gutter.
-The values of the variables `default-gutter', `top-gutter',
-`left-gutter', `right-gutter', and `bottom-gutter' are always
-gutter specifiers.
-
-Valid gutter instantiators are called "gutter descriptors"
-and are lists of vectors.  See `default-gutter' for a description
-of the exact format.
+
+See `make-gutter-specifier' for a description of possible gutter
+instantiators.
 */
        (object))
 {
@@ -546,22 +881,50 @@ recompute_overlaying_specifier (Lisp_Object real_one[4])
 
 static void
 gutter_specs_changed (Lisp_Object specifier, struct window *w,
-                      Lisp_Object oldval)
+                      Lisp_Object oldval, enum gutter_pos pos)
 {
-  int pos;
-  for (pos = 0; pos< 4; pos++)
+  w->real_gutter[pos] = construct_window_gutter_spec (w, pos);
+  w->real_gutter_size[pos] = w->gutter_size[pos];
+
+  if (EQ (w->real_gutter_size[pos], Qautodetect)
+      && !NILP (w->gutter_visible_p[pos]))
     {
-      w->real_gutter_size[pos] = w->gutter_size[pos];
-      if (EQ (w->real_gutter_size[pos], Qautodetect)
-         && !NILP (w->gutter_visible_p[pos]))
-       {
-         w->real_gutter_size [pos] = calculate_gutter_size (w, pos);
-       }
+      w->real_gutter_size [pos] = calculate_gutter_size (w, pos);
     }
   MARK_GUTTER_CHANGED;
+  MARK_MODELINE_CHANGED;
   MARK_WINDOWS_CHANGED (w);
 }
 
+/* We define all of these so we can access which actual gutter changed. */
+static void
+top_gutter_specs_changed (Lisp_Object specifier, struct window *w,
+                         Lisp_Object oldval)
+{
+  gutter_specs_changed (specifier, w, oldval, TOP_GUTTER);
+}
+
+static void
+bottom_gutter_specs_changed (Lisp_Object specifier, struct window *w,
+                            Lisp_Object oldval)
+{
+  gutter_specs_changed (specifier, w, oldval, BOTTOM_GUTTER);
+}
+
+static void
+left_gutter_specs_changed (Lisp_Object specifier, struct window *w,
+                          Lisp_Object oldval)
+{
+  gutter_specs_changed (specifier, w, oldval, LEFT_GUTTER);
+}
+
+static void
+right_gutter_specs_changed (Lisp_Object specifier, struct window *w,
+                           Lisp_Object oldval)
+{
+  gutter_specs_changed (specifier, w, oldval, RIGHT_GUTTER);
+}
+
 static void
 default_gutter_specs_changed (Lisp_Object specifier, struct window *w,
                               Lisp_Object oldval)
@@ -573,8 +936,8 @@ static void
 gutter_geometry_changed_in_window (Lisp_Object specifier, struct window *w,
                                    Lisp_Object oldval)
 {
-  int pos;
-  for (pos = 0; pos< 4; pos++)
+  enum gutter_pos pos;
+  GUTTER_POS_LOOP (pos)
     {
       w->real_gutter_size[pos] = w->gutter_size[pos];
       if (EQ (w->real_gutter_size[pos], Qautodetect)
@@ -583,8 +946,9 @@ gutter_geometry_changed_in_window (Lisp_Object specifier, struct window *w,
          w->real_gutter_size [pos] = calculate_gutter_size (w, pos);
        }
     }
-  
+
   MARK_GUTTER_CHANGED;
+  MARK_MODELINE_CHANGED;
   MARK_WINDOWS_CHANGED (w);
 }
 
@@ -609,18 +973,143 @@ default_gutter_visible_p_changed_in_window (Lisp_Object specifier,
                                             Lisp_Object oldval)
 {
   recompute_overlaying_specifier (Vgutter_visible_p);
+  /* Need to reconstruct the gutter specifier as it is affected by the
+     visibility. */
+  recompute_overlaying_specifier (Vgutter);
+}
+
+
+DECLARE_SPECIFIER_TYPE (gutter_size);
+#define GUTTER_SIZE_SPECIFIERP(x) SPECIFIER_TYPEP (x, gutter_size)
+DEFINE_SPECIFIER_TYPE (gutter_size);
+
+static void
+gutter_size_validate (Lisp_Object instantiator)
+{
+  if (NILP (instantiator))
+    return;
+
+  if (!INTP (instantiator) && !EQ (instantiator, Qautodetect))
+    signal_simple_error ("Gutter size must be an integer or 'autodetect", instantiator);
+}
+
+DEFUN ("gutter-size-specifier-p", Fgutter_size_specifier_p, 1, 1, 0, /*
+Return non-nil if OBJECT is a gutter-size specifier.
+
+See `make-gutter-size-specifier' for a description of possible gutter-size
+instantiators.
+*/
+       (object))
+{
+  return GUTTER_SIZE_SPECIFIERP (object) ? Qt : Qnil;
+}
+
+DECLARE_SPECIFIER_TYPE (gutter_visible);
+#define GUTTER_VISIBLE_SPECIFIERP(x) SPECIFIER_TYPEP (x, gutter_visible)
+DEFINE_SPECIFIER_TYPE (gutter_visible);
+
+static void
+gutter_visible_validate (Lisp_Object instantiator)
+{
+  if (NILP (instantiator))
+    return;
+
+  if (!NILP (instantiator) && !EQ (instantiator, Qt) && !CONSP (instantiator))
+    signal_simple_error ("Gutter visibility must be a boolean or list of symbols",
+                        instantiator);
+
+  if (CONSP (instantiator))
+    {
+      Lisp_Object rest;
+
+      EXTERNAL_LIST_LOOP (rest, instantiator)
+       {
+         if (!SYMBOLP (XCAR (rest)))
+             signal_simple_error ("Gutter visibility must be a boolean or list of symbols",
+                                  instantiator);
+       }
+    }
+}
+
+DEFUN ("gutter-visible-specifier-p", Fgutter_visible_specifier_p, 1, 1, 0, /*
+Return non-nil if OBJECT is a gutter-visible specifier.
+
+See `make-gutter-visible-specifier' for a description of possible
+gutter-visible instantiators.
+*/
+       (object))
+{
+  return GUTTER_VISIBLE_SPECIFIERP (object) ? Qt : Qnil;
+}
+
+DEFUN ("redisplay-gutter-area", Fredisplay_gutter_area, 0, 0, 0, /*
+Ensure that all gutters are correctly showing their gutter specifier.
+*/
+       ())
+{
+  Lisp_Object devcons, concons;
+
+  DEVICE_LOOP_NO_BREAK (devcons, concons)
+    {
+      struct device *d = XDEVICE (XCAR (devcons));
+      Lisp_Object frmcons;
+
+      DEVICE_FRAME_LOOP (frmcons, d)
+       {
+         struct frame *f = XFRAME (XCAR (frmcons));
+
+         MAYBE_DEVMETH (d, frame_output_begin, (f));
+
+         /* Sequence is quite important here. We not only want to
+          redisplay the gutter area but we also want to flush any
+          frame size changes out so that the gutter redisplay happens
+          in a kosha environment.
+
+          This is not only so that things look right but so that
+          glyph redisplay optimization kicks in, by default display
+          lines will be completely re-output if
+          f->windows_structure_changed is 1, and this is true if
+          frame size changes haven't been flushed out. Once frame
+          size changes have been flushed out we then need to
+          redisplay the frame in order to flush out pending window
+          size changes. */
+         update_frame_gutter_geometry (f);
+
+         if (f->windows_structure_changed)
+           redisplay_frame (f, 1);
+         else if (FRAME_REPAINT_P (f))
+           {
+             /* We have to be "in display" when we output the gutter
+                 - make it so. */
+             hold_frame_size_changes ();
+             update_frame_gutters (f);
+             unhold_one_frame_size_changes (f);
+           }
+
+         MAYBE_DEVMETH (d, frame_output_end, (f));
+      }
+
+      d->gutter_changed = 0;
+    }
+
+  /* This is so that further changes to the gutters will trigger redisplay. */
+  gutter_changed_set = 0;
+  gutter_changed = 0;
+
+  return Qnil;
 }
 
 void
 init_frame_gutters (struct frame *f)
 {
-  int pos;
+  enum gutter_pos pos;
   struct window* w = XWINDOW (FRAME_LAST_NONMINIBUF_WINDOW (f));
   /* We are here as far in frame creation so cached specifiers are
      already recomputed, and possibly modified by resource
      initialization. We need to recalculate autodetected gutters. */
-  for (pos = 0; pos< 4; pos++)
+  GUTTER_POS_LOOP (pos)
     {
+      w->real_gutter[pos] = construct_window_gutter_spec (w, pos);
       w->real_gutter_size[pos] = w->gutter_size[pos];
       if (EQ (w->gutter_size[pos], Qautodetect)
          && !NILP (w->gutter_visible_p[pos]))
@@ -630,16 +1119,30 @@ init_frame_gutters (struct frame *f)
          MARK_WINDOWS_CHANGED (w);
        }
     }
+
+  /* Keep a record of the current sizes of things. */
+  GUTTER_POS_LOOP (pos)
+    {
+      f->current_gutter_bounds[pos] = FRAME_GUTTER_BOUNDS (f, pos);
+    }
 }
 
 void
 syms_of_gutter (void)
 {
   DEFSUBR (Fgutter_specifier_p);
+  DEFSUBR (Fgutter_size_specifier_p);
+  DEFSUBR (Fgutter_visible_specifier_p);
   DEFSUBR (Fset_default_gutter_position);
   DEFSUBR (Fdefault_gutter_position);
   DEFSUBR (Fgutter_pixel_height);
   DEFSUBR (Fgutter_pixel_width);
+  DEFSUBR (Fredisplay_gutter_area);
+
+  defsymbol (&Qgutter_size, "gutter-size");
+  defsymbol (&Qgutter_visible, "gutter-visible");
+  defsymbol (&Qdefault_gutter_position_changed_hook,
+            "default-gutter-position-changed-hook");
 }
 
 void
@@ -655,9 +1158,22 @@ void
 specifier_type_create_gutter (void)
 {
   INITIALIZE_SPECIFIER_TYPE (gutter, "gutter", "gutter-specifier-p");
-
   SPECIFIER_HAS_METHOD (gutter, validate);
   SPECIFIER_HAS_METHOD (gutter, after_change);
+
+  INITIALIZE_SPECIFIER_TYPE (gutter_size, "gutter-size", "gutter-size-specifier-p");
+  SPECIFIER_HAS_METHOD (gutter_size, validate);
+
+  INITIALIZE_SPECIFIER_TYPE (gutter_visible, "gutter-visible", "gutter-visible-specifier-p");
+  SPECIFIER_HAS_METHOD (gutter_visible, validate);
+}
+
+void
+reinit_specifier_type_create_gutter (void)
+{
+  REINITIALIZE_SPECIFIER_TYPE (gutter);
+  REINITIALIZE_SPECIFIER_TYPE (gutter_size);
+  REINITIALIZE_SPECIFIER_TYPE (gutter_visible);
 }
 
 void
@@ -698,21 +1214,20 @@ the visibility and thickness using `default-gutter-visible-p',
 using position-specific specifiers.  That way, you will get sane
 behavior if the user changes the default gutter position.
 
-The gutter value should be a string or nil. You can attach extents and
-glyphs to the string and hence display glyphs and text in other fonts
-in the gutter area.
-
-*/ );
+The gutter value should be a string, a property list of strings or
+nil. You can attach extents and glyphs to the string and hence display
+glyphs and text in other fonts in the gutter area. If the gutter value
+is a property list then the strings will be concatenated together
+before being displayed.  */ );
 
   Vdefault_gutter = Fmake_specifier (Qgutter);
   /* #### It would be even nicer if the specifier caching
      automatically knew about specifier fallbacks, so we didn't
      have to do it ourselves. */
   set_specifier_caching (Vdefault_gutter,
-                        slot_offset (struct window,
-                                     default_gutter),
+                        offsetof (struct window, default_gutter),
                         default_gutter_specs_changed,
-                        0, 0);
+                        0, 0, 1);
 
   DEFVAR_SPECIFIER ("top-gutter",
                    &Vgutter[TOP_GUTTER] /*
@@ -722,10 +1237,9 @@ See `default-gutter' for a description of a valid gutter instantiator.
 */ );
   Vgutter[TOP_GUTTER] = Fmake_specifier (Qgutter);
   set_specifier_caching (Vgutter[TOP_GUTTER],
-                        slot_offset (struct window,
-                                     gutter[TOP_GUTTER]),
-                        gutter_specs_changed,
-                        0, 0);
+                        offsetof (struct window, gutter[TOP_GUTTER]),
+                        top_gutter_specs_changed,
+                        0, 0, 1);
 
   DEFVAR_SPECIFIER ("bottom-gutter",
                    &Vgutter[BOTTOM_GUTTER] /*
@@ -740,10 +1254,9 @@ displayed even if you provide a value for `bottom-gutter'.
 */ );
   Vgutter[BOTTOM_GUTTER] = Fmake_specifier (Qgutter);
   set_specifier_caching (Vgutter[BOTTOM_GUTTER],
-                        slot_offset (struct window,
-                                     gutter[BOTTOM_GUTTER]),
-                        gutter_specs_changed,
-                        0, 0);
+                        offsetof (struct window, gutter[BOTTOM_GUTTER]),
+                        bottom_gutter_specs_changed,
+                        0, 0, 1);
 
   DEFVAR_SPECIFIER ("left-gutter",
                    &Vgutter[LEFT_GUTTER] /*
@@ -758,10 +1271,9 @@ displayed even if you provide a value for `left-gutter'.
 */ );
   Vgutter[LEFT_GUTTER] = Fmake_specifier (Qgutter);
   set_specifier_caching (Vgutter[LEFT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter[LEFT_GUTTER]),
-                        gutter_specs_changed,
-                        0, 0);
+                        offsetof (struct window, gutter[LEFT_GUTTER]),
+                        left_gutter_specs_changed,
+                        0, 0, 1);
 
   DEFVAR_SPECIFIER ("right-gutter",
                    &Vgutter[RIGHT_GUTTER] /*
@@ -776,10 +1288,9 @@ displayed even if you provide a value for `right-gutter'.
 */ );
   Vgutter[RIGHT_GUTTER] = Fmake_specifier (Qgutter);
   set_specifier_caching (Vgutter[RIGHT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter[RIGHT_GUTTER]),
-                        gutter_specs_changed,
-                        0, 0);
+                        offsetof (struct window, gutter[RIGHT_GUTTER]),
+                        right_gutter_specs_changed,
+                        0, 0, 1);
 
   /* initially, top inherits from default; this can be
      changed with `set-default-gutter-position'. */
@@ -815,12 +1326,11 @@ If you set the height to 'autodetect the size of the gutter will be
 calculated to be large enough to hold the contents of the gutter. This
 is the default.
 */ );
-  Vdefault_gutter_height = Fmake_specifier (Qgeneric);
+  Vdefault_gutter_height = Fmake_specifier (Qgutter_size);
   set_specifier_caching (Vdefault_gutter_height,
-                        slot_offset (struct window,
-                                     default_gutter_height),
+                        offsetof (struct window, default_gutter_height),
                         default_gutter_size_changed_in_window,
-                        0, 0);
+                        0, 0, 1);
 
   DEFVAR_SPECIFIER ("default-gutter-width", &Vdefault_gutter_width /*
 *Width of the default gutter, if it's oriented vertically.
@@ -828,12 +1338,11 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-height' for more information.
 */ );
-  Vdefault_gutter_width = Fmake_specifier (Qnatnum);
+  Vdefault_gutter_width = Fmake_specifier (Qgutter_size);
   set_specifier_caching (Vdefault_gutter_width,
-                        slot_offset (struct window,
-                                     default_gutter_width),
+                        offsetof (struct window, default_gutter_width),
                         default_gutter_size_changed_in_window,
-                        0, 0);
+                        0, 0, 1);
 
   DEFVAR_SPECIFIER ("top-gutter-height",
                    &Vgutter_size[TOP_GUTTER] /*
@@ -842,12 +1351,10 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-height' for more information.
 */ );
-  Vgutter_size[TOP_GUTTER] = Fmake_specifier (Qgeneric);
+  Vgutter_size[TOP_GUTTER] = Fmake_specifier (Qgutter_size);
   set_specifier_caching (Vgutter_size[TOP_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_size[TOP_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window, gutter_size[TOP_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 1);
 
   DEFVAR_SPECIFIER ("bottom-gutter-height",
                    &Vgutter_size[BOTTOM_GUTTER] /*
@@ -856,12 +1363,10 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-height' for more information.
 */ );
-  Vgutter_size[BOTTOM_GUTTER] = Fmake_specifier (Qgeneric);
+  Vgutter_size[BOTTOM_GUTTER] = Fmake_specifier (Qgutter_size);
   set_specifier_caching (Vgutter_size[BOTTOM_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_size[BOTTOM_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window, gutter_size[BOTTOM_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 1);
 
   DEFVAR_SPECIFIER ("left-gutter-width",
                    &Vgutter_size[LEFT_GUTTER] /*
@@ -870,12 +1375,10 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-height' for more information.
 */ );
-  Vgutter_size[LEFT_GUTTER] = Fmake_specifier (Qnatnum);
+  Vgutter_size[LEFT_GUTTER] = Fmake_specifier (Qgutter_size);
   set_specifier_caching (Vgutter_size[LEFT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_size[LEFT_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window, gutter_size[LEFT_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 1);
 
   DEFVAR_SPECIFIER ("right-gutter-width",
                    &Vgutter_size[RIGHT_GUTTER] /*
@@ -884,21 +1387,23 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-height' for more information.
 */ );
-  Vgutter_size[RIGHT_GUTTER] = Fmake_specifier (Qnatnum);
+  Vgutter_size[RIGHT_GUTTER] = Fmake_specifier (Qgutter_size);
   set_specifier_caching (Vgutter_size[RIGHT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_size[RIGHT_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window, gutter_size[RIGHT_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 1);
 
   fb = Qnil;
 #ifdef HAVE_TTY
   fb = Fcons (Fcons (list1 (Qtty), Qautodetect), fb);
 #endif
+#ifdef HAVE_GTK
+  fb = Fcons (Fcons (list1 (Qgtk), Qautodetect), fb);
+#endif
 #ifdef HAVE_X_WINDOWS
   fb = Fcons (Fcons (list1 (Qx), Qautodetect), fb);
 #endif
 #ifdef HAVE_MS_WINDOWS
+  fb = Fcons (Fcons (list1 (Qmsprinter), Qautodetect), fb);
   fb = Fcons (Fcons (list1 (Qmswindows), Qautodetect), fb);
 #endif
   if (!NILP (fb))
@@ -906,14 +1411,17 @@ See `default-gutter-height' for more information.
 
   fb = Qnil;
 #ifdef HAVE_TTY
-  fb = Fcons (Fcons (list1 (Qtty), Qzero), fb);
+  fb = Fcons (Fcons (list1 (Qtty), Qautodetect), fb);
 #endif
 #ifdef HAVE_X_WINDOWS
-  fb = Fcons (Fcons (list1 (Qx), make_int (DEFAULT_GUTTER_WIDTH)), fb);
+  fb = Fcons (Fcons (list1 (Qx), Qautodetect), fb);
+#endif
+#ifdef HAVE_GTK
+  fb = Fcons (Fcons (list1 (Qgtk), Qautodetect), fb);
 #endif
 #ifdef HAVE_MS_WINDOWS
-  fb = Fcons (Fcons (list1 (Qmswindows), 
-                    make_int (DEFAULT_GUTTER_WIDTH)), fb);
+  fb = Fcons (Fcons (list1 (Qmsprinter), Qautodetect), fb);
+  fb = Fcons (Fcons (list1 (Qmswindows), Qautodetect), fb);
 #endif
   if (!NILP (fb))
     set_specifier_fallback (Vdefault_gutter_width, fb);
@@ -940,10 +1448,9 @@ instead.
 */ );
   Vdefault_gutter_border_width = Fmake_specifier (Qnatnum);
   set_specifier_caching (Vdefault_gutter_border_width,
-                        slot_offset (struct window,
-                                     default_gutter_border_width),
+                        offsetof (struct window, default_gutter_border_width),
                         default_gutter_border_width_changed_in_window,
-                        0, 0);
+                        0, 0, 0);
 
   DEFVAR_SPECIFIER ("top-gutter-border-width",
                    &Vgutter_border_width[TOP_GUTTER] /*
@@ -954,10 +1461,9 @@ See `default-gutter-height' for more information.
 */ );
   Vgutter_border_width[TOP_GUTTER] = Fmake_specifier (Qnatnum);
   set_specifier_caching (Vgutter_border_width[TOP_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_border_width[TOP_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_border_width[TOP_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 0);
 
   DEFVAR_SPECIFIER ("bottom-gutter-border-width",
                    &Vgutter_border_width[BOTTOM_GUTTER] /*
@@ -968,10 +1474,9 @@ See `default-gutter-height' for more information.
 */ );
   Vgutter_border_width[BOTTOM_GUTTER] = Fmake_specifier (Qnatnum);
   set_specifier_caching (Vgutter_border_width[BOTTOM_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_border_width[BOTTOM_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_border_width[BOTTOM_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 0);
 
   DEFVAR_SPECIFIER ("left-gutter-border-width",
                    &Vgutter_border_width[LEFT_GUTTER] /*
@@ -982,10 +1487,9 @@ See `default-gutter-height' for more information.
 */ );
   Vgutter_border_width[LEFT_GUTTER] = Fmake_specifier (Qnatnum);
   set_specifier_caching (Vgutter_border_width[LEFT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_border_width[LEFT_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_border_width[LEFT_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 0);
 
   DEFVAR_SPECIFIER ("right-gutter-border-width",
                    &Vgutter_border_width[RIGHT_GUTTER] /*
@@ -996,10 +1500,9 @@ See `default-gutter-height' for more information.
 */ );
   Vgutter_border_width[RIGHT_GUTTER] = Fmake_specifier (Qnatnum);
   set_specifier_caching (Vgutter_border_width[RIGHT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_border_width[RIGHT_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_border_width[RIGHT_GUTTER]),
+                        gutter_geometry_changed_in_window, 0, 0, 0);
 
   fb = Qnil;
 #ifdef HAVE_TTY
@@ -1009,6 +1512,7 @@ See `default-gutter-height' for more information.
   fb = Fcons (Fcons (list1 (Qx), make_int (DEFAULT_GUTTER_BORDER_WIDTH)), fb);
 #endif
 #ifdef HAVE_MS_WINDOWS
+  fb = Fcons (Fcons (list1 (Qmsprinter), Qzero), fb);
   fb = Fcons (Fcons (list1 (Qmswindows), make_int (DEFAULT_GUTTER_BORDER_WIDTH)), fb);
 #endif
   if (!NILP (fb))
@@ -1035,12 +1539,12 @@ instead.
 `default-gutter-visible-p' and all of the position-specific gutter
 visibility specifiers have a fallback value of true.
 */ );
-  Vdefault_gutter_visible_p = Fmake_specifier (Qboolean);
+  Vdefault_gutter_visible_p = Fmake_specifier (Qgutter_visible);
   set_specifier_caching (Vdefault_gutter_visible_p,
-                        slot_offset (struct window,
-                                     default_gutter_visible_p),
+                        offsetof (struct window,
+                                  default_gutter_visible_p),
                         default_gutter_visible_p_changed_in_window,
-                        0, 0);
+                        0, 0, 0);
 
   DEFVAR_SPECIFIER ("top-gutter-visible-p",
                    &Vgutter_visible_p[TOP_GUTTER] /*
@@ -1049,12 +1553,11 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-visible-p' for more information.
 */ );
-  Vgutter_visible_p[TOP_GUTTER] = Fmake_specifier (Qboolean);
+  Vgutter_visible_p[TOP_GUTTER] = Fmake_specifier (Qgutter_visible);
   set_specifier_caching (Vgutter_visible_p[TOP_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_visible_p[TOP_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_visible_p[TOP_GUTTER]),
+                        top_gutter_specs_changed, 0, 0, 0);
 
   DEFVAR_SPECIFIER ("bottom-gutter-visible-p",
                    &Vgutter_visible_p[BOTTOM_GUTTER] /*
@@ -1063,12 +1566,11 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-visible-p' for more information.
 */ );
-  Vgutter_visible_p[BOTTOM_GUTTER] = Fmake_specifier (Qboolean);
+  Vgutter_visible_p[BOTTOM_GUTTER] = Fmake_specifier (Qgutter_visible);
   set_specifier_caching (Vgutter_visible_p[BOTTOM_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_visible_p[BOTTOM_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_visible_p[BOTTOM_GUTTER]),
+                        bottom_gutter_specs_changed, 0, 0, 0);
 
   DEFVAR_SPECIFIER ("left-gutter-visible-p",
                    &Vgutter_visible_p[LEFT_GUTTER] /*
@@ -1077,12 +1579,11 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-visible-p' for more information.
 */ );
-  Vgutter_visible_p[LEFT_GUTTER] = Fmake_specifier (Qboolean);
+  Vgutter_visible_p[LEFT_GUTTER] = Fmake_specifier (Qgutter_visible);
   set_specifier_caching (Vgutter_visible_p[LEFT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_visible_p[LEFT_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_visible_p[LEFT_GUTTER]),
+                        left_gutter_specs_changed, 0, 0, 0);
 
   DEFVAR_SPECIFIER ("right-gutter-visible-p",
                    &Vgutter_visible_p[RIGHT_GUTTER] /*
@@ -1091,12 +1592,11 @@ This is a specifier; use `set-specifier' to change it.
 
 See `default-gutter-visible-p' for more information.
 */ );
-  Vgutter_visible_p[RIGHT_GUTTER] = Fmake_specifier (Qboolean);
+  Vgutter_visible_p[RIGHT_GUTTER] = Fmake_specifier (Qgutter_visible);
   set_specifier_caching (Vgutter_visible_p[RIGHT_GUTTER],
-                        slot_offset (struct window,
-                                     gutter_visible_p[RIGHT_GUTTER]),
-                        gutter_geometry_changed_in_window,
-                        0, 0);
+                        offsetof (struct window,
+                                  gutter_visible_p[RIGHT_GUTTER]),
+                        right_gutter_specs_changed, 0, 0, 0);
 
   /* initially, top inherits from default; this can be
      changed with `set-default-gutter-position'. */
@@ -1107,5 +1607,4 @@ See `default-gutter-visible-p' for more information.
   set_specifier_fallback (Vgutter_visible_p[BOTTOM_GUTTER], fb);
   set_specifier_fallback (Vgutter_visible_p[LEFT_GUTTER],   fb);
   set_specifier_fallback (Vgutter_visible_p[RIGHT_GUTTER],  fb);
-
 }