Mercurial > sdl-ios-xcode

changeset 4846:4f1573996a65

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fixed a bug in X11 shaping that refused to use color-key mode.
author Eli Gottlieb <eligottlieb@gmail.com>
date Tue, 10 Aug 2010 16:10:26 -0400
parents 61cb2d20a46f
children 4201864f77e7 40b46225e3cf
files include/SDL_shape.h src/video/SDL_shape.c src/video/x11/SDL_x11shape.c
diffstat 3 files changed, 6 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/include/SDL_shape.h	Tue Aug 10 15:45:59 2010 -0400
+++ b/include/SDL_shape.h	Tue Aug 10 16:10:26 2010 -0400
@@ -89,6 +89,9 @@
 	/** \brief A color key is applied. */
 	ShapeModeColorKey
 } WindowShapeMode;
+
+#define SDL_SHAPEMODEALPHA(mode) (mode == ShapeModeDefault || mode == ShapeModeBinarizeAlpha || mode == ShapeModeReverseBinarizeAlpha)
+
 /** \brief A union containing parameters for shaped windows. */
 typedef union {
 	/** \brief a cutoff alpha value for binarization of the window shape's alpha channel. */
--- a/src/video/SDL_shape.c	Tue Aug 10 15:45:59 2010 -0400
+++ b/src/video/SDL_shape.c	Tue Aug 10 16:10:26 2010 -0400
@@ -104,7 +104,7 @@
                     break;
                 case(ShapeModeColorKey):
                     key = mode.parameters.colorKey;
-                    mask_value = ((key.r != r && key.g != g && key.b != b) ? 1 : 0);
+                    mask_value = ((key.r != r || key.g != g || key.b != b) ? 1 : 0);
                     break;
             }
             bitmap[bitmap_pixel / ppb] |= mask_value << (7 - ((ppb - 1) - (bitmap_pixel % ppb)));
@@ -237,7 +237,6 @@
     
     if(shapeMode != NULL)
         window->shaper->mode = *shapeMode;
-    //TODO: Platform-specific implementations of SetWindowShape.  X11 is finished.  Win32 is finished.  Debugging is in progress on both.
     result = window->display->device->shape_driver.SetWindowShape(window->shaper,shape,shapeMode);
     window->shaper->hasshape = SDL_TRUE;
     if((window->shaper->usershownflag & SDL_WINDOW_SHOWN) == SDL_WINDOW_SHOWN) {
--- a/src/video/x11/SDL_x11shape.c	Tue Aug 10 15:45:59 2010 -0400
+++ b/src/video/x11/SDL_x11shape.c	Tue Aug 10 16:10:26 2010 -0400
@@ -81,12 +81,12 @@
 }
     
 int
-X11_SetWindowShape(SDL_WindowShaper *shaper,SDL_Surface *shape,SDL_WindowShapeMode *shapeMode) {
+X11_SetWindowShape(SDL_WindowShaper *shaper,SDL_Surface *shape,SDL_WindowShapeMode *shape_mode) {
     if(shaper == NULL || shape == NULL || shaper->driverdata == NULL)
         return -1;
 
 #if SDL_VIDEO_DRIVER_X11_XSHAPE
-    if(!SDL_ISPIXELFORMAT_ALPHA(SDL_MasksToPixelFormatEnum(shape->format->BitsPerPixel,shape->format->Rmask,shape->format->Gmask,shape->format->Bmask,shape->format->Amask)))
+    if(shape->format->Amask == 0 && SDL_SHAPEMODEALPHA(shape_mode->mode))
         return -2;
     if(shape->w != shaper->window->w || shape->h != shaper->window->h)
         return -3;