Patch for SDL_VIDEO_CENTERED & OpenGL bug

I reported on this bug in previous emails, but I’ll include the basics
again:

Using SDL_VIDEO_CENTERED in Linux OpenGL mode issue an unwanted
ConfigureNotify event after the SDL screen has already been resized.
When going from a smaller to a larger screen, this event causes the mouse
tracking to be clamped at the smaller screen size, even though that
screen no longer exists.

The fix is to not issue a ConfigureNotify when the window is moved because
of the SDL_VIDEO_CENTERED environment variable. The included patch fixes
this bug. It seems the hints must be set before the window is moved.

Hopefully this can be incorporated into the next release. If there isn’t
going to be a next release, I’m not sure how to work around it my app.

Thanks,
SA
-------------- next part --------------
A non-text attachment was scrubbed…
Name: sdl_video_centered.diff
Type: text/x-patch
Size: 708 bytes
Desc: not available
URL: http://lists.libsdl.org/pipermail/sdl-libsdl.org/attachments/20100801/224c405f/attachment.bin

Thanks, your patch has been added!
http://hg.libsdl.org/SDL/rev/67d3be4ec9f2On Sun, Aug 1, 2010 at 2:53 PM, Stephen Anthony wrote:

I reported on this bug in previous emails, but I’ll include the basics
again:

Using SDL_VIDEO_CENTERED in Linux OpenGL mode issue an unwanted
ConfigureNotify event after the SDL screen has already been resized.
When going from a smaller to a larger screen, this event causes the mouse
tracking to be clamped at the smaller screen size, even though that
screen no longer exists.

The fix is to not issue a ConfigureNotify when the window is moved because
of the SDL_VIDEO_CENTERED environment variable. ?The included patch fixes
this bug. ?It seems the hints must be set before the window is moved.

Hopefully this can be incorporated into the next release. ?If there isn’t
going to be a next release, I’m not sure how to work around it my app.

Thanks,
SA


SDL mailing list
SDL at lists.libsdl.org
http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org


? ? -Sam Lantinga, Founder and CEO, Galaxy Gameworks

OK, it’s good to see the patch has been added, but I’m not sure of the
relevance anymore, since there won’t be a new release of SDL 1.2 (or am
I mistaken in that assumption?).

Thanks,
Steve A.On 11-02-24 10:13 PM, Sam Lantinga wrote:

Thanks, your patch has been added!
http://hg.libsdl.org/SDL/rev/67d3be4ec9f2