On Fri, Jan 9, 2015 at 5:21 PM,  <michael.w.ma...@intel.com> wrote:
> From: Mike Mason <michael.w.ma...@intel.com>
>
> Changes the initial internal format of a render buffer
> to GL_RGBA4. This fixes a failure in the following
> DrawElements test:
>
> dEQP-GLES3.functional.state_query.rbo.renderbuffer_internal_format
> ---
>  src/mesa/main/renderbuffer.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/renderbuffer.c b/src/mesa/main/renderbuffer.c
> index 0bc7f2b..b339491 100644
> --- a/src/mesa/main/renderbuffer.c
> +++ b/src/mesa/main/renderbuffer.c
> @@ -53,7 +53,11 @@ _mesa_init_renderbuffer(struct gl_renderbuffer *rb, GLuint 
> name)
>     rb->Width = 0;
>     rb->Height = 0;
>     rb->Depth = 0;
> -   rb->InternalFormat = GL_RGBA;
> +   /* Default internal format should be GL_RGBA4, per page 258,
> +    * table 6.15 of the GLES 3.0.4 spec. Same default is expected
> +    * in all OpenGL specs that support BindRenderbuffer().
> +    */
OpenGL expects GL_RGBA as initial internal format. See table
23.27, page 581 of OpenGL 4.5 spec. Similar behavior is expected
in earlier versions as well. So, make this change only for GLES 3.0.

> +   rb->InternalFormat = GL_RGBA4;
>     rb->Format = MESA_FORMAT_NONE;
>  }
>
> --
> 1.9.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to