From 1d2ade2f1c638649bfc70ba69ea298ed3527562f Mon Sep 17 00:00:00 2001 From: rkrolib Date: Fri, 14 Nov 2003 11:04:11 +0000 Subject: [PATCH] Noted one of a few cases where we do something immediately that could more efficiently be postponed, quite easily---and which GLUT promises *is* to be postponed. (It is unlikely that the difference will break any apps, but postponing should be easy...) No real change, just a comment in the code. git-svn-id: https://svn.code.sf.net/p/freeglut/code/trunk@341 7f0cb862-5218-0410-a997-914c9d46530a --- freeglut/freeglut/src/freeglut_window.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/freeglut/freeglut/src/freeglut_window.c b/freeglut/freeglut/src/freeglut_window.c index 34d3b90..61df8fd 100644 --- a/freeglut/freeglut/src/freeglut_window.c +++ b/freeglut/freeglut/src/freeglut_window.c @@ -798,6 +798,18 @@ void FGAPIENTRY glutReshapeWindow( int width, int height ) XResizeWindow( fgDisplay.Display, fgStructure.Window->Window.Handle, width, height ); XFlush( fgDisplay.Display ); /* XXX Shouldn't need this */ + /* + * XXX REALLY shouldn't be done. GLUT docs state that this + * XXX isn't even processed immediately, but rather waits + * XXX for return to the mainloop. "This allows multiple + * XXX glutReshapeWindow, glutPositionWindow, and glutFullScreen + * XXX requests to the same window to be coalesced." (This is + * XXX having some deleterious effect on a sample program of mine.) + * XXX Not only does GLUT not flush at this point, GLUT doesn't even + * XXX *do* the reshape at this point! We should probably rip this + * XXX out and do what GLUT promises. It would be more efficient, and + * XXX might be more compatible. + */ #elif TARGET_HOST_WIN32