[Mesa-dev] [PATCH v2 03/18] fbobject: don't allow LUMINANCE/INTENSITY/ALPHA fbo on ES/Core

[Kenneth Graunke]

On 01/12/2013 08:10 PM, Jordan Justen wrote:
> v2:
>   * Only allow on GL Legacy contexts
>
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> ---
>   src/mesa/main/fbobject.c |    3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
> index c2be3ff..a219398 100644
> --- a/src/mesa/main/fbobject.c
> +++ b/src/mesa/main/fbobject.c
> @@ -511,7 +511,8 @@ _mesa_is_legal_color_format(const struct gl_context *ctx, GLenum baseFormat)
>      case GL_LUMINANCE_ALPHA:
>      case GL_INTENSITY:
>      case GL_ALPHA:
> -      return ctx->Extensions.ARB_framebuffer_object;
> +      return ctx->API == API_OPENGL_COMPAT &&
> +             ctx->Extensions.ARB_framebuffer_object;
>      case GL_RED:
>      case GL_RG:
>         return ctx->Extensions.ARB_texture_rg;

Could we get a spec citation for this?  I sure see a lot of references 
to these formats in the ES 3.0 spec, and it isn't immediately clear to 
me that they're gone.  (Though, they should be, and I definitely hope 
they are!)