Skip to content
Permalink
Browse files

Always do either FULL or NO acceleration for Windows OpenGL.

Don't ever leave it to the OS to decide, because older ATI drivers apparently
 break in this case, and generally apps want FULL by default anyhow.
  • Loading branch information
icculus committed Nov 6, 2011
1 parent 7e476e9 commit 7d9c66cb188c1d5f154211aa9fa50ad527ffc858
Showing with 12 additions and 6 deletions.
  1. +12 −6 src/video/wincommon/SDL_wingl.c
@@ -226,7 +226,18 @@ int WIN_GL_SetupWindow(_THIS)
*iAttr++ = this->gl_config.green_size;
*iAttr++ = WGL_BLUE_BITS_ARB;
*iAttr++ = this->gl_config.blue_size;


/* We always choose either FULL or NO accel on Windows, because of flaky
drivers. If the app didn't specify, we use FULL, because that's
probably what they wanted (and if you didn't care and got FULL, that's
a perfectly valid result in any case. */
*iAttr++ = WGL_ACCELERATION_ARB;
if (this->gl_config.accelerated) {
*iAttr++ = WGL_FULL_ACCELERATION_ARB;
} else {
*iAttr++ = WGL_NO_ACCELERATION_ARB;
}

if ( this->gl_config.alpha_size ) {
*iAttr++ = WGL_ALPHA_BITS_ARB;
*iAttr++ = this->gl_config.alpha_size;
@@ -278,11 +289,6 @@ int WIN_GL_SetupWindow(_THIS)
*iAttr++ = this->gl_config.multisamplesamples;
}

if ( this->gl_config.accelerated >= 0 ) {
*iAttr++ = WGL_ACCELERATION_ARB;
*iAttr++ = (this->gl_config.accelerated ? WGL_FULL_ACCELERATION_ARB : WGL_NO_ACCELERATION_ARB);
}

*iAttr = 0;

for ( i=0; ; ++i ) {

0 comments on commit 7d9c66c

Please sign in to comment.