emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Emacs-diffs] Changes to emacs/src/window.c


From: Kim F. Storm
Subject: [Emacs-diffs] Changes to emacs/src/window.c
Date: Sat, 24 May 2003 18:07:52 -0400

Index: emacs/src/window.c
diff -c emacs/src/window.c:1.439 emacs/src/window.c:1.440
*** emacs/src/window.c:1.439    Sat May 17 17:18:25 2003
--- emacs/src/window.c  Sat May 24 18:07:51 2003
***************
*** 242,254 ****
  
    p = allocate_window ();
    XSETFASTINT (p->sequence_number, ++sequence_number);
!   XSETFASTINT (p->left, 0);
!   XSETFASTINT (p->top, 0);
!   XSETFASTINT (p->height, 0);
!   XSETFASTINT (p->width, 0);
    XSETFASTINT (p->hscroll, 0);
    XSETFASTINT (p->min_hscroll, 0);
!   p->orig_top = p->orig_height = Qnil;
    p->start = Fmake_marker ();
    p->pointm = Fmake_marker ();
    XSETFASTINT (p->use_time, 0);
--- 242,254 ----
  
    p = allocate_window ();
    XSETFASTINT (p->sequence_number, ++sequence_number);
!   XSETFASTINT (p->left_col, 0);
!   XSETFASTINT (p->top_line, 0);
!   XSETFASTINT (p->total_lines, 0);
!   XSETFASTINT (p->total_cols, 0);
    XSETFASTINT (p->hscroll, 0);
    XSETFASTINT (p->min_hscroll, 0);
!   p->orig_top_line = p->orig_total_lines = Qnil;
    p->start = Fmake_marker ();
    p->pointm = Fmake_marker ();
    XSETFASTINT (p->use_time, 0);
***************
*** 272,277 ****
--- 272,284 ----
    p->frozen_window_start_p = 0;
    p->height_fixed_p = 0;
    p->last_cursor_off_p = p->cursor_off_p = 0;
+   p->left_margin_cols = Qnil;
+   p->right_margin_cols = Qnil;
+   p->left_fringe_width = Qnil;
+   p->right_fringe_width = Qnil;
+   p->fringes_outside_margins = Qnil;
+   p->scroll_bar_width = Qnil;
+   p->vertical_scroll_bar_type = Qt;
  
    Vwindow_list = Qnil;
    return val;
***************
*** 398,404 ****
       (window)
       Lisp_Object window;
  {
!   return decode_window (window)->height;
  }
  
  DEFUN ("window-width", Fwindow_width, Swindow_width, 0, 1, 0,
--- 405,411 ----
       (window)
       Lisp_Object window;
  {
!   return decode_window (window)->total_lines;
  }
  
  DEFUN ("window-width", Fwindow_width, Swindow_width, 0, 1, 0,
***************
*** 409,415 ****
       (window)
       Lisp_Object window;
  {
!   return make_number (window_internal_width (decode_window (window)));
  }
  
  DEFUN ("window-hscroll", Fwindow_hscroll, Swindow_hscroll, 0, 1, 0,
--- 416,422 ----
       (window)
       Lisp_Object window;
  {
!   return make_number (window_box_text_cols (decode_window (window)));
  }
  
  DEFUN ("window-hscroll", Fwindow_hscroll, Swindow_hscroll, 0, 1, 0,
***************
*** 482,492 ****
  {
    register struct window *w = decode_window (window);
  
!   return Fcons (w->left, Fcons (w->top,
!            Fcons (make_number (WINDOW_RIGHT_EDGE (w)),
!                 Fcons (make_number (XFASTINT (w->top)
!                                     + XFASTINT (w->height)),
!                        Qnil))));
  }
  
  /* Test if the character at column *X, row *Y is within window W.
--- 489,499 ----
  {
    register struct window *w = decode_window (window);
  
!   return Fcons (make_number (WINDOW_LEFT_EDGE_COL (w)),
!        Fcons (make_number (WINDOW_TOP_EDGE_LINE (w)),
!        Fcons (make_number (WINDOW_RIGHT_EDGE_COL (w)),
!        Fcons (make_number (WINDOW_BOTTOM_EDGE_LINE (w)),
!               Qnil))));
  }
  
  /* Test if the character at column *X, row *Y is within window W.
***************
*** 513,530 ****
       register struct window *w;
       register int *x, *y;
  {
-   /* Let's make this a global enum later, instead of using numbers
-      everywhere.  */
    struct frame *f = XFRAME (WINDOW_FRAME (w));
    int left_x, right_x, top_y, bottom_y;
    enum window_part part;
!   int ux = CANON_X_UNIT (f);
!   int x0 = XFASTINT (w->left) * ux;
!   int x1 = x0 + XFASTINT (w->width) * ux;
    /* The width of the area where the vertical line can be dragged.
       (Between mode lines for instance.  */
    int grabbable_width = ux;
!   int lmargin_width = 0, rmargin_width = 0;
  
    if (*x < x0 || *x >= x1)
      return ON_NOTHING;
--- 520,535 ----
       register struct window *w;
       register int *x, *y;
  {
    struct frame *f = XFRAME (WINDOW_FRAME (w));
    int left_x, right_x, top_y, bottom_y;
    enum window_part part;
!   int ux = FRAME_COLUMN_WIDTH (f);
!   int x0 = WINDOW_LEFT_EDGE_X (w);
!   int x1 = WINDOW_RIGHT_EDGE_X (w);
    /* The width of the area where the vertical line can be dragged.
       (Between mode lines for instance.  */
    int grabbable_width = ux;
!   int lmargin_width, rmargin_width, text_left, text_right;
  
    if (*x < x0 || *x >= x1)
      return ON_NOTHING;
***************
*** 534,553 ****
    if (w->pseudo_window_p)
      {
        left_x = 0;
!       right_x = XFASTINT (w->width) * CANON_X_UNIT (f) - 1;
!       top_y = WINDOW_DISPLAY_TOP_EDGE_PIXEL_Y (w);
!       bottom_y = WINDOW_DISPLAY_BOTTOM_EDGE_PIXEL_Y (w);
      }
    else
      {
!       left_x = (WINDOW_DISPLAY_LEFT_EDGE_PIXEL_X (w)
!               - FRAME_INTERNAL_BORDER_WIDTH_SAFE (f));
!       right_x = WINDOW_DISPLAY_RIGHT_EDGE_PIXEL_X (w) - 1;
!       top_y = (WINDOW_DISPLAY_TOP_EDGE_PIXEL_Y (w)
!              - FRAME_INTERNAL_BORDER_WIDTH_SAFE (f));
!       bottom_y = WINDOW_DISPLAY_BOTTOM_EDGE_PIXEL_Y (w);
      }
  
    /* On the mode line or header line?  If it's near the start of
       the mode or header line of window that's has a horizontal
       sibling, say it's on the vertical line.  That's to be able
--- 539,560 ----
    if (w->pseudo_window_p)
      {
        left_x = 0;
!       right_x = WINDOW_TOTAL_WIDTH (w) - 1;
!       top_y = WINDOW_TOP_EDGE_Y (w);
!       bottom_y = WINDOW_BOTTOM_EDGE_Y (w);
      }
    else
      {
!       left_x = WINDOW_BOX_LEFT_EDGE_X (w);
!       right_x = WINDOW_BOX_RIGHT_EDGE_X (w) - 1;
!       top_y = WINDOW_TOP_EDGE_Y (w);
!       bottom_y = WINDOW_BOTTOM_EDGE_Y (w);
      }
  
+   /* Outside any interesting row?  */
+   if (*y < top_y || *y >= bottom_y)
+     return ON_NOTHING;
+ 
    /* On the mode line or header line?  If it's near the start of
       the mode or header line of window that's has a horizontal
       sibling, say it's on the vertical line.  That's to be able
***************
*** 558,687 ****
        && *y >= bottom_y - CURRENT_MODE_LINE_HEIGHT (w)
        && *y < bottom_y)
      {
        /* We're somewhere on the mode line.  We consider the place
         between mode lines of horizontally adjacent mode lines
         as the vertical border.    If scroll bars on the left,
         return the right window.  */
!       part = ON_MODE_LINE;
! 
!       if (FRAME_HAS_VERTICAL_SCROLL_BARS_ON_LEFT (f))
        {
!         if (abs (*x - x0) < grabbable_width)
!           part = ON_VERTICAL_BORDER;
        }
!       else if (!WINDOW_RIGHTMOST_P (w) && abs (*x - x1) < grabbable_width)
!       part = ON_VERTICAL_BORDER;
!     }
!   else if (WINDOW_WANTS_HEADER_LINE_P (w)
!          && *y < top_y + CURRENT_HEADER_LINE_HEIGHT (w)
!          && *y >= top_y)
!     {
!       part = ON_HEADER_LINE;
! 
!       if (FRAME_HAS_VERTICAL_SCROLL_BARS_ON_LEFT (f))
        {
!         if (abs (*x - x0) < grabbable_width)
!           part = ON_VERTICAL_BORDER;
        }
!       else if (!WINDOW_RIGHTMOST_P (w) && abs (*x - x1) < grabbable_width)
!       part = ON_VERTICAL_BORDER;
      }
!   /* Outside anything interesting?  */
!   else if (*y < top_y
!          || *y >= bottom_y
!          || *x < (left_x
!                   - FRAME_LEFT_FRINGE_WIDTH (f)
!                   - FRAME_LEFT_SCROLL_BAR_WIDTH (f) * ux)
!          || *x > (right_x
!                   + FRAME_RIGHT_FRINGE_WIDTH (f)
!                   + FRAME_RIGHT_SCROLL_BAR_WIDTH (f) * ux))
      {
!       part = ON_NOTHING;
      }
!   else if (FRAME_WINDOW_P (f))
      {
        if (!w->pseudo_window_p
!         && !FRAME_HAS_VERTICAL_SCROLL_BARS (f)
          && !WINDOW_RIGHTMOST_P (w)
!         && (abs (*x - right_x - FRAME_RIGHT_FRINGE_WIDTH (f)) < 
grabbable_width))
!       {
!         part = ON_VERTICAL_BORDER;
!       }
!       else if (*x < left_x || *x > right_x)
!       {
!         /* Other lines than the mode line don't include fringes and
!            scroll bars on the left.  */
! 
!         /* Convert X and Y to window-relative pixel coordinates.  */
!         *x -= left_x;
!         *y -= top_y;
!         part = *x < left_x ? ON_LEFT_FRINGE : ON_RIGHT_FRINGE;
!       }
!       else
!       {
!         lmargin_width = window_box_width (w, LEFT_MARGIN_AREA);
!         rmargin_width = window_box_width (w, RIGHT_MARGIN_AREA);
!         /* You can never be on a margin area if its width is zero.  */
!         if (lmargin_width
!             && *x <= window_box_right (w, LEFT_MARGIN_AREA))
!           part = ON_LEFT_MARGIN;
!         else if (rmargin_width
!                  && *x >= window_box_left (w, RIGHT_MARGIN_AREA))
!           part = ON_RIGHT_MARGIN;
!         else
!           {
!             part = ON_TEXT;
!             *x -= left_x;
!             *y -= top_y;
!           }
!       }
      }
    else
      {
        /* Need to say "*x > right_x" rather than >=, since on character
         terminals, the vertical line's x coordinate is right_x.  */
!       if (*x < left_x || *x > right_x)
!       {
!         /* Other lines than the mode line don't include fringes and
!            scroll bars on the left.  */
! 
!         /* Convert X and Y to window-relative pixel coordinates.  */
!         *x -= left_x;
!         *y -= top_y;
!         part = *x < left_x ? ON_LEFT_FRINGE : ON_RIGHT_FRINGE;
!       }
!       /* Here, too, "*x > right_x" is because of character terminals.  */
!       else if (!w->pseudo_window_p
!              && !WINDOW_RIGHTMOST_P (w)
!              && *x > right_x - ux)
        {
          /* On the border on the right side of the window?  Assume that
             this area begins at RIGHT_X minus a canonical char width.  */
!         part = ON_VERTICAL_BORDER;
!       }
!       else
!       {
!         lmargin_width = window_box_width (w, LEFT_MARGIN_AREA);
!         rmargin_width = window_box_width (w, RIGHT_MARGIN_AREA);
!         /* You can never be on a margin area if its width is zero.
!            This is especially important for character terminals.  */
!         if (lmargin_width
!             && *x <= window_box_right (w, LEFT_MARGIN_AREA))
!           part = ON_LEFT_MARGIN;
!         else if (rmargin_width
!                  && *x >= window_box_left (w, RIGHT_MARGIN_AREA))
!           part = ON_RIGHT_MARGIN;
!         else
!           {
!             part = ON_TEXT;
!             /* Convert X and Y to window-relative pixel coordinates.  */
!             *x -= left_x;
!             *y -= top_y;
!           }
        }
      }
  
!   return part;
  }
  
  
--- 565,664 ----
        && *y >= bottom_y - CURRENT_MODE_LINE_HEIGHT (w)
        && *y < bottom_y)
      {
+       part = ON_MODE_LINE;
+ 
+     header_vertical_border_check:
        /* We're somewhere on the mode line.  We consider the place
         between mode lines of horizontally adjacent mode lines
         as the vertical border.    If scroll bars on the left,
         return the right window.  */
!       if (WINDOW_HAS_VERTICAL_SCROLL_BAR_ON_LEFT (w)
!         || WINDOW_RIGHTMOST_P (w))
        {
!         if (!WINDOW_LEFTMOST_P (w) && abs (*x - x0) < grabbable_width)
!           return ON_VERTICAL_BORDER;
        }
!       else 
        {
!         if (abs (*x - x1) < grabbable_width)
!           return ON_VERTICAL_BORDER;
        }
! 
!       return part;
      }
! 
!   if (WINDOW_WANTS_HEADER_LINE_P (w)
!       && *y >= top_y
!       && *y < top_y + CURRENT_HEADER_LINE_HEIGHT (w))
      {
!       part = ON_HEADER_LINE;
!       goto header_vertical_border_check;
      }
! 
!   /* Outside any interesting column?  */
!   if (*x < left_x || *x > right_x)
!     return ON_NOTHING;
! 
!   lmargin_width = window_box_width (w, LEFT_MARGIN_AREA);
!   rmargin_width = window_box_width (w, RIGHT_MARGIN_AREA);
!   
!   text_left = window_box_left (w, TEXT_AREA);
!   text_right = text_left + window_box_width (w, TEXT_AREA);
! 
!   if (FRAME_WINDOW_P (f))
      {
        if (!w->pseudo_window_p
!         && !WINDOW_HAS_VERTICAL_SCROLL_BAR (w)
          && !WINDOW_RIGHTMOST_P (w)
!         && (abs (*x - right_x) < grabbable_width))
!       return ON_VERTICAL_BORDER;
      }
    else
      {
        /* Need to say "*x > right_x" rather than >=, since on character
         terminals, the vertical line's x coordinate is right_x.  */
!       if (!w->pseudo_window_p
!         && !WINDOW_RIGHTMOST_P (w)
!         && *x > right_x - ux)
        {
          /* On the border on the right side of the window?  Assume that
             this area begins at RIGHT_X minus a canonical char width.  */
!         return ON_VERTICAL_BORDER;
        }
      }
  
!   if (*x < text_left)
!     {
!       if (lmargin_width > 0
!         && (WINDOW_HAS_FRINGES_OUTSIDE_MARGINS (w)
!             ? (*x >= left_x + WINDOW_LEFT_FRINGE_WIDTH (w))
!             : (*x < left_x + lmargin_width)))
!       return ON_LEFT_MARGIN;
! 
!       /* Convert X and Y to window-relative pixel coordinates.  */
!       *x -= left_x;
!       *y -= top_y;
!       return ON_LEFT_FRINGE;
!     }
! 
!   if (*x >= text_right)
!     {
!       if (rmargin_width > 0
!         && (WINDOW_HAS_FRINGES_OUTSIDE_MARGINS (w)
!             ? (*x < right_x - WINDOW_RIGHT_FRINGE_WIDTH (w))
!             : (*x >= right_x - rmargin_width)))
!       return ON_RIGHT_MARGIN;
! 
!       /* Convert X and Y to window-relative pixel coordinates.  */
!       *x -= left_x + WINDOW_LEFT_FRINGE_WIDTH (w);
!       *y -= top_y;
!       return ON_RIGHT_FRINGE;
!     }
! 
!   /* Everything special ruled out - must be on text area */
!   *x -= left_x + WINDOW_LEFT_FRINGE_WIDTH (w);
!   *y -= top_y;
!   return ON_TEXT;
  }
  
  
***************
*** 718,725 ****
    ly = Fcdr (coordinates);
    CHECK_NUMBER_OR_FLOAT (lx);
    CHECK_NUMBER_OR_FLOAT (ly);
!   x = PIXEL_X_FROM_CANON_X (f, lx);
!   y = PIXEL_Y_FROM_CANON_Y (f, ly);
  
    switch (coordinates_in_window (w, &x, &y))
      {
--- 695,702 ----
    ly = Fcdr (coordinates);
    CHECK_NUMBER_OR_FLOAT (lx);
    CHECK_NUMBER_OR_FLOAT (ly);
!   x = FRAME_PIXEL_X_FROM_CANON_X (f, lx);
!   y = FRAME_PIXEL_Y_FROM_CANON_Y (f, ly);
  
    switch (coordinates_in_window (w, &x, &y))
      {
***************
*** 729,736 ****
      case ON_TEXT:
        /* X and Y are now window relative pixel coordinates.  Convert
         them to canonical char units before returning them.  */
!       return Fcons (CANON_X_FROM_PIXEL_X (f, x),
!                   CANON_Y_FROM_PIXEL_Y (f, y));
  
      case ON_MODE_LINE:
        return Qmode_line;
--- 706,713 ----
      case ON_TEXT:
        /* X and Y are now window relative pixel coordinates.  Convert
         them to canonical char units before returning them.  */
!       return Fcons (FRAME_CANON_X_FROM_PIXEL_X (f, x),
!                   FRAME_CANON_Y_FROM_PIXEL_Y (f, y));
  
      case ON_MODE_LINE:
        return Qmode_line;
***************
*** 797,804 ****
  
  
  /* Find the window containing frame-relative pixel position X/Y and
!    return it as a Lisp_Object.  If X, Y is on one of the window's 
!    special `window_part' elements, set *PART to the id of that element.
     If there is no window under X, Y return nil and leave *PART
     unmodified.  TOOL_BAR_P non-zero means detect tool-bar windows.
  
--- 774,785 ----
  
  
  /* Find the window containing frame-relative pixel position X/Y and
!    return it as a Lisp_Object.
! 
!    If X, Y is on one of the window's special `window_part' elements,
!    set *PART to the id of that element, and return X and Y converted
!    to window relative coordinates in WX and WY.
! 
     If there is no window under X, Y return nil and leave *PART
     unmodified.  TOOL_BAR_P non-zero means detect tool-bar windows.
  
***************
*** 812,821 ****
     case.  */
  
  Lisp_Object
! window_from_coordinates (f, x, y, part, tool_bar_p)
       struct frame *f;
       int x, y;
       enum window_part *part;
       int tool_bar_p;
  {
    Lisp_Object window;
--- 793,803 ----
     case.  */
  
  Lisp_Object
! window_from_coordinates (f, x, y, part, wx, wy, tool_bar_p)
       struct frame *f;
       int x, y;
       enum window_part *part;
+      int *wx, *wy;
       int tool_bar_p;
  {
    Lisp_Object window;
***************
*** 834,840 ****
    if (NILP (window)
        && tool_bar_p
        && WINDOWP (f->tool_bar_window)
!       && XINT (XWINDOW (f->tool_bar_window)->height) > 0
        && (coordinates_in_window (XWINDOW (f->tool_bar_window), &x, &y)
          != ON_NOTHING))
      {
--- 816,822 ----
    if (NILP (window)
        && tool_bar_p
        && WINDOWP (f->tool_bar_window)
!       && WINDOW_TOTAL_LINES (XWINDOW (f->tool_bar_window)) > 0
        && (coordinates_in_window (XWINDOW (f->tool_bar_window), &x, &y)
          != ON_NOTHING))
      {
***************
*** 842,847 ****
--- 824,832 ----
        window = f->tool_bar_window;
      }
  
+   if (wx) *wx = x;
+   if (wy) *wy = y;
+ 
    return window;
  }
  
***************
*** 865,873 ****
    CHECK_NUMBER_OR_FLOAT (y);
  
    return window_from_coordinates (f,
!                                 PIXEL_X_FROM_CANON_X (f, x),
!                                 PIXEL_Y_FROM_CANON_Y (f, y),
!                                 0, 0);
  }
  
  DEFUN ("window-point", Fwindow_point, Swindow_point, 0, 1, 0,
--- 850,858 ----
    CHECK_NUMBER_OR_FLOAT (y);
  
    return window_from_coordinates (f,
!                                 FRAME_PIXEL_X_FROM_CANON_X (f, x),
!                                 FRAME_PIXEL_Y_FROM_CANON_Y (f, y),
!                                 0, 0, 0, 0);
  }
  
  DEFUN ("window-point", Fwindow_point, Swindow_point, 0, 1, 0,
***************
*** 1167,1176 ****
    if (EQ (old, FRAME_ROOT_WINDOW (XFRAME (o->frame))))
      FRAME_ROOT_WINDOW (XFRAME (o->frame)) = replacement;
  
!   p->left = o->left;
!   p->top = o->top;
!   p->width = o->width;
!   p->height = o->height;
    p->desired_matrix = p->current_matrix = 0;
    p->vscroll = 0;
    bzero (&p->cursor, sizeof (p->cursor));
--- 1152,1161 ----
    if (EQ (old, FRAME_ROOT_WINDOW (XFRAME (o->frame))))
      FRAME_ROOT_WINDOW (XFRAME (o->frame)) = replacement;
  
!   p->left_col = o->left_col;
!   p->top_line = o->top_line;
!   p->total_cols = o->total_cols;
!   p->total_lines = o->total_lines;
    p->desired_matrix = p->current_matrix = 0;
    p->vscroll = 0;
    bzero (&p->cursor, sizeof (p->cursor));
***************
*** 1184,1190 ****
    XSETFASTINT (p->window_end_pos, 0);
    p->window_end_valid = Qnil;
    p->frozen_window_start_p = 0;
!   p->orig_top = p->orig_height = Qnil;
  
    p->next = tem = o->next;
    if (!NILP (tem))
--- 1169,1175 ----
    XSETFASTINT (p->window_end_pos, 0);
    p->window_end_valid = Qnil;
    p->frozen_window_start_p = 0;
!   p->orig_top_line = p->orig_total_lines = Qnil;
  
    p->next = tem = o->next;
    if (!NILP (tem))
***************
*** 1343,1360 ****
         set_window_{height,width} will re-position the sibling's
         children.  */
        sib = p->next;
!       XWINDOW (sib)->top = p->top;
!       XWINDOW (sib)->left = p->left;
      }
  
    /* Stretch that sibling.  */
    if (!NILP (par->vchild))
      set_window_height (sib,
!                      XFASTINT (XWINDOW (sib)->height) + XFASTINT (p->height),
                       1);
    if (!NILP (par->hchild))
      set_window_width (sib,
!                     XFASTINT (XWINDOW (sib)->width) + XFASTINT (p->width),
                      1);
  
    /* If parent now has only one child,
--- 1328,1345 ----
         set_window_{height,width} will re-position the sibling's
         children.  */
        sib = p->next;
!       XWINDOW (sib)->top_line = p->top_line;
!       XWINDOW (sib)->left_col = p->left_col;
      }
  
    /* Stretch that sibling.  */
    if (!NILP (par->vchild))
      set_window_height (sib,
!                      XFASTINT (XWINDOW (sib)->total_lines) + XFASTINT 
(p->total_lines),
                       1);
    if (!NILP (par->hchild))
      set_window_width (sib,
!                     XFASTINT (XWINDOW (sib)->total_cols) + XFASTINT 
(p->total_cols),
                      1);
  
    /* If parent now has only one child,
***************
*** 1890,1896 ****
                       display there.  */
                    Lisp_Object buffer;
                    buffer = Fother_buffer (obj, Qnil, w->frame);
!                   Fset_window_buffer (window, buffer);
                    if (EQ (window, selected_window))
                      Fset_buffer (w->buffer);
                  }
--- 1875,1881 ----
                       display there.  */
                    Lisp_Object buffer;
                    buffer = Fother_buffer (obj, Qnil, w->frame);
!                   Fset_window_buffer (window, buffer, Qnil);
                    if (EQ (window, selected_window))
                      Fset_buffer (w->buffer);
                  }
***************
*** 1910,1917 ****
              else
                {
                  struct window *b = XWINDOW (best_window);
!                 if (XFASTINT (w->height) * XFASTINT (w->width)
!                     > XFASTINT (b->height) * XFASTINT (b->width))
                    best_window = window;
                }
            }
--- 1895,1902 ----
              else
                {
                  struct window *b = XWINDOW (best_window);
!                 if (XFASTINT (w->total_lines) * XFASTINT (w->total_cols)
!                     > XFASTINT (b->total_lines) * XFASTINT (b->total_cols))
                    best_window = window;
                }
            }
***************
*** 1954,1960 ****
                  {
                    /* Otherwise show a different buffer in the window.  */
                    w->dedicated = Qnil;
!                   Fset_window_buffer (window, buffer);
                    if (EQ (window, selected_window))
                      Fset_buffer (w->buffer);
                  }
--- 1939,1945 ----
                  {
                    /* Otherwise show a different buffer in the window.  */
                    w->dedicated = Qnil;
!                   Fset_window_buffer (window, buffer, Qnil);
                    if (EQ (window, selected_window))
                      Fset_buffer (w->buffer);
                  }
***************
*** 2059,2065 ****
    w = XWINDOW (window);
  
    startpos = marker_position (w->start);
!   top = XFASTINT (w->top) - FRAME_TOP_MARGIN (XFRAME (WINDOW_FRAME (w)));
  
    if (MINI_WINDOW_P (w) && top > 0)
      error ("Can't expand minibuffer to full frame");
--- 2044,2050 ----
    w = XWINDOW (window);
  
    startpos = marker_position (w->start);
!   top = WINDOW_TOP_EDGE_LINE (w) - FRAME_TOP_MARGIN (XFRAME (WINDOW_FRAME 
(w)));
  
    if (MINI_WINDOW_P (w) && top > 0)
      error ("Can't expand minibuffer to full frame");
***************
*** 2071,2077 ****
       on the frame.  But don't try to do this if the window start is
       outside the visible portion (as might happen when the display is
       not current, due to typeahead).  */
!   new_top = XFASTINT (w->top) - FRAME_TOP_MARGIN (XFRAME (WINDOW_FRAME (w)));
    if (new_top != top
        && startpos >= BUF_BEGV (XBUFFER (w->buffer))
        && startpos <= BUF_ZV (XBUFFER (w->buffer)))
--- 2056,2062 ----
       on the frame.  But don't try to do this if the window start is
       outside the visible portion (as might happen when the display is
       not current, due to typeahead).  */
!   new_top = WINDOW_TOP_EDGE_LINE (w) - FRAME_TOP_MARGIN (XFRAME (WINDOW_FRAME 
(w)));
    if (new_top != top
        && startpos >= BUF_BEGV (XBUFFER (w->buffer))
        && startpos <= BUF_ZV (XBUFFER (w->buffer)))
***************
*** 2426,2432 ****
      *fixed = fixed_p;
  
    if (fixed_p)
!     size = width_p ? XFASTINT (w->width) : XFASTINT (w->height);
    else
      size = window_min_size_1 (w, width_p);
  
--- 2411,2417 ----
      *fixed = fixed_p;
  
    if (fixed_p)
!     size = width_p ? XFASTINT (w->total_cols) : XFASTINT (w->total_lines);
    else
      size = window_min_size_1 (w, width_p);
  
***************
*** 2434,2439 ****
--- 2419,2461 ----
  }
  
  
+ /* Adjust the margins of window W if text area is too small.  
+    Return 1 if window width is ok after adjustment; 0 if window
+    is still too narrow.  */
+ 
+ static int
+ adjust_window_margins (w)
+      struct window *w;
+ {
+   int box_cols = (WINDOW_TOTAL_COLS (w)
+                 - WINDOW_FRINGE_COLS (w)
+                 - WINDOW_SCROLL_BAR_COLS (w));
+   int margin_cols = (WINDOW_LEFT_MARGIN_COLS (w)
+                    + WINDOW_RIGHT_MARGIN_COLS (w));
+ 
+   if (box_cols - margin_cols >= MIN_SAFE_WINDOW_WIDTH)
+     return 1;
+ 
+   if (margin_cols < 0 || box_cols < MIN_SAFE_WINDOW_WIDTH)
+     return 0;
+ 
+   /* Window's text area is too narrow, but reducing the window
+      margins will fix that.  */
+   margin_cols = box_cols - MIN_SAFE_WINDOW_WIDTH;
+   if (WINDOW_RIGHT_MARGIN_COLS (w) > 0)
+     {
+       if (WINDOW_LEFT_MARGIN_COLS (w) > 0)
+       w->left_margin_cols = w->right_margin_cols
+         = make_number (margin_cols/2);
+       else
+       w->right_margin_cols = make_number (margin_cols);
+     }
+   else
+     w->left_margin_cols = make_number (margin_cols);
+   return 1;
+ }
+ 
+ 
  /* Set WINDOW's height or width to SIZE.  WIDTH_P non-zero means set
     WINDOW's width.  Resize WINDOW's children, if any, so that they
     keep their proportionate size relative to WINDOW.  Propagate
***************
*** 2451,2458 ****
    struct window *w = XWINDOW (window);
    struct window *c;
    Lisp_Object child, *forward, *sideward;
!   int old_size, min_size;
  
    if (nodelete_p == 2)
      nodelete_p = 0;
  
--- 2473,2482 ----
    struct window *w = XWINDOW (window);
    struct window *c;
    Lisp_Object child, *forward, *sideward;
!   int old_size, min_size, safe_min_size;
  
+   /* We test nodelete_p != 2 and nodelete_p != 1 below, so it
+      seems like it's too soon to do this here.  ++KFS.  */
    if (nodelete_p == 2)
      nodelete_p = 0;
  
***************
*** 2464,2476 ****
       Preserve it as long as that is at all possible.  */
    if (width_p)
      {
!       old_size = XINT (w->width);
        min_size = window_min_width;
      }
    else
      {
!       old_size = XINT (w->height);
        min_size = window_min_height;
      }
  
    if (old_size < min_size && nodelete_p != 2)
--- 2488,2506 ----
       Preserve it as long as that is at all possible.  */
    if (width_p)
      {
!       old_size = WINDOW_TOTAL_COLS (w);
        min_size = window_min_width;
+       /* Ensure that there is room for the scroll bar and fringes!
+          We may reduce display margins though.  */
+       safe_min_size = (MIN_SAFE_WINDOW_WIDTH
+                      + WINDOW_FRINGE_COLS (w)
+                      + WINDOW_SCROLL_BAR_COLS (w));
      }
    else
      {
!       old_size = XINT (w->total_lines);
        min_size = window_min_height;
+       safe_min_size = MIN_SAFE_WINDOW_HEIGHT;
      }
  
    if (old_size < min_size && nodelete_p != 2)
***************
*** 2481,2489 ****
      {
        if (!MINI_WINDOW_P (w) && !NILP (w->too_small_ok))
        min_size = width_p ? MIN_SAFE_WINDOW_WIDTH : MIN_SAFE_WINDOW_HEIGHT;
!       else
!       min_size = width_p ? window_min_width : window_min_height;
! 
        if (size < min_size)
        {
          delete_window (window);
--- 2511,2518 ----
      {
        if (!MINI_WINDOW_P (w) && !NILP (w->too_small_ok))
        min_size = width_p ? MIN_SAFE_WINDOW_WIDTH : MIN_SAFE_WINDOW_HEIGHT;
!       if (min_size < safe_min_size)
!       min_size = safe_min_size;
        if (size < min_size)
        {
          delete_window (window);
***************
*** 2501,2514 ****
      {
        sideward = &w->vchild;
        forward = &w->hchild;
!       w->width = make_number (size);
      }
    else
      {
        sideward = &w->hchild;
        forward = &w->vchild;
!       w->height = make_number (size);
!       w->orig_height = Qnil;
      }
  
    if (!NILP (*sideward))
--- 2530,2544 ----
      {
        sideward = &w->vchild;
        forward = &w->hchild;
!       w->total_cols = make_number (size);
!       adjust_window_margins (w);
      }
    else
      {
        sideward = &w->hchild;
        forward = &w->vchild;
!       w->total_lines = make_number (size);
!       w->orig_total_lines = Qnil;
      }
  
    if (!NILP (*sideward))
***************
*** 2517,2525 ****
        {
          c = XWINDOW (child);
          if (width_p)
!           c->left = w->left;
          else
!           c->top = w->top;
          size_window (child, size, width_p, nodelete_p);
        }
      }
--- 2547,2555 ----
        {
          c = XWINDOW (child);
          if (width_p)
!           c->left_col = w->left_col;
          else
!           c->top_line = w->top_line;
          size_window (child, size, width_p, nodelete_p);
        }
      }
***************
*** 2537,2543 ****
          int child_size;
  
          c = XWINDOW (child);
!         child_size = width_p ? XINT (c->width) : XINT (c->height);
          total += child_size;
  
          if (window_fixed_size_p (c, width_p, 0))
--- 2567,2573 ----
          int child_size;
  
          c = XWINDOW (child);
!         child_size = width_p ? XINT (c->total_cols) : XINT (c->total_lines);
          total += child_size;
  
          if (window_fixed_size_p (c, width_p, 0))
***************
*** 2559,2580 ****
        extra = (size - total) - n * each;
  
        /* Compute new children heights and edge positions.  */
!       first_pos = width_p ? XINT (w->left) : XINT (w->top);
        last_pos = first_pos;
        for (child = *forward; !NILP (child); child = c->next)
        {
          int new_size, old_size;
  
          c = XWINDOW (child);
!         old_size = width_p ? XFASTINT (c->width) : XFASTINT (c->height);
          new_size = old_size;
  
          /* The top or left edge position of this child equals the
             bottom or right edge of its predecessor.  */
          if (width_p)
!           c->left = make_number (last_pos);
          else
!           c->top = make_number (last_pos);
  
          /* If this child can be resized, do it.  */
          if (resize_fixed_p || !window_fixed_size_p (c, width_p, 0))
--- 2589,2610 ----
        extra = (size - total) - n * each;
  
        /* Compute new children heights and edge positions.  */
!       first_pos = width_p ? XINT (w->left_col) : XINT (w->top_line);
        last_pos = first_pos;
        for (child = *forward; !NILP (child); child = c->next)
        {
          int new_size, old_size;
  
          c = XWINDOW (child);
!         old_size = width_p ? XFASTINT (c->total_cols) : XFASTINT 
(c->total_lines);
          new_size = old_size;
  
          /* The top or left edge position of this child equals the
             bottom or right edge of its predecessor.  */
          if (width_p)
!           c->left_col = make_number (last_pos);
          else
!           c->top_line = make_number (last_pos);
  
          /* If this child can be resized, do it.  */
          if (resize_fixed_p || !window_fixed_size_p (c, width_p, 0))
***************
*** 2602,2608 ****
          {
            int child_size;
            c = XWINDOW (child);
!           child_size = width_p ? XINT (c->width) : XINT (c->height);
            size_window (child, child_size, width_p, 2);
          }
      }
--- 2632,2638 ----
          {
            int child_size;
            c = XWINDOW (child);
!           child_size = width_p ? XINT (c->total_cols) : XINT (c->total_lines);
            size_window (child, child_size, width_p, 2);
          }
      }
***************
*** 2646,2658 ****
  {
    struct window *w = XWINDOW (window);
  
!   XSETFASTINT (w->top, XFASTINT (w->top) + n);
!   XSETFASTINT (w->height, XFASTINT (w->height) - n);
  
!   if (INTEGERP (w->orig_top))
!     XSETFASTINT (w->orig_top, XFASTINT (w->orig_top) + n);
!   if (INTEGERP (w->orig_height))
!     XSETFASTINT (w->orig_height, XFASTINT (w->orig_height) - n);
  
    /* Handle just the top child in a vertical split.  */
    if (!NILP (w->vchild))
--- 2676,2688 ----
  {
    struct window *w = XWINDOW (window);
  
!   XSETFASTINT (w->top_line, XFASTINT (w->top_line) + n);
!   XSETFASTINT (w->total_lines, XFASTINT (w->total_lines) - n);
  
!   if (INTEGERP (w->orig_top_line))
!     XSETFASTINT (w->orig_top_line, XFASTINT (w->orig_top_line) + n);
!   if (INTEGERP (w->orig_total_lines))
!     XSETFASTINT (w->orig_total_lines, XFASTINT (w->orig_total_lines) - n);
  
    /* Handle just the top child in a vertical split.  */
    if (!NILP (w->vchild))
***************
*** 2680,2691 ****
  
  /* Make WINDOW display BUFFER as its contents.  RUN_HOOKS_P non-zero
     means it's allowed to run hooks.  See make_frame for a case where
!    it's not allowed.  */
  
  void
! set_window_buffer (window, buffer, run_hooks_p)
       Lisp_Object window, buffer;
!      int run_hooks_p;
  {
    struct window *w = XWINDOW (window);
    struct buffer *b = XBUFFER (buffer);
--- 2710,2723 ----
  
  /* Make WINDOW display BUFFER as its contents.  RUN_HOOKS_P non-zero
     means it's allowed to run hooks.  See make_frame for a case where
!    it's not allowed.  KEEP_MARGINS_P non-zero means that the current
!    margins, fringes, and scroll-bar settings of the window are not
!    reset from the buffer's local settings.  */
  
  void
! set_window_buffer (window, buffer, run_hooks_p, keep_margins_p)
       Lisp_Object window, buffer;
!      int run_hooks_p, keep_margins_p;
  {
    struct window *w = XWINDOW (window);
    struct buffer *b = XBUFFER (buffer);
***************
*** 2730,2737 ****
        Fset_buffer (buffer);
      }
  
!   /* Set left and right marginal area width from buffer.  */
!   Fset_window_margins (window, b->left_margin_width, b->right_margin_width);
  
    if (run_hooks_p)
      {
--- 2762,2786 ----
        Fset_buffer (buffer);
      }
  
!   if (!keep_margins_p)
!     {
!       /* Set left and right marginal area width etc. from buffer.  */
! 
!       /* This may call adjust_window_margins three times, so 
!        temporarily disable window margins.  */
!       w->left_margin_cols = w->right_margin_cols = Qnil;
! 
!       Fset_window_fringes (window,
!                          b->left_fringe_width, b->right_fringe_width,
!                          b->fringes_outside_margins);
! 
!       Fset_window_scroll_bars (window,
!                              b->scroll_bar_width,
!                              b->vertical_scroll_bar_type, Qnil);
! 
!       Fset_window_margins (window,
!                          b->left_margin_cols, b->right_margin_cols);
!     }
  
    if (run_hooks_p)
      {
***************
*** 2748,2757 ****
  }
  
  
! DEFUN ("set-window-buffer", Fset_window_buffer, Sset_window_buffer, 2, 2, 0,
         doc: /* Make WINDOW display BUFFER as its contents.
! BUFFER can be a buffer or buffer name.  */)
!      (window, buffer)
       register Lisp_Object window, buffer;
  {
    register Lisp_Object tem;
--- 2797,2810 ----
  }
  
  
! DEFUN ("set-window-buffer", Fset_window_buffer, Sset_window_buffer, 2, 3, 0,
         doc: /* Make WINDOW display BUFFER as its contents.
! BUFFER can be a buffer or buffer name.  
! Optional third arg KEEP_MARGINS non-nil means that WINDOW's current
! display margins, fringe widths, and scroll bar settings are maintained;
! the default is to reset these from BUFFER's local settings or the frame
! defaults.  */)
!      (window, buffer, keep_margins)
       register Lisp_Object window, buffer;
  {
    register Lisp_Object tem;
***************
*** 2777,2783 ****
        unshow_buffer (w);
      }
  
!   set_window_buffer (window, buffer, 1);
    return Qnil;
  }
  
--- 2830,2836 ----
        unshow_buffer (w);
      }
  
!   set_window_buffer (window, buffer, 1, !NILP (keep_margins));
    return Qnil;
  }
  
***************
*** 3047,3053 ****
    if (pop_up_frames || last_nonminibuf_frame == 0)
      {
        window = Fframe_selected_window (call0 (Vpop_up_frame_function));
!       Fset_window_buffer (window, buffer);
        return display_buffer_1 (window);
      }
  
--- 3100,3106 ----
    if (pop_up_frames || last_nonminibuf_frame == 0)
      {
        window = Fframe_selected_window (call0 (Vpop_up_frame_function));
!       Fset_window_buffer (window, buffer, Qnil);
        return display_buffer_1 (window);
      }
  
***************
*** 3136,3149 ****
          if (!NILP (other)
              && !NILP (Veven_window_heights)
              /* Check that OTHER and WINDOW are vertically arrayed.  */
!             && !EQ (XWINDOW (other)->top, XWINDOW (window)->top)
!             && (XFASTINT (XWINDOW (other)->height)
!                 > XFASTINT (XWINDOW (window)->height)))
            {
!             int total = (XFASTINT (XWINDOW (other)->height)
!                          + XFASTINT (XWINDOW (window)->height));
              enlarge_window (upper,
!                             total / 2 - XFASTINT (XWINDOW (upper)->height),
                              0, 0);
            }
        }
--- 3189,3202 ----
          if (!NILP (other)
              && !NILP (Veven_window_heights)
              /* Check that OTHER and WINDOW are vertically arrayed.  */
!             && !EQ (XWINDOW (other)->top_line, XWINDOW (window)->top_line)
!             && (XFASTINT (XWINDOW (other)->total_lines)
!                 > XFASTINT (XWINDOW (window)->total_lines)))
            {
!             int total = (XFASTINT (XWINDOW (other)->total_lines)
!                          + XFASTINT (XWINDOW (window)->total_lines));
              enlarge_window (upper,
!                             total / 2 - XFASTINT (XWINDOW 
(upper)->total_lines),
                              0, 0);
            }
        }
***************
*** 3151,3157 ****
    else
      window = Fget_lru_window (Qnil);
  
!   Fset_window_buffer (window, buffer);
    return display_buffer_1 (window);
  }
  
--- 3204,3210 ----
    else
      window = Fget_lru_window (Qnil);
  
!   Fset_window_buffer (window, buffer, Qnil);
    return display_buffer_1 (window);
  }
  
***************
*** 3282,3290 ****
           the usable space in columns by two.
           We round up, since the left-hand window may include
           a dividing line, while the right-hand may not.  */
!       size_int = (XFASTINT (o->width) + 1) >> 1;
        else
!       size_int = XFASTINT (o->height) >> 1;
      }
    else
      {
--- 3335,3343 ----
           the usable space in columns by two.
           We round up, since the left-hand window may include
           a dividing line, while the right-hand may not.  */
!       size_int = (XFASTINT (o->total_cols) + 1) >> 1;
        else
!       size_int = XFASTINT (o->total_lines) >> 1;
      }
    else
      {
***************
*** 3303,3311 ****
      {
        if (size_int < window_min_height)
        error ("Window height %d too small (after splitting)", size_int);
!       if (size_int + window_min_height > XFASTINT (o->height))
        error ("Window height %d too small (after splitting)",
!              XFASTINT (o->height) - size_int);
        if (NILP (o->parent)
          || NILP (XWINDOW (o->parent)->vchild))
        {
--- 3356,3364 ----
      {
        if (size_int < window_min_height)
        error ("Window height %d too small (after splitting)", size_int);
!       if (size_int + window_min_height > XFASTINT (o->total_lines))
        error ("Window height %d too small (after splitting)",
!              XFASTINT (o->total_lines) - size_int);
        if (NILP (o->parent)
          || NILP (XWINDOW (o->parent)->vchild))
        {
***************
*** 3319,3327 ****
        if (size_int < window_min_width)
        error ("Window width %d too small (after splitting)", size_int);
  
!       if (size_int + window_min_width > XFASTINT (o->width))
        error ("Window width %d too small (after splitting)",
!              XFASTINT (o->width) - size_int);
        if (NILP (o->parent)
          || NILP (XWINDOW (o->parent)->hchild))
        {
--- 3372,3380 ----
        if (size_int < window_min_width)
        error ("Window width %d too small (after splitting)", size_int);
  
!       if (size_int + window_min_width > XFASTINT (o->total_cols))
        error ("Window width %d too small (after splitting)",
!              XFASTINT (o->total_cols) - size_int);
        if (NILP (o->parent)
          || NILP (XWINDOW (o->parent)->hchild))
        {
***************
*** 3351,3378 ****
    p->window_end_valid = Qnil;
    bzero (&p->last_cursor, sizeof p->last_cursor);
  
    /* Apportion the available frame space among the two new windows */
  
    if (!NILP (horflag))
      {
!       p->height = o->height;
!       p->top = o->top;
!       XSETFASTINT (p->width, XFASTINT (o->width) - size_int);
!       XSETFASTINT (o->width, size_int);
!       XSETFASTINT (p->left, XFASTINT (o->left) + size_int);
      }
    else
      {
!       p->left = o->left;
!       p->width = o->width;
!       XSETFASTINT (p->height, XFASTINT (o->height) - size_int);
!       XSETFASTINT (o->height, size_int);
!       XSETFASTINT (p->top, XFASTINT (o->top) + size_int);
      }
  
    /* Adjust glyph matrices.  */
    adjust_glyphs (fo);
!   Fset_window_buffer (new, o->buffer);
    return new;
  }
  
--- 3404,3444 ----
    p->window_end_valid = Qnil;
    bzero (&p->last_cursor, sizeof p->last_cursor);
  
+   /* Duplicate special geometry settings.  */
+ 
+   p->left_margin_cols = o->left_margin_cols;
+   p->right_margin_cols = o->right_margin_cols;
+   p->left_fringe_width = o->left_fringe_width;
+   p->right_fringe_width = o->right_fringe_width;
+   p->fringes_outside_margins = o->fringes_outside_margins;
+   p->scroll_bar_width = o->scroll_bar_width;
+   p->vertical_scroll_bar_type = o->vertical_scroll_bar_type;
+ 
    /* Apportion the available frame space among the two new windows */
  
    if (!NILP (horflag))
      {
!       p->total_lines = o->total_lines;
!       p->top_line = o->top_line;
!       XSETFASTINT (p->total_cols, XFASTINT (o->total_cols) - size_int);
!       XSETFASTINT (o->total_cols, size_int);
!       XSETFASTINT (p->left_col, XFASTINT (o->left_col) + size_int);
!       adjust_window_margins (p);
!       adjust_window_margins (o);
      }
    else
      {
!       p->left_col = o->left_col;
!       p->total_cols = o->total_cols;
!       XSETFASTINT (p->total_lines, XFASTINT (o->total_lines) - size_int);
!       XSETFASTINT (o->total_lines, size_int);
!       XSETFASTINT (p->top_line, XFASTINT (o->top_line) + size_int);
      }
  
    /* Adjust glyph matrices.  */
    adjust_glyphs (fo);
! 
!   Fset_window_buffer (new, o->buffer, Qt);
    return new;
  }
  
***************
*** 3423,3429 ****
       Lisp_Object window;
  {
    register struct window *p = XWINDOW (window);
!   return XFASTINT (p->height);
  }
  
  int
--- 3489,3495 ----
       Lisp_Object window;
  {
    register struct window *p = XWINDOW (window);
!   return WINDOW_TOTAL_LINES (p);
  }
  
  int
***************
*** 3431,3445 ****
       Lisp_Object window;
  {
    register struct window *p = XWINDOW (window);
!   return XFASTINT (p->width);
  }
  
  
  #define CURBEG(w) \
!   *(widthflag ? &(XWINDOW (w)->left) : &(XWINDOW (w)->top))
  
  #define CURSIZE(w) \
!   *(widthflag ? &(XWINDOW (w)->width) : &(XWINDOW (w)->height))
  
  
  /* Enlarge WINDOW by DELTA.  WIDTHFLAG non-zero means
--- 3497,3511 ----
       Lisp_Object window;
  {
    register struct window *p = XWINDOW (window);
!   return WINDOW_TOTAL_COLS (p);
  }
  
  
  #define CURBEG(w) \
!   *(widthflag ? &(XWINDOW (w)->left_col) : &(XWINDOW (w)->top_line))
  
  #define CURSIZE(w) \
!   *(widthflag ? &(XWINDOW (w)->total_cols) : &(XWINDOW (w)->total_lines))
  
  
  /* Enlarge WINDOW by DELTA.  WIDTHFLAG non-zero means
***************
*** 3770,3784 ****
    windows_or_buffers_changed++;
    FRAME_WINDOW_SIZES_CHANGED (XFRAME (WINDOW_FRAME (w))) = 1;
  
!   old_height = XFASTINT (w->height);
!   XSETFASTINT (w->height, height);
  
    if (!NILP (w->hchild))
      {
        for (child = w->hchild; !NILP (child); child = c->next)
        {
          c = XWINDOW (child);
!         c->top = w->top;
          shrink_window_lowest_first (c, height);
        }
      }
--- 3836,3850 ----
    windows_or_buffers_changed++;
    FRAME_WINDOW_SIZES_CHANGED (XFRAME (WINDOW_FRAME (w))) = 1;
  
!   old_height = XFASTINT (w->total_lines);
!   XSETFASTINT (w->total_lines, height);
  
    if (!NILP (w->hchild))
      {
        for (child = w->hchild; !NILP (child); child = c->next)
        {
          c = XWINDOW (child);
!         c->top_line = w->top_line;
          shrink_window_lowest_first (c, height);
        }
      }
***************
*** 3802,3824 ****
          int this_one;
  
          c = XWINDOW (child);
!         this_one = XFASTINT (c->height) - MIN_SAFE_WINDOW_HEIGHT;
  
          if (this_one > delta)
            this_one = delta;
  
!         shrink_window_lowest_first (c, XFASTINT (c->height) - this_one);
          delta -= this_one;
        }
  
        /* Compute new positions.  */
!       last_top = XINT (w->top);
        for (child = w->vchild; !NILP (child); child = c->next)
        {
          c = XWINDOW (child);
!         c->top = make_number (last_top);
!         shrink_window_lowest_first (c, XFASTINT (c->height));
!         last_top += XFASTINT (c->height);
        }
      }
  }
--- 3868,3890 ----
          int this_one;
  
          c = XWINDOW (child);
!         this_one = XFASTINT (c->total_lines) - MIN_SAFE_WINDOW_HEIGHT;
  
          if (this_one > delta)
            this_one = delta;
  
!         shrink_window_lowest_first (c, XFASTINT (c->total_lines) - this_one);
          delta -= this_one;
        }
  
        /* Compute new positions.  */
!       last_top = XINT (w->top_line);
        for (child = w->vchild; !NILP (child); child = c->next)
        {
          c = XWINDOW (child);
!         c->top_line = make_number (last_top);
!         shrink_window_lowest_first (c, XFASTINT (c->total_lines));
!         last_top += XFASTINT (c->total_lines);
        }
      }
  }
***************
*** 3827,3841 ****
  /* Save, restore, or check positions and sizes in the window tree
     rooted at W.  ACTION says what to do.
  
!    If ACTION is CHECK_ORIG_SIZES, check if orig_top and orig_height
!    members are valid for all windows in the window tree.  Value is
!    non-zero if they are valid.
  
     If ACTION is SAVE_ORIG_SIZES, save members top and height in
!    orig_top and orig_height for all windows in the tree.
  
!    If ACTION is RESTORE_ORIG_SIZES, restore top and height from
!    values stored in orig_top and orig_height for all windows.  */
  
  static int
  save_restore_orig_size (w, action)
--- 3893,3907 ----
  /* Save, restore, or check positions and sizes in the window tree
     rooted at W.  ACTION says what to do.
  
!    If ACTION is CHECK_ORIG_SIZES, check if orig_top_line and
!    orig_total_lines members are valid for all windows in the window
!    tree.  Value is non-zero if they are valid.
  
     If ACTION is SAVE_ORIG_SIZES, save members top and height in
!    orig_top_line and orig_total_lines for all windows in the tree.
  
!    If ACTION is RESTORE_ORIG_SIZES, restore top and height from values
!    stored in orig_top_line and orig_total_lines for all windows.  */
  
  static int
  save_restore_orig_size (w, action)
***************
*** 3860,3881 ****
        switch (action)
        {
        case CHECK_ORIG_SIZES:
!         if (!INTEGERP (w->orig_top) || !INTEGERP (w->orig_height))
            return 0;
          break;
  
        case SAVE_ORIG_SIZES:
!         w->orig_top = w->top;
!         w->orig_height = w->height;
            XSETFASTINT (w->last_modified, 0);
            XSETFASTINT (w->last_overlay_modified, 0);
          break;
  
        case RESTORE_ORIG_SIZES:
!         xassert (INTEGERP (w->orig_top) && INTEGERP (w->orig_height));
!         w->top = w->orig_top;
!         w->height = w->orig_height;
!         w->orig_height = w->orig_top = Qnil;
            XSETFASTINT (w->last_modified, 0);
            XSETFASTINT (w->last_overlay_modified, 0);
          break;
--- 3926,3947 ----
        switch (action)
        {
        case CHECK_ORIG_SIZES:
!         if (!INTEGERP (w->orig_top_line) || !INTEGERP (w->orig_total_lines))
            return 0;
          break;
  
        case SAVE_ORIG_SIZES:
!         w->orig_top_line = w->top_line;
!         w->orig_total_lines = w->total_lines;
            XSETFASTINT (w->last_modified, 0);
            XSETFASTINT (w->last_overlay_modified, 0);
          break;
  
        case RESTORE_ORIG_SIZES:
!         xassert (INTEGERP (w->orig_top_line) && INTEGERP 
(w->orig_total_lines));
!         w->top_line = w->orig_top_line;
!         w->total_lines = w->orig_total_lines;
!         w->orig_total_lines = w->orig_top_line = Qnil;
            XSETFASTINT (w->last_modified, 0);
            XSETFASTINT (w->last_overlay_modified, 0);
          break;
***************
*** 3915,3924 ****
    if (delta)
      {
        int min_height = window_min_size (root, 0, 0, 0);
!       if (XFASTINT (root->height) - delta < min_height)
        /* Note that the root window may already be smaller than
           min_height.  */
!       delta = max (0, XFASTINT (root->height) - min_height);
      }
  
    if (delta)
--- 3981,3990 ----
    if (delta)
      {
        int min_height = window_min_size (root, 0, 0, 0);
!       if (XFASTINT (root->total_lines) - delta < min_height)
        /* Note that the root window may already be smaller than
           min_height.  */
!       delta = max (0, XFASTINT (root->total_lines) - min_height);
      }
  
    if (delta)
***************
*** 3928,3938 ****
        save_restore_orig_size (root, SAVE_ORIG_SIZES);
  
        /* Shrink other windows.  */
!       shrink_window_lowest_first (root, XFASTINT (root->height) - delta);
  
        /* Grow the mini-window.  */
!       w->top = make_number (XFASTINT (root->top) + XFASTINT (root->height));
!       w->height = make_number (XFASTINT (w->height) + delta);
        XSETFASTINT (w->last_modified, 0);
        XSETFASTINT (w->last_overlay_modified, 0);
  
--- 3994,4004 ----
        save_restore_orig_size (root, SAVE_ORIG_SIZES);
  
        /* Shrink other windows.  */
!       shrink_window_lowest_first (root, XFASTINT (root->total_lines) - delta);
  
        /* Grow the mini-window.  */
!       w->top_line = make_number (XFASTINT (root->top_line) + XFASTINT 
(root->total_lines));
!       w->total_lines = make_number (XFASTINT (w->total_lines) + delta);
        XSETFASTINT (w->last_modified, 0);
        XSETFASTINT (w->last_overlay_modified, 0);
  
***************
*** 3960,3972 ****
        FRAME_WINDOW_SIZES_CHANGED (f) = 1;
        windows_or_buffers_changed = 1;
      }
!   else if (XFASTINT (w->height) > 1)
      {
        /* Distribute the additional lines of the mini-window
         among the other windows.  */
        Lisp_Object window;
        XSETWINDOW (window, w);
!       enlarge_window (window, 1 - XFASTINT (w->height), 0, 0);
      }
  }
  
--- 4026,4038 ----
        FRAME_WINDOW_SIZES_CHANGED (f) = 1;
        windows_or_buffers_changed = 1;
      }
!   else if (XFASTINT (w->total_lines) > 1)
      {
        /* Distribute the additional lines of the mini-window
         among the other windows.  */
        Lisp_Object window;
        XSETWINDOW (window, w);
!       enlarge_window (window, 1 - XFASTINT (w->total_lines), 0, 0);
      }
  }
  
***************
*** 4001,4007 ****
  window_internal_height (w)
       struct window *w;
  {
!   int ht = XFASTINT (w->height);
  
    if (!MINI_WINDOW_P (w))
      {
--- 4067,4073 ----
  window_internal_height (w)
       struct window *w;
  {
!   int ht = XFASTINT (w->total_lines);
  
    if (!MINI_WINDOW_P (w))
      {
***************
*** 4026,4049 ****
     separating W from the sibling to its right.  */
  
  int
! window_internal_width (w)
       struct window *w;
  {
    struct frame *f = XFRAME (WINDOW_FRAME (w));
!   int width = XINT (w->width);
  
!   if (FRAME_HAS_VERTICAL_SCROLL_BARS (f))
      /* Scroll bars occupy a few columns.  */
!     width -= FRAME_SCROLL_BAR_COLS (f);
!   else if (!WINDOW_RIGHTMOST_P (w) && !WINDOW_FULL_WIDTH_P (w))
      /* The column of `|' characters separating side-by-side windows
         occupies one column only.  */
      width -= 1;
  
-   /* On window-systems, areas to the left and right of the window
-      are used as fringes.  */
    if (FRAME_WINDOW_P (f))
!     width -= FRAME_FRINGE_COLS (f);
  
    return width;
  }
--- 4092,4118 ----
     separating W from the sibling to its right.  */
  
  int
! window_box_text_cols (w)
       struct window *w;
  {
    struct frame *f = XFRAME (WINDOW_FRAME (w));
!   int width = XINT (w->total_cols);
  
!   if (WINDOW_HAS_VERTICAL_SCROLL_BAR (w))
      /* Scroll bars occupy a few columns.  */
!     width -= WINDOW_CONFIG_SCROLL_BAR_COLS (w);
!   else if (!FRAME_WINDOW_P (f)
!          && !WINDOW_RIGHTMOST_P (w) && !WINDOW_FULL_WIDTH_P (w))
      /* The column of `|' characters separating side-by-side windows
         occupies one column only.  */
      width -= 1;
  
    if (FRAME_WINDOW_P (f))
!     /* On window-systems, fringes and display margins cannot be
!        used for normal text.  */
!     width -= (WINDOW_FRINGE_COLS (w)
!             + WINDOW_LEFT_MARGIN_COLS (w)
!             + WINDOW_RIGHT_MARGIN_COLS (w));
  
    return width;
  }
***************
*** 4148,4154 ****
    if (whole)
      {
        int screen_full = (window_box_height (w)
!                        - next_screen_context_lines * CANON_Y_UNIT (it.f));
        int dy = n * screen_full;
  
        /* Note that move_it_vertically always moves the iterator to the
--- 4217,4223 ----
    if (whole)
      {
        int screen_full = (window_box_height (w)
!                        - next_screen_context_lines * FRAME_LINE_HEIGHT 
(it.f));
        int dy = n * screen_full;
  
        /* Note that move_it_vertically always moves the iterator to the
***************
*** 4245,4252 ****
      {
        /* Move PT out of scroll margins.  */
        this_scroll_margin = max (0, scroll_margin);
!       this_scroll_margin = min (this_scroll_margin, XFASTINT (w->height) / 4);
!       this_scroll_margin *= CANON_Y_UNIT (it.f);
  
        if (n > 0)
        {
--- 4314,4321 ----
      {
        /* Move PT out of scroll margins.  */
        this_scroll_margin = max (0, scroll_margin);
!       this_scroll_margin = min (this_scroll_margin, XFASTINT (w->total_lines) 
/ 4);
!       this_scroll_margin *= FRAME_LINE_HEIGHT (it.f);
  
        if (n > 0)
        {
***************
*** 4323,4329 ****
  
    posit = *compute_motion (startpos, 0, 0, 0,
                           PT, ht, 0,
!                          window_internal_width (w), XINT (w->hscroll),
                           0, w);
    original_vpos = posit.vpos;
  
--- 4392,4398 ----
  
    posit = *compute_motion (startpos, 0, 0, 0,
                           PT, ht, 0,
!                          window_box_text_cols (w), XINT (w->hscroll),
                           0, w);
    original_vpos = posit.vpos;
  
***************
*** 4360,4367 ****
        if (this_scroll_margin < 0)
        this_scroll_margin = 0;
  
!       if (XINT (w->height) < 4 * scroll_margin)
!       this_scroll_margin = XINT (w->height) / 4;
  
        set_marker_restricted_both (w->start, w->buffer, pos, pos_byte);
        w->start_at_line_beg = bolp;
--- 4429,4436 ----
        if (this_scroll_margin < 0)
        this_scroll_margin = 0;
  
!       if (XINT (w->total_lines) < 4 * scroll_margin)
!       this_scroll_margin = XINT (w->total_lines) / 4;
  
        set_marker_restricted_both (w->start, w->buffer, pos, pos_byte);
        w->start_at_line_beg = bolp;
***************
*** 4611,4617 ****
    struct window *w = XWINDOW (selected_window);
  
    if (NILP (arg))
!     XSETFASTINT (arg, window_internal_width (w) - 2);
    else
      arg = Fprefix_numeric_value (arg);
  
--- 4680,4686 ----
    struct window *w = XWINDOW (selected_window);
  
    if (NILP (arg))
!     XSETFASTINT (arg, window_box_text_cols (w) - 2);
    else
      arg = Fprefix_numeric_value (arg);
  
***************
*** 4641,4647 ****
    struct window *w = XWINDOW (selected_window);
  
    if (NILP (arg))
!     XSETFASTINT (arg, window_internal_width (w) - 2);
    else
      arg = Fprefix_numeric_value (arg);
  
--- 4710,4716 ----
    struct window *w = XWINDOW (selected_window);
  
    if (NILP (arg))
!     XSETFASTINT (arg, window_box_text_cols (w) - 2);
    else
      arg = Fprefix_numeric_value (arg);
  
***************
*** 4715,4721 ****
    /* Add in empty lines at the bottom of the window.  */
    if (bottom_y < height)
      {
!       int uy = CANON_Y_UNIT (it.f);
        it.vpos += (height - bottom_y + uy - 1) / uy;
      }
  
--- 4784,4790 ----
    /* Add in empty lines at the bottom of the window.  */
    if (bottom_y < height)
      {
!       int uy = FRAME_LINE_HEIGHT (it.f);
        it.vpos += (height - bottom_y + uy - 1) / uy;
      }
  
***************
*** 4807,4813 ****
          /* If we can't move down NLINES lines because we hit
             the end of the buffer, count in some empty lines.  */
          if (it.vpos < nlines)
!           y1 += (nlines - it.vpos) * CANON_Y_UNIT (it.f);
  
          h = window_box_height (w) - (y1 - y0);
  
--- 4876,4882 ----
          /* If we can't move down NLINES lines because we hit
             the end of the buffer, count in some empty lines.  */
          if (it.vpos < nlines)
!           y1 += (nlines - it.vpos) * FRAME_LINE_HEIGHT (it.f);
  
          h = window_box_height (w) - (y1 - y0);
  
***************
*** 4865,4871 ****
  {
    struct window *w = decode_window (window);
    int pixel_height = window_box_height (w);
!   int line_height = pixel_height / CANON_Y_UNIT (XFRAME (w->frame));
    return make_number (line_height);
  }
  
--- 4934,4940 ----
  {
    struct window *w = decode_window (window);
    int pixel_height = window_box_height (w);
!   int line_height = pixel_height / FRAME_LINE_HEIGHT (XFRAME (w->frame));
    return make_number (line_height);
  }
  
***************
*** 4924,4930 ****
    {
      EMACS_INT size_from_Lisp_Vector_struct;
      struct Lisp_Vector *next_from_Lisp_Vector_struct;
!     Lisp_Object frame_width, frame_height, frame_menu_bar_lines;
      Lisp_Object frame_tool_bar_lines;
      Lisp_Object selected_frame;
      Lisp_Object current_window;
--- 4993,4999 ----
    {
      EMACS_INT size_from_Lisp_Vector_struct;
      struct Lisp_Vector *next_from_Lisp_Vector_struct;
!     Lisp_Object frame_cols, frame_lines, frame_menu_bar_lines;
      Lisp_Object frame_tool_bar_lines;
      Lisp_Object selected_frame;
      Lisp_Object current_window;
***************
*** 4950,4963 ****
  
    Lisp_Object window;
    Lisp_Object buffer, start, pointm, mark;
!   Lisp_Object left, top, width, height, hscroll, min_hscroll;
    Lisp_Object parent, prev;
    Lisp_Object start_at_line_beg;
    Lisp_Object display_table;
!   Lisp_Object orig_top, orig_height;
  };
  
! #define SAVED_WINDOW_VECTOR_SIZE 17 /* Arg to Fmake_vector */
  
  #define SAVED_WINDOW_N(swv,n) \
    ((struct saved_window *) (XVECTOR ((swv)->contents[(n)])))
--- 5019,5036 ----
  
    Lisp_Object window;
    Lisp_Object buffer, start, pointm, mark;
!   Lisp_Object left_col, top_line, total_cols, total_lines;
!   Lisp_Object hscroll, min_hscroll;
    Lisp_Object parent, prev;
    Lisp_Object start_at_line_beg;
    Lisp_Object display_table;
!   Lisp_Object orig_top_line, orig_total_lines;
!   Lisp_Object left_margin_cols, right_margin_cols;
!   Lisp_Object left_fringe_width, right_fringe_width, fringes_outside_margins;
!   Lisp_Object scroll_bar_width, vertical_scroll_bar_type;
  };
  
! #define SAVED_WINDOW_VECTOR_SIZE 24 /* Arg to Fmake_vector */
  
  #define SAVED_WINDOW_N(swv,n) \
    ((struct saved_window *) (XVECTOR ((swv)->contents[(n)])))
***************
*** 5041,5048 ****
         made, we change the frame to the size specified in the
         configuration, restore the configuration, and then resize it
         back.  We keep track of the prevailing height in these variables.  */
!       int previous_frame_height = FRAME_HEIGHT (f);
!       int previous_frame_width =  FRAME_WIDTH  (f);
        int previous_frame_menu_bar_lines = FRAME_MENU_BAR_LINES (f);
        int previous_frame_tool_bar_lines = FRAME_TOOL_BAR_LINES (f);
  
--- 5114,5121 ----
         made, we change the frame to the size specified in the
         configuration, restore the configuration, and then resize it
         back.  We keep track of the prevailing height in these variables.  */
!       int previous_frame_lines = FRAME_LINES (f);
!       int previous_frame_cols =  FRAME_COLS  (f);
        int previous_frame_menu_bar_lines = FRAME_MENU_BAR_LINES (f);
        int previous_frame_tool_bar_lines = FRAME_TOOL_BAR_LINES (f);
  
***************
*** 5050,5059 ****
         if it runs during this.  */
        BLOCK_INPUT;
  
!       if (XFASTINT (data->frame_height) != previous_frame_height
!         || XFASTINT (data->frame_width) != previous_frame_width)
!       change_frame_size (f, XFASTINT (data->frame_height),
!                          XFASTINT (data->frame_width), 0, 0, 0);
  #if defined (HAVE_WINDOW_SYSTEM) || defined (MSDOS)
        if (XFASTINT (data->frame_menu_bar_lines)
          != previous_frame_menu_bar_lines)
--- 5123,5132 ----
         if it runs during this.  */
        BLOCK_INPUT;
  
!       if (XFASTINT (data->frame_lines) != previous_frame_lines
!         || XFASTINT (data->frame_cols) != previous_frame_cols)
!       change_frame_size (f, XFASTINT (data->frame_lines),
!                          XFASTINT (data->frame_cols), 0, 0, 0);
  #if defined (HAVE_WINDOW_SYSTEM) || defined (MSDOS)
        if (XFASTINT (data->frame_menu_bar_lines)
          != previous_frame_menu_bar_lines)
***************
*** 5128,5134 ****
              w->prev = Qnil;
              if (!NILP (w->parent))
                {
!                 if (EQ (p->width, XWINDOW (w->parent)->width))
                    {
                      XWINDOW (w->parent)->vchild = p->window;
                      XWINDOW (w->parent)->hchild = Qnil;
--- 5201,5207 ----
              w->prev = Qnil;
              if (!NILP (w->parent))
                {
!                 if (EQ (p->total_cols, XWINDOW (w->parent)->total_cols))
                    {
                      XWINDOW (w->parent)->vchild = p->window;
                      XWINDOW (w->parent)->hchild = Qnil;
***************
*** 5143,5159 ****
  
          /* If we squirreled away the buffer in the window's height,
             restore it now.  */
!         if (BUFFERP (w->height))
!           w->buffer = w->height;
!         w->left = p->left;
!         w->top = p->top;
!         w->width = p->width;
!         w->height = p->height;
          w->hscroll = p->hscroll;
          w->min_hscroll = p->min_hscroll;
          w->display_table = p->display_table;
!         w->orig_top = p->orig_top;
!         w->orig_height = p->orig_height;
          XSETFASTINT (w->last_modified, 0);
          XSETFASTINT (w->last_overlay_modified, 0);
  
--- 5216,5239 ----
  
          /* If we squirreled away the buffer in the window's height,
             restore it now.  */
!         if (BUFFERP (w->total_lines))
!           w->buffer = w->total_lines;
!         w->left_col = p->left_col;
!         w->top_line = p->top_line;
!         w->total_cols = p->total_cols;
!         w->total_lines = p->total_lines;
          w->hscroll = p->hscroll;
          w->min_hscroll = p->min_hscroll;
          w->display_table = p->display_table;
!         w->orig_top_line = p->orig_top_line;
!         w->orig_total_lines = p->orig_total_lines;
!         w->left_margin_cols = p->left_margin_cols;
!         w->right_margin_cols = p->right_margin_cols;
!         w->left_fringe_width = p->left_fringe_width;
!         w->right_fringe_width = p->right_fringe_width;
!         w->fringes_outside_margins = p->fringes_outside_margins;
!         w->scroll_bar_width = p->scroll_bar_width;
!         w->vertical_scroll_bar_type = p->vertical_scroll_bar_type;
          XSETFASTINT (w->last_modified, 0);
          XSETFASTINT (w->last_overlay_modified, 0);
  
***************
*** 5235,5243 ****
  #endif
  
        /* Set the screen height to the value it had before this function.  */
!       if (previous_frame_height != FRAME_HEIGHT (f)
!         || previous_frame_width != FRAME_WIDTH (f))
!       change_frame_size (f, previous_frame_height, previous_frame_width,
                           0, 0, 0);
  #if defined (HAVE_WINDOW_SYSTEM) || defined (MSDOS)
        if (previous_frame_menu_bar_lines != FRAME_MENU_BAR_LINES (f))
--- 5315,5323 ----
  #endif
  
        /* Set the screen height to the value it had before this function.  */
!       if (previous_frame_lines != FRAME_LINES (f)
!         || previous_frame_cols != FRAME_COLS (f))
!       change_frame_size (f, previous_frame_lines, previous_frame_cols,
                           0, 0, 0);
  #if defined (HAVE_WINDOW_SYSTEM) || defined (MSDOS)
        if (previous_frame_menu_bar_lines != FRAME_MENU_BAR_LINES (f))
***************
*** 5308,5314 ****
    if (!NILP (w->hchild))
      delete_all_subwindows (XWINDOW (w->hchild));
  
!   w->height = w->buffer;       /* See Fset_window_configuration for excuse.  
*/
  
    if (!NILP (w->buffer))
      unshow_buffer (w);
--- 5388,5394 ----
    if (!NILP (w->hchild))
      delete_all_subwindows (XWINDOW (w->hchild));
  
!   w->total_lines = w->buffer;       /* See Fset_window_configuration for 
excuse.  */
  
    if (!NILP (w->buffer))
      unshow_buffer (w);
***************
*** 5406,5420 ****
        XSETFASTINT (w->temslot, i++);
        p->window = window;
        p->buffer = w->buffer;
!       p->left = w->left;
!       p->top = w->top;
!       p->width = w->width;
!       p->height = w->height;
        p->hscroll = w->hscroll;
        p->min_hscroll = w->min_hscroll;
        p->display_table = w->display_table;
!       p->orig_top = w->orig_top;
!       p->orig_height = w->orig_height;
        if (!NILP (w->buffer))
        {
          /* Save w's value of point in the window configuration.
--- 5486,5507 ----
        XSETFASTINT (w->temslot, i++);
        p->window = window;
        p->buffer = w->buffer;
!       p->left_col = w->left_col;
!       p->top_line = w->top_line;
!       p->total_cols = w->total_cols;
!       p->total_lines = w->total_lines;
        p->hscroll = w->hscroll;
        p->min_hscroll = w->min_hscroll;
        p->display_table = w->display_table;
!       p->orig_top_line = w->orig_top_line;
!       p->orig_total_lines = w->orig_total_lines;
!       p->left_margin_cols = w->left_margin_cols;
!       p->right_margin_cols = w->right_margin_cols;
!       p->left_fringe_width = w->left_fringe_width;
!       p->right_fringe_width = w->right_fringe_width;
!       p->fringes_outside_margins = w->fringes_outside_margins;
!       p->scroll_bar_width = w->scroll_bar_width;
!       p->vertical_scroll_bar_type = w->vertical_scroll_bar_type;
        if (!NILP (w->buffer))
        {
          /* Save w's value of point in the window configuration.
***************
*** 5492,5499 ****
    vec = allocate_other_vector (VECSIZE (struct save_window_data));
    data = (struct save_window_data *)vec;
  
!   XSETFASTINT (data->frame_width, FRAME_WIDTH (f));
!   XSETFASTINT (data->frame_height, FRAME_HEIGHT (f));
    XSETFASTINT (data->frame_menu_bar_lines, FRAME_MENU_BAR_LINES (f));
    XSETFASTINT (data->frame_tool_bar_lines, FRAME_TOOL_BAR_LINES (f));
    data->selected_frame = selected_frame;
--- 5579,5586 ----
    vec = allocate_other_vector (VECSIZE (struct save_window_data));
    data = (struct save_window_data *)vec;
  
!   XSETFASTINT (data->frame_cols, FRAME_COLS (f));
!   XSETFASTINT (data->frame_lines, FRAME_LINES (f));
    XSETFASTINT (data->frame_menu_bar_lines, FRAME_MENU_BAR_LINES (f));
    XSETFASTINT (data->frame_tool_bar_lines, FRAME_TOOL_BAR_LINES (f));
    data->selected_frame = selected_frame;
***************
*** 5554,5559 ****
--- 5641,5650 ----
  {
    struct window *w = decode_window (window);
  
+   /* TODO: It doesn't make sense to use FLOATs here, since
+      the rest of the code assumes they are integers.
+      So don't allow floats!  ++KFS */
+ 
    if (!NILP (left))
      CHECK_NUMBER_OR_FLOAT (left);
    if (!NILP (right))
***************
*** 5573,5583 ****
    if (INTEGERP (right) && XFASTINT (right) == 0)
      right = Qnil;
  
!   w->left_margin_width = left;
!   w->right_margin_width = right;
  
-   ++windows_or_buffers_changed;
-   adjust_glyphs (XFRAME (WINDOW_FRAME (w)));
    return Qnil;
  }
  
--- 5664,5681 ----
    if (INTEGERP (right) && XFASTINT (right) == 0)
      right = Qnil;
  
!   if (!EQ (w->left_margin_cols, left)
!       || !EQ (w->right_margin_cols, right))
!     {
!       w->left_margin_cols = left;
!       w->right_margin_cols = right;
! 
!       adjust_window_margins (w);
! 
!       ++windows_or_buffers_changed;
!       adjust_glyphs (XFRAME (WINDOW_FRAME (w)));
!     }
  
    return Qnil;
  }
  
***************
*** 5593,5599 ****
       Lisp_Object window;
  {
    struct window *w = decode_window (window);
!   return Fcons (w->left_margin_width, w->right_margin_width);
  }
  
  
--- 5691,5824 ----
       Lisp_Object window;
  {
    struct window *w = decode_window (window);
!   return Fcons (w->left_margin_cols, w->right_margin_cols);
! }
! 
! 
! 
! /***********************************************************************
!                           Fringes
!  ***********************************************************************/
! 
! DEFUN ("set-window-fringes", Fset_window_fringes, Sset_window_fringes,
!        2, 4, 0,
!        doc: /* Set width of fringes of window WINDOW.
! 
! If window is nil, set fringes of the currently selected window.
! Second parameter LEFT-WIDTH specifies the number of pixels to reserve
! for the left fringe.  Third parameter RIGHT-WIDTH does the same for
! the right fringe.  Fourth parameter OUTSIDE-MARGINS non-nil specifies 
! that fringes are drawn outside of the display margins; by default, fringes
! are drawn between display marginal areas and the text area.
! A nil width parameter means to use the frame's corresponding fringe width.  
*/)
!      (window, left, right, outside_margins)
!      Lisp_Object window, left, right, outside_margins;
! {
!   struct window *w = decode_window (window);
! 
!   if (!NILP (left))
!     CHECK_NUMBER (left);
!   if (!NILP (right))
!     CHECK_NUMBER (right);
! 
!   if (!EQ (w->left_fringe_width, left)
!       || !EQ (w->right_fringe_width, right)
!       || !EQ (w->fringes_outside_margins, outside_margins))
!     {
!       w->left_fringe_width = left;
!       w->right_fringe_width = right;
!       w->fringes_outside_margins = outside_margins;
! 
!       adjust_window_margins (w);
! 
!       clear_glyph_matrix (w->current_matrix);
!       w->window_end_valid = Qnil;
! 
!       ++windows_or_buffers_changed;
!       adjust_glyphs (XFRAME (WINDOW_FRAME (w)));
!     }
! 
!   return Qnil;
! }
! 
! 
! DEFUN ("window-fringes", Fwindow_fringes, Swindow_fringes,
!        0, 1, 0,
!        doc: /* Get width of fringes of window WINDOW.
! If WINDOW is omitted or nil, use the currently selected window.
! Value is a list of the form (LEFT-WIDTH RIGHT-WIDTH OUTSIDE-MARGINS).
! If a window specific fringe width is not set, its width will be returned
! as nil.  */)
!      (window)
!      Lisp_Object window;
! {
!   struct window *w = decode_window (window);
!   return Fcons (make_number (WINDOW_LEFT_FRINGE_WIDTH (w)),
!               Fcons (make_number (WINDOW_RIGHT_FRINGE_WIDTH (w)),
!                      Fcons ((WINDOW_HAS_FRINGES_OUTSIDE_MARGINS (w) ?
!                              Qt : Qnil), Qnil)));
! }
! 
! 
! 
! /***********************************************************************
!                           Scroll bars
!  ***********************************************************************/
! 
! DEFUN ("set-window-scroll-bars", Fset_window_scroll_bars, 
Sset_window_scroll_bars,
!        2, 4, 0,
!        doc: /* Set width and type of scroll bars of window WINDOW.
! If window is nil, set scroll bars of the currently selected window.
! Second parameter WIDTH specifies the pixel width for the scroll bar;
! this is automatically adjusted to a multiple of the frame column width.
! Third parameter VERTICAL-TYPE specifies the type of the vertical scroll
! bar: left, right, or nil.
! A width of nil and type of t means to use the frame's corresponding value.  
*/)
!      (window, width, vertical_type, horisontal_type)
!      Lisp_Object window, width, vertical_type, horisontal_type;
! {
!   struct window *w = decode_window (window);
! 
!   if (!NILP (width))
!     CHECK_NUMBER (width);
! 
!   if (XINT (width) == 0)
!     vertical_type = Qnil;
! 
!   if (!EQ (w->scroll_bar_width, width)
!       || !EQ (w->vertical_scroll_bar_type, vertical_type))
!     {
!       w->scroll_bar_width = width;
!       w->vertical_scroll_bar_type = vertical_type;
! 
!       adjust_window_margins (w);
! 
!       clear_glyph_matrix (w->current_matrix);
!       w->window_end_valid = Qnil;
! 
!       ++windows_or_buffers_changed;
!       adjust_glyphs (XFRAME (WINDOW_FRAME (w)));
!     }
! 
!   return Qnil;
! }
! 
! 
! DEFUN ("window-scroll-bars", Fwindow_scroll_bars, Swindow_scroll_bars,
!        0, 1, 0,
!        doc: /* Get width and type of scroll bars of window WINDOW.
! If WINDOW is omitted or nil, use the currently selected window.
! Value is a list of the form (WIDTH COLS VERTICAL-TYPE HORISONTAL-TYPE).  */)
!      (window)
!      Lisp_Object window;
! {
!   struct window *w = decode_window (window);
!   return Fcons (make_number ((WINDOW_CONFIG_SCROLL_BAR_WIDTH (w)
!                             ? WINDOW_CONFIG_SCROLL_BAR_WIDTH (w)
!                             : WINDOW_SCROLL_BAR_AREA_WIDTH (w))),
!               Fcons (make_number (WINDOW_SCROLL_BAR_COLS (w)),
!                      Fcons (w->vertical_scroll_bar_type,
!                             Fcons (Qnil, Qnil))));
  }
  
  
***************
*** 5621,5627 ****
    f = XFRAME (w->frame);
  
    if (FRAME_WINDOW_P (f))
!     result = CANON_Y_FROM_PIXEL_Y (f, -w->vscroll);
    else
      result = make_number (0);
    return result;
--- 5846,5852 ----
    f = XFRAME (w->frame);
  
    if (FRAME_WINDOW_P (f))
!     result = FRAME_CANON_Y_FROM_PIXEL_Y (f, -w->vscroll);
    else
      result = make_number (0);
    return result;
***************
*** 5652,5658 ****
      {
        int old_dy = w->vscroll;
  
!       w->vscroll = - CANON_Y_UNIT (f) * XFLOATINT (vscroll);
        w->vscroll = min (w->vscroll, 0);
  
        /* Adjust glyph matrix of the frame if the virtual display
--- 5877,5883 ----
      {
        int old_dy = w->vscroll;
  
!       w->vscroll = - FRAME_LINE_HEIGHT (f) * XFLOATINT (vscroll);
        w->vscroll = min (w->vscroll, 0);
  
        /* Adjust glyph matrix of the frame if the virtual display
***************
*** 5771,5779 ****
    sw1 = XVECTOR (d1->saved_windows);
    sw2 = XVECTOR (d2->saved_windows);
  
!   if (! EQ (d1->frame_width, d2->frame_width))
      return 0;
!   if (! EQ (d1->frame_height, d2->frame_height))
      return 0;
    if (! EQ (d1->frame_menu_bar_lines, d2->frame_menu_bar_lines))
      return 0;
--- 5996,6004 ----
    sw1 = XVECTOR (d1->saved_windows);
    sw2 = XVECTOR (d2->saved_windows);
  
!   if (! EQ (d1->frame_cols, d2->frame_cols))
      return 0;
!   if (! EQ (d1->frame_lines, d2->frame_lines))
      return 0;
    if (! EQ (d1->frame_menu_bar_lines, d2->frame_menu_bar_lines))
      return 0;
***************
*** 5825,5837 ****
        /* Verify that the corresponding windows do match.  */
        if (! EQ (p1->buffer, p2->buffer))
        return 0;
!       if (! EQ (p1->left, p2->left))
        return 0;
!       if (! EQ (p1->top, p2->top))
        return 0;
!       if (! EQ (p1->width, p2->width))
        return 0;
!       if (! EQ (p1->height, p2->height))
        return 0;
        if (! EQ (p1->display_table, p2->display_table))
        return 0;
--- 6050,6062 ----
        /* Verify that the corresponding windows do match.  */
        if (! EQ (p1->buffer, p2->buffer))
        return 0;
!       if (! EQ (p1->left_col, p2->left_col))
        return 0;
!       if (! EQ (p1->top_line, p2->top_line))
        return 0;
!       if (! EQ (p1->total_cols, p2->total_cols))
        return 0;
!       if (! EQ (p1->total_lines, p2->total_lines))
        return 0;
        if (! EQ (p1->display_table, p2->display_table))
        return 0;
***************
*** 5854,5859 ****
--- 6079,6098 ----
          if (NILP (Fequal (p1->mark, p2->mark)))
            return 0;
        }
+       if (! EQ (p1->left_margin_cols, p2->left_margin_cols))
+       return 0;
+       if (! EQ (p1->right_margin_cols, p2->right_margin_cols))
+       return 0;
+       if (! EQ (p1->left_fringe_width, p2->left_fringe_width))
+       return 0;
+       if (! EQ (p1->right_fringe_width, p2->right_fringe_width))
+       return 0;
+       if (! EQ (p1->fringes_outside_margins, p2->fringes_outside_margins))
+       return 0;
+       if (! EQ (p1->scroll_bar_width, p2->scroll_bar_width))
+       return 0;
+       if (! EQ (p1->vertical_scroll_bar_type, p2->vertical_scroll_bar_type))
+       return 0;
      }
  
    return 1;
***************
*** 6152,6157 ****
--- 6391,6400 ----
    defsubr (&Ssave_window_excursion);
    defsubr (&Sset_window_margins);
    defsubr (&Swindow_margins);
+   defsubr (&Sset_window_fringes);
+   defsubr (&Swindow_fringes);
+   defsubr (&Sset_window_scroll_bars);
+   defsubr (&Swindow_scroll_bars);
    defsubr (&Swindow_vscroll);
    defsubr (&Sset_window_vscroll);
    defsubr (&Scompare_window_configurations);




reply via email to

[Prev in Thread] Current Thread [Next in Thread]