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
This commit is contained in:
parent
4c4112fe92
commit
1d2ade2f1c
@ -798,6 +798,18 @@ void FGAPIENTRY glutReshapeWindow( int width, int height )
|
|||||||
XResizeWindow( fgDisplay.Display, fgStructure.Window->Window.Handle,
|
XResizeWindow( fgDisplay.Display, fgStructure.Window->Window.Handle,
|
||||||
width, height );
|
width, height );
|
||||||
XFlush( fgDisplay.Display ); /* XXX Shouldn't need this */
|
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
|
#elif TARGET_HOST_WIN32
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user