Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
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 7d9c66c
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions src/video/wincommon/SDL_wingl.c
Expand Up @@ -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;
Expand Down Expand Up @@ -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 ) {
Expand Down

0 comments on commit 7d9c66c

Please sign in to comment.