Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fixed bug #197
On servers with the composite extension enabled, visuals with 32-bit depth
have an alpha mask.  This is pretty neat, but SDL needs a bit more work to
handle these properly, so for now, we'll just use 24 bit depth visuals.
  • Loading branch information
slouken committed Apr 17, 2006
1 parent a4de08f commit 7a30946
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 1 deletion.
4 changes: 4 additions & 0 deletions src/video/x11/SDL_x11modes.c
Expand Up @@ -620,7 +620,11 @@ int X11_GetVideoModes(_THIS)
#endif /* SDL_VIDEO_DRIVER_X11_XME */

{
/* It's interesting to note that if we allow 32 bit depths,
we get a visual with an alpha mask on composite servers.
static int depth_list[] = { 32, 24, 16, 15, 8 };
*/
static int depth_list[] = { 24, 16, 15, 8 };
int j, np;
int use_directcolor = 1;
XPixmapFormatValues *pf;
Expand Down
12 changes: 11 additions & 1 deletion src/video/x11/SDL_x11video.c
Expand Up @@ -557,6 +557,9 @@ static int X11_VideoInit(_THIS, SDL_PixelFormat *vformat)
vformat->Gmask = SDL_Visual->green_mask;
vformat->Bmask = SDL_Visual->blue_mask;
}
if ( this->hidden->depth == 32 ) {
vformat->Amask = (0xFFFFFFFF & ~(vformat->Rmask|vformat->Gmask|vformat->Bmask));
}
X11_SaveVidModeGamma(this);

/* See if we have been passed a window to use */
Expand Down Expand Up @@ -772,6 +775,7 @@ static int X11_CreateWindow(_THIS, SDL_Surface *screen,
int i, depth;
Visual *vis;
int vis_change;
Uint32 Amask;

/* If a window is already present, destroy it and start fresh */
if ( SDL_Window ) {
Expand Down Expand Up @@ -822,9 +826,15 @@ static int X11_CreateWindow(_THIS, SDL_Surface *screen,
this->hidden->depth = depth;

/* Allocate the new pixel format for this video mode */
if ( this->hidden->depth == 32 ) {
Amask = (0xFFFFFFFF & ~(vis->red_mask|vis->green_mask|vis->blue_mask));
} else {
Amask = 0;
}
if ( ! SDL_ReallocFormat(screen, bpp,
vis->red_mask, vis->green_mask, vis->blue_mask, 0) )
vis->red_mask, vis->green_mask, vis->blue_mask, Amask) ) {
return -1;
}

/* Create the appropriate colormap */
if ( SDL_XColorMap != SDL_DisplayColormap ) {
Expand Down

0 comments on commit 7a30946

Please sign in to comment.