intel: Make RGB renderbuffers use XRGB8888 like we do for RGB system buffers.
authorEric Anholt <eric@anholt.net>
Wed, 2 Dec 2009 21:00:15 +0000 (13:00 -0800)
committerEric Anholt <eric@anholt.net>
Thu, 3 Dec 2009 00:19:37 +0000 (16:19 -0800)
src/mesa/drivers/dri/intel/intel_fbo.c

index 5615040946fc4fb9b331a13d38279285e4eb1abe..b6e0d823ed213398f60f480fd5bc5af24d5632db 100644 (file)
@@ -126,7 +126,7 @@ intel_alloc_renderbuffer_storage(GLcontext * ctx, struct gl_renderbuffer *rb,
    case GL_RGB16:
       rb->Format = MESA_FORMAT_ARGB8888;
       rb->DataType = GL_UNSIGNED_BYTE;
-      irb->texformat = MESA_FORMAT_ARGB8888; /* XXX: Need xrgb8888 */
+      irb->texformat = MESA_FORMAT_XRGB8888;
       cpp = 4;
       break;
    case GL_RGBA:
@@ -314,10 +314,6 @@ intel_create_renderbuffer(gl_format format)
       irb->Base.DataType = GL_UNSIGNED_BYTE;
       break;
    case MESA_FORMAT_XRGB8888:
-      /* XXX this is a hack since XRGB surfaces don't seem to work
-       * properly yet.  Reading the alpha channel returns 0 instead of 1.
-       */
-      format = MESA_FORMAT_ARGB8888;
       irb->Base._BaseFormat = GL_RGB;
       irb->Base.DataType = GL_UNSIGNED_BYTE;
       break;