more frame-sizing cleanups -------------------- ChangeLog entries follow: -------------------- man/ChangeLog addition: 2010-02-16 Ben Wing <ben@xemacs.org> * internals/internals.texi (Top): * internals/internals.texi (Modules for the Basic Displayable Lisp Objects): * internals/internals.texi (Creating a Window-System Type): * internals/internals.texi (Window and Frame Geometry): * internals/internals.texi (Intro to Window and Frame Geometry): * internals/internals.texi (The Frame): * internals/internals.texi (The Non-Client Area): * internals/internals.texi (The Client Area): * internals/internals.texi (The Paned Area): * internals/internals.texi (Text Areas): * internals/internals.texi (The Displayable Area): * internals/internals.texi (Which Functions Use Which?): * internals/internals.texi (The Redisplay Mechanism): Integrate the long comment in frame.c into the internals manual. src/ChangeLog addition: 2010-02-16 Ben Wing <ben@xemacs.org> * frame-impl.h: * frame-impl.h (FRAME_INTERNAL_BORDER_WIDTH): * frame-impl.h (FRAME_REAL_TOOLBAR_BOUNDS): * frame-impl.h (FRAME_REAL_TOP_TOOLBAR_BOUNDS): * frame-impl.h (FRAME_BOTTOM_BORDER_START): * frame-impl.h (FRAME_LEFT_BORDER_START): * frame-impl.h (FRAME_RIGHT_BORDER_START): * frame.c (frame_conversion_internal_1): * frame.c (change_frame_size_1): * redisplay-output.c (clear_left_border): * redisplay-output.c (clear_right_border): * redisplay-output.c (redisplay_clear_top_of_window): * redisplay-output.c (redisplay_clear_to_window_end): * redisplay-output.c (redisplay_clear_bottom_of_window): Rename FRAME_BORDER_* to FRAME_INTERNAL_BORDER_*. Add general FRAME_INTERNAL_BORDER_SIZE(). Add FRAME_REAL_TOOLBAR_BOUNDS() to encompass the entire size of the toolbar including its border. Add specific top/left/bottom/right versions of this macro. Rewrite FRAME_*_BORDER_START and FRAME_*_BORDER_END to take into use FRAME_REAL_*_TOOLBAR_BOUNDS(). Add some comments about existing problems in frame sizing and how they might be fixed. Simplify change_frame_size_1() using the macros just created.

/* XPM */
static char * undo[] = {
"33 33 5 1",
"X	c Gray75 s backgroundToolBarColor",
"o	c Gray60",
"O	c Gray60",
"+	c Gray90",
"@	c white",
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXXooXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXooOOooooXXXXXX",
"XXXXXXXXXXXXXXXXXooOOOOOOOOooXXXX",
"XXXXXXXXXXXXXXXooOOOOOOOOOooXXXXX",
"XXXXXXXXXXXXXoo++ooooOOOooOoXXXXX",
"XXXXXXXXXXXoo++++++++oooOOoXXXXXX",
"XXXXXXXXXoo+++++++++oooOOOoXXXXXX",
"XXXXXXXoo+++++++++oo++oOOoXXXXXXX",
"XXXXXXoo++++++++oo+++oOooXXXXXXXX",
"XXXXXo++oooo++oo+++++ooXXXXXXXXXX",
"XXXXXo++++++oo+++++ooXXXXXXXXXXXX",
"XXXXo+++++++o++++ooXXXXXXXXXXXXXX",
"XXXXo+++++++o++ooXXXXXXXXXXXXXXXX",
"XXXoo++++++o+ooXXXXXXXXXXXXXXXXXX",
"XXX@@oooo++ooXXXXXXXXXXXXXXXXXXXX",
"XXX@@@@@@ooXXXXXXXXXXXXXXXXXXXXXX",
"XXX@@@@@@XXXXXXXXXXXXXXXXXXXXXXXX",
"XXX@@@@XXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXX@@@XXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXX@XXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXoXXXXXXXXXXXX",
"XXXXXXoXXXoXXXXXXXXXoXXXXXXXXXXXX",
"XXXXXXoXXXoXoooXXXoooXXooXXXXXXXX",
"XXXXXXoXXXoXoXXoXoXXoXoXXoXXXXXXX",
"XXXXXXoXXXoXoXXoXoXXoXoXXoXXXXXXX",
"XXXXXXoXXXoXoXXoXoXXoXoXXoXXXXXXX",
"XXXXXXXoooXXoXXoXXoooXXooXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"};