# HG changeset patch
# User Sam Lantinga <slouken@libsdl.org>
# Date 1084730568 0
# Node ID eac8c69b57069e8c96ce7b52659bcb1bb97e1d4c
# Parent  07def9d033158d43e693e71e59bd75d86290b273
ate: Fri, 09 Apr 2004 20:07:10 +0200
From: Hibernatus
Subject: [SDL] BUG: changing video mode in opengl + windows

In WIN_GL_SetupWindow (SDL12/src/video/wincommon/SDL_wingl.c) you call
WIN_GL_ResetWindow when SetPixelFormat fails.
WIN_GL_ResetWindow cannot properly work in its current state, because there
are several things to update once you've created a new window. At least
DirectInput doesn't work anymore here (which is somewhat annoying for the
player :).
I think you can simply remove that function. Many games don't try to
recover from that exception, that's why the newest ATI drivers emulate 16
bpp display in 32 bpp mode so the error never occurs (but the games may run
much slower).

Having SDL_SetVideoMode return NULL would be better than actually setting
the video mode and losing focus, directinput and maybe directsound too. It
also made my previous drivers crash (when switching from 32 bpp windowed to
16 bpp fullscreen).

diff -r 07def9d03315 -r eac8c69b5706 src/video/wincommon/SDL_wingl.c
--- a/src/video/wincommon/SDL_wingl.c	Sun May 16 17:40:32 2004 +0000
+++ b/src/video/wincommon/SDL_wingl.c	Sun May 16 18:02:48 2004 +0000
@@ -50,6 +50,7 @@
 	if ( SDL_windowid ) {
 		can_reset = 0;
 	}
+#if 0 /* This doesn't work with DirectX code (see CVS comments) */
 #ifndef _WIN32_WCE /* FIXME WinCE needs the UNICODE version of CreateWindow() */
 	if ( can_reset ) {
 		/* Save the existing window attributes */
@@ -72,6 +73,7 @@
 		}
 	} else
 #endif /* !_WIN32_WCE */
+#endif
 	{
 		SDL_SetError("Unable to reset window for OpenGL context");
 		status = -1;