APPROVE 21.5
Tested per Ron's message. This one is theoretically right, but so
unobviously correct that I think the only way to prove it is to commit
it.
Ron, do you have any reservations about having the log in your name
with your company affiliation?
Ron Isaacson writes:
Ron Isaacson wrote:
>
> > I gotta get some sleep, but I think that moving the reset of
> > size_changed_pending to the beginning of change_frame_size_1 might help.
>
> I agree. change_frame_size_1 is what actually effects the pending
> change. I'll give it a try...
Sure enough, that did it. Patch attached. And the screen(1) thing
turned out to be a total red herring... it so happens that all of my
console-mode usage is within screen, but the same problem happens
without it. :-)
Index: src/ChangeLog
===================================================================
RCS file: /pack/xemacscvs/XEmacs/xemacs/src/ChangeLog,v
retrieving revision 1.1110
diff -u -U0 -r1.1110 ChangeLog
--- src/ChangeLog 29 Nov 2007 13:37:54 -0000 1.1110
+++ src/ChangeLog 4 Dec 2007 05:49:49 -0000
@@ -0,0 +1,6 @@
+2007-12-02 Ron Isaacson <ron.isaacson(a)morganstanley.com>
+
+ * frame.c (change_frame_size):
+ (change_frame_size_1):
+ Must reset pending flag per-frame, so do it in the latter.
+
Index: src/frame.c
===================================================================
RCS file: /pack/xemacscvs/XEmacs/xemacs/src/frame.c,v
retrieving revision 1.76
diff -u -r1.76 frame.c
--- src/frame.c 3 Oct 2007 10:06:41 -0000 1.76
+++ src/frame.c 4 Dec 2007 05:49:54 -0000
@@ -3273,6 +3273,7 @@
default_face_height_and_width_1 (frame, &font_height, &font_width);
/* This size-change overrides any pending one for this frame. */
+ f->size_change_pending = 0;
FRAME_NEW_HEIGHT (f) = 0;
FRAME_NEW_WIDTH (f) = 0;
@@ -3450,7 +3451,6 @@
return;
}
- f->size_change_pending = 0;
/* For TTY frames, it's like one, like all ...
Can't have two TTY frames of different sizes on the same device. */
if (FRAME_TTY_P (f))
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-patches