Mesa (mesa_7_7_branch): intel: Make RGB renderbuffers use XRGB8888 like we do for RGB system buffers.

Eric Anholt anholt at kemper.freedesktop.org
Thu Dec 3 00:21:17 UTC 2009


Module: Mesa
Branch: mesa_7_7_branch
Commit: 4598942b1b88a2a7d5af7febae7e79eedf00e385
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=4598942b1b88a2a7d5af7febae7e79eedf00e385

Author: Eric Anholt <eric at anholt.net>
Date:   Wed Dec  2 13:00:15 2009 -0800

intel: Make RGB renderbuffers use XRGB8888 like we do for RGB system buffers.

---

 src/mesa/drivers/dri/intel/intel_fbo.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/src/mesa/drivers/dri/intel/intel_fbo.c b/src/mesa/drivers/dri/intel/intel_fbo.c
index 5615040..b6e0d82 100644
--- a/src/mesa/drivers/dri/intel/intel_fbo.c
+++ b/src/mesa/drivers/dri/intel/intel_fbo.c
@@ -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;




More information about the mesa-commit mailing list