From 5efc7f1ec295474817dd1ed99c04180f8f19c8d9 Mon Sep 17 00:00:00 2001 From: Richard Rauch Date: Tue, 2 Dec 2003 21:14:32 +0000 Subject: [PATCH] Added an "XXX" earmark for post-release contemplation. (The point is definitely in need of consideration, I believe, but is not immediately a major bug, so I'd rather not mess with it at the risk of postponing a release "soon".) git-svn-id: svn+ssh://svn.code.sf.net/p/freeglut/code/trunk/freeglut/freeglut@376 7f0cb862-5218-0410-a997-914c9d46530a --- src/freeglut_main.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/freeglut_main.c b/src/freeglut_main.c index 4936528..144453d 100644 --- a/src/freeglut_main.c +++ b/src/freeglut_main.c @@ -191,6 +191,21 @@ static void fghcbDisplayWindow( SFG_Window *window, if( window->State.Redisplay && window->State.Visible ) { + /* + * XXX Resizing should *not* depend upon whether there + * XXX is a pending redisplay flag, as far as I can tell. + * XXX + * XXX Note, too, that the {NeedToResize} flag is a little + * XXX fuzzy in its meaning, since for WIN32, this also + * XXX means "we need to tell the application that the window has + * XXX changed size", while in X11, it only means "we need + * XXX to ask the window system to resize the window. + * XXX Splitting the flag's meaning might be desirable, but + * XXX that could complicate the code more. (On X11, the + * XXX user callback is called as soon as the event is + * XXX discovered, but resizing the window is postponed + * XXX until after other events.) + */ if( window->State.NeedToResize ) { SFG_Window *current_window = fgStructure.Window; -- 1.7.10.4