From: Richard Rauch <rkr@olib.org>
Date: Tue, 2 Dec 2003 21:14:32 +0000 (+0000)
Subject: Added an "XXX" earmark for post-release contemplation.  (The point is
X-Git-Url: http://git.mutantstargoat.com/user/nuclear/?a=commitdiff_plain;h=5efc7f1ec295474817dd1ed99c04180f8f19c8d9;hp=7b835f24f622aa3232182ab8b35485c60b9b7e76;p=freeglut

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
---

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;