pandory500/SDL-mirror
2
0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Ignore old ConfigureNotify events during X11 window resize.

Browse source 
Fixes Bugzilla #1049.

Thanks to Andrew Church for the patch!

--HG--
branch : SDL-1.2
This commit is contained in:
Ryan C. Gordon 2011-08-21 11:27:37 -04:00
parent 07422a4c00
commit 0881f2aa01
3 changed files with 23 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/video/x11/SDL_x11events.c
View file

@ -57,6 +57,12 @@ static SDLKey ODD_keymap[256];
static SDLKey MISC_keymap[256];
SDLKey X11_TranslateKeycode(Display *display, KeyCode kc);
/*
Pending resize target for ConfigureNotify (so outdated events don't
cause inappropriate resize events)
*/
int X11_PendingConfigureNotifyWidth = -1;
int X11_PendingConfigureNotifyHeight = -1;
#ifdef X_HAVE_UTF8_STRING
Uint32 Utf8ToUcs4(const Uint8 *utf8)
@ -819,6 +825,16 @@ printf("MapNotify!\n");
#ifdef DEBUG_XEVENTS
printf("ConfigureNotify! (resize: %dx%d)\n", xevent.xconfigure.width, xevent.xconfigure.height);
#endif
if ((X11_PendingConfigureNotifyWidth != -1) &&
(X11_PendingConfigureNotifyHeight != -1)) {
if ((xevent.xconfigure.width != X11_PendingConfigureNotifyWidth) &&
(xevent.xconfigure.height != X11_PendingConfigureNotifyHeight)) {
/* Event is from before the resize, so ignore. */
break;
}
X11_PendingConfigureNotifyWidth = -1;
X11_PendingConfigureNotifyHeight = -1;
}
if ( SDL_VideoSurface ) {
if ((xevent.xconfigure.width != SDL_VideoSurface->w) ||
(xevent.xconfigure.height != SDL_VideoSurface->h)) {

5
src/video/x11/SDL_x11events_c.h
View file

@ -27,3 +27,8 @@
extern void X11_InitOSKeymap(_THIS);
extern void X11_PumpEvents(_THIS);
extern void X11_SetKeyboardState(Display *display, const char *key_vec);
/* Variables to be exported */
extern int X11_PendingConfigureNotifyWidth;
extern int X11_PendingConfigureNotifyHeight;

2
src/video/x11/SDL_x11video.c
View file

@ -1161,6 +1161,8 @@ SDL_Surface *X11_SetVideoMode(_THIS, SDL_Surface *current,
current = NULL;
goto done;
}
X11_PendingConfigureNotifyWidth = width;
X11_PendingConfigureNotifyHeight = height;
} else {
if (X11_CreateWindow(this,current,width,height,bpp,flags) < 0) {
current = NULL;