Added an "XXX" earmark for post-release contemplation. (The point is

[freeglut] / src / freeglut_main.c

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 )
     {
+        /*
+         * 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;