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

Ian Romanick idr at kemper.freedesktop.org
Thu Dec 10 23:17:20 UTC 2009


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

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Tue Dec  8 21:15:36 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 970ffb2..608f75b 100644
--- a/src/mesa/drivers/dri/intel/intel_fbo.c
+++ b/src/mesa/drivers/dri/intel/intel_fbo.c
@@ -123,7 +123,7 @@ intel_alloc_renderbuffer_storage(GLcontext * ctx, struct gl_renderbuffer *rb,
    case GL_RGB10:
    case GL_RGB12:
    case GL_RGB16:
-      rb->Format = MESA_FORMAT_ARGB8888; /* XXX: Need xrgb8888 */
+      rb->Format = MESA_FORMAT_XRGB8888;
       rb->DataType = GL_UNSIGNED_BYTE;
       break;
    case GL_RGBA:
@@ -294,10 +294,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