When changing frame sizes round before applying new sizes. (Bug#9723)

* dispnew.c (change_frame_size_1): Calculate new_frame_total_cols
after rounding frame sizes.  (Bug#9723)

diff --git a/src/ChangeLog b/src/ChangeLog
index 40ad7a9f507d7425bc9144248386a5b7ee46a268..95f6201b201d12754582459a2ea88addb95000bf 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,8 @@
+2012-02-04  Martin Rudalics  <rudalics@gmx.at>
+
+       * dispnew.c (change_frame_size_1): Calculate new_frame_total_cols
+       after rounding frame sizes.  (Bug#9723)
+
 2012-02-04  Eli Zaretskii  <eliz@gnu.org>
 
        * keyboard.c (adjust_point_for_property): Don't position point

diff --git a/src/dispnew.c b/src/dispnew.c
index 9e57bbb28bf8dba49b228560998b0c4d95cc3792..d302e717ec27414269a3476a5df4a5b456a5e658 100644 (file)
--- a/src/dispnew.c
+++ b/src/dispnew.c
@@ -5761,19 +5761,22 @@ change_frame_size_1 (register struct frame *f, int newheight, int newwidth, int
   if (newwidth == 0)
     newwidth  = FRAME_COLS  (f);
 
-  /* Compute width of windows in F.
-     This is the width of the frame without vertical scroll bars.  */
-  new_frame_total_cols = FRAME_TOTAL_COLS_ARG (f, newwidth);
-
+  /* Compute width of windows in F.  */
   /* Round up to the smallest acceptable size.  */
   check_frame_size (f, &newheight, &newwidth);
 
+  /* This is the width of the frame with vertical scroll bars and fringe
+     columns.  Do this after rounding - see discussion of bug#9723.  */
+  new_frame_total_cols = FRAME_TOTAL_COLS_ARG (f, newwidth);
+
   /* If we're not changing the frame size, quit now.  */
-  /* Frame width may be unchanged but the text portion may change, for example,
-     fullscreen and remove/add scroll bar.  */
+  /* Frame width may be unchanged but the text portion may change, for
+     example, fullscreen and remove/add scroll bar.  */
   if (newheight == FRAME_LINES (f)
-      && newwidth == FRAME_COLS  (f) // text portion unchanged
-      && new_frame_total_cols == FRAME_TOTAL_COLS (f)) // frame width unchanged
+      /* Text portion unchanged?  */
+      && newwidth == FRAME_COLS  (f)
+      /* Frame width unchanged?  */
+      && new_frame_total_cols == FRAME_TOTAL_COLS (f))
     return;
 
   BLOCK_INPUT;