Commit f17f1c01 authored by Sam Lantinga's avatar Sam Lantinga

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).

--HG--
extra : convert_revision : svn%3Ac70aab31-4412-0410-b14c-859654838e24/trunk%40890
parent 2156b933
...@@ -50,6 +50,7 @@ static int WIN_GL_ResetWindow(_THIS) ...@@ -50,6 +50,7 @@ static int WIN_GL_ResetWindow(_THIS)
if ( SDL_windowid ) { if ( SDL_windowid ) {
can_reset = 0; 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() */ #ifndef _WIN32_WCE /* FIXME WinCE needs the UNICODE version of CreateWindow() */
if ( can_reset ) { if ( can_reset ) {
/* Save the existing window attributes */ /* Save the existing window attributes */
...@@ -72,6 +73,7 @@ static int WIN_GL_ResetWindow(_THIS) ...@@ -72,6 +73,7 @@ static int WIN_GL_ResetWindow(_THIS)
} }
} else } else
#endif /* !_WIN32_WCE */ #endif /* !_WIN32_WCE */
#endif
{ {
SDL_SetError("Unable to reset window for OpenGL context"); SDL_SetError("Unable to reset window for OpenGL context");
status = -1; status = -1;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment