mesa: Fix render buffer initial internal format in GLES 3
authorMike Mason <michael.w.mason@intel.com>
Wed, 14 Jan 2015 20:12:27 +0000 (12:12 -0800)
committerChad Versace <chad.versace@intel.com>
Thu, 15 Jan 2015 21:29:48 +0000 (13:29 -0800)
Changes the initial internal format of a render buffer
to GL_RGBA4 in GLES 3. This fixes a failure in the following
DrawElements test:

  dEQP-GLES3.functional.state_query.rbo.renderbuffer_internal_format

Reviewed-by: Chad Versace <chad.versace@intel.com>
src/mesa/main/renderbuffer.c

index 0bc7f2b965447fae46375c6331a3723918969b97..cc97ab22bed931ecc34ca81fab3037b483d5a735 100644 (file)
@@ -53,7 +53,24 @@ _mesa_init_renderbuffer(struct gl_renderbuffer *rb, GLuint name)
    rb->Width = 0;
    rb->Height = 0;
    rb->Depth = 0;
-   rb->InternalFormat = GL_RGBA;
+
+   /* In GL 3, the initial format is GL_RGBA according to Table 6.26
+    * on page 302 of the GL 3.3 spec.
+    *
+    * In GLES 3, the initial format is GL_RGBA4 according to Table 6.15
+    * on page 258 of the GLES 3.0.4 spec.
+    *
+    * If the context is current, set the initial format based on the
+    * specs. If the context is not current, we cannot determine the
+    * API, so default to GL_RGBA.
+    */
+   GET_CURRENT_CONTEXT(ctx);
+   if (ctx && _mesa_is_gles3(ctx)) {
+      rb->InternalFormat = GL_RGBA4;
+   } else {
+      rb->InternalFormat = GL_RGBA;
+   }
+
    rb->Format = MESA_FORMAT_NONE;
 }