st/mesa: simplify a bit of the previous patch
authorBrian Paul <brianp@vmware.com>
Fri, 15 Apr 2011 14:32:12 +0000 (08:32 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 15 Apr 2011 14:33:20 +0000 (08:33 -0600)
Grrr, this was supposed to go in the previous commit.

src/mesa/state_tracker/st_cb_readpixels.c

index 1b57e9b6f7e1474a770396f3b7414a3fc7f9b338..2a63799bdbc3235277c3fea268642d2b5c1ea5d5 100644 (file)
@@ -211,9 +211,11 @@ st_fast_readpixels(struct gl_context *ctx, struct st_renderbuffer *strb,
 
    if (strb->format == PIPE_FORMAT_B8G8R8A8_UNORM) {
       alphaORoperand = 0;
-   } else if (strb->format == PIPE_FORMAT_B8G8R8X8_UNORM ) {
+   }
+   else if (strb->format == PIPE_FORMAT_B8G8R8X8_UNORM ) {
       alphaORoperand = 0xff;
-   } else {
+   }
+   else {
       return GL_FALSE;
    }
 
@@ -312,9 +314,10 @@ st_fast_readpixels(struct gl_context *ctx, struct st_renderbuffer *strb,
             const GLubyte *src = map + y * trans->stride;
             memcpy(dst, src, 4 * width);
             if (alphaORoperand) {
-                for (col = 0; col < width; col++) {
-                    dst[col*4+3] |= alphaORoperand;
-                }
+               assert(alphaORoperand == 0xff);
+               for (col = 0; col < width; col++) {
+                  dst[col*4+3] = 0xff;
+               }
             }
             dst += dstStride;
             y += dy;