Sun, 21 Jul 2013 12:21:22 -0700Fixed cursor leak when quitting the mouse subsystem
Sam Lantinga [Sun, 21 Jul 2013 12:21:22 -0700] rev 7493
Fixed cursor leak when quitting the mouse subsystem

Sun, 21 Jul 2013 11:52:16 -0700Fixed bug 1976 - SDL2.m4 shares same variables as SDL.m4 so they cache values
Sam Lantinga [Sun, 21 Jul 2013 11:52:16 -0700] rev 7492
Fixed bug 1976 - SDL2.m4 shares same variables as SDL.m4 so they cache values

Edgar Simo

So basically the issue is that:

AC_PATH_PROG(SDL_CONFIG, sdl2-config, no, [$PATH])

in sdl2.m4 shares the same variable (SDL_CONFIG) as sdl.m4:

AC_PATH_PROG(SDL_CONFIG, sdl-config, no, [$PATH])

What happens is if you run the AM_PATH_SDL2 and then AM_PATH_SDL to say fall back on SDL if SDL2 isn't found, the AM_PATH_PROG command gets cached (ac_cv_path_SDL_CONFIG) so that unless you unset this cache variable, the result for AM_PATH_SDL will be EXACTLY the same as for AM_PATH_SDL2.

To fix I would recommend renaming SDL_CONFIG to SDL2_CONFIG in sdl2.m4, replacing all instances so it won't cache to the same variable.

Sun, 21 Jul 2013 11:09:53 -0700Fixed bug 1979 - X11_SetGammaRamp doesn't check the return value of XStoreColors SDL-1.2
Sam Lantinga [Sun, 21 Jul 2013 11:09:53 -0700] rev 7491
Fixed bug 1979 - X11_SetGammaRamp doesn't check the return value of XStoreColors

Sven Hesse

Here on my laptop with an ATI Mobility Radeon 9600 PRO Turbo (RV350), the free radeon drivers and X.org 1.12.4, SDL_SetGamma() fails because it tries to use SDL_SetGammaRamp() if available. SDL_SetGammaRamp() uses XStoreColors(), and that call always fails with BadRequest (for whatever reason, I don't know). The SDL_Visual class check goes through, though.

Now, XStoreColors failing is maybe another bug in SDL, or in Xorg, or in the radeon drivers or anywhere else, but I think SDL should still check that the XStoreColors() doesn't error as well.

Sun, 21 Jul 2013 00:57:31 -0400Add support for the _NET_WM_BYPASS_COMPOSITOR hint
Jasper St. Pierre [Sun, 21 Jul 2013 00:57:31 -0400] rev 7490
Add support for the _NET_WM_BYPASS_COMPOSITOR hint

This hints to window managers to unredirect the window to improve
performance.

Sun, 21 Jul 2013 00:07:43 -0400Cursor mask was uninitialized data, causing problems.
Ryan C. Gordon [Sun, 21 Jul 2013 00:07:43 -0400] rev 7489
Cursor mask was uninitialized data, causing problems.

Fixes Bugzilla #1461.

Sat, 20 Jul 2013 21:22:37 -0400Include SDL_assert.h to fix broken build.
Ryan C. Gordon [Sat, 20 Jul 2013 21:22:37 -0400] rev 7488
Include SDL_assert.h to fix broken build.

Sat, 20 Jul 2013 21:19:20 -0400Don't allocate memory if we're just going to fail when checking parameters.
Ryan C. Gordon [Sat, 20 Jul 2013 21:19:20 -0400] rev 7487
Don't allocate memory if we're just going to fail when checking parameters.

Sat, 20 Jul 2013 21:10:05 -0400Don't try to clear errors in GL_ActivateRenderer() before we MakeCurrent.
Ryan C. Gordon [Sat, 20 Jul 2013 21:10:05 -0400] rev 7486
Don't try to clear errors in GL_ActivateRenderer() before we MakeCurrent.

Otherwise, if we destroyed a different renderer, next time this one draws,
it'll clear errors forever (GL_INVALID_OPERATION for having no current
context, at least on Windows), hanging up the program in an infinite loop.

Fixes Bugzilla #1775.

Sat, 20 Jul 2013 20:12:36 -0400Fixed another compiler warning.
Ryan C. Gordon [Sat, 20 Jul 2013 20:12:36 -0400] rev 7485
Fixed another compiler warning.

This one is compliments of the FreeBSD buildbot finally having the
X11 libraries installed. :)

Sat, 20 Jul 2013 19:59:31 -0400One more compiler warning.
Ryan C. Gordon [Sat, 20 Jul 2013 19:59:31 -0400] rev 7484
One more compiler warning.