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 (file)
--- 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 )
     {
     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;
         if( window->State.NeedToResize )
         {
             SFG_Window *current_window = fgStructure.Window;