[Mesa-dev] [PATCH 2/4] mesa: Draw Indirect is not allowed when no vertex array binding exists.

[Matt Turner]

On Tue, Oct 20, 2015 at 7:19 AM, Marta Lofstedt
<marta.lofstedt at linux.intel.com> wrote:
> From: Marta Lofstedt <marta.lofstedt at intel.com>
>
> OpenGL ES 3.1 spec. section 10.5:
> "An INVALID_OPERATION error is generated if zero is bound
> to VERTEX_ARRAY_BINDING, DRAW_INDIRECT_BUFFER or to
> any enabled vertex array."
>
> Signed-off-by: Marta Lofstedt <marta.lofstedt at linux.intel.com>
> ---
>  src/mesa/main/api_validate.c | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/src/mesa/main/api_validate.c b/src/mesa/main/api_validate.c
> index c5628f5..7062cbd 100644
> --- a/src/mesa/main/api_validate.c
> +++ b/src/mesa/main/api_validate.c
> @@ -711,6 +711,20 @@ valid_draw_indirect(struct gl_context *ctx,
>        return GL_FALSE;
>     }
>
> +   /*

Comment is supposed to start on this^ line.

> +    * OpenGL ES 3.1 spec. section 10.5:
> +    * "An INVALID_OPERATION error is generated if zero is bound to
> +    * VERTEX_ARRAY_BINDING, DRAW_INDIRECT_BUFFER or to any enabled
> +    * vertex array."

Can we really not format the spec citations like everywhere else in Mesa?

/* The OpenGL ES 3.1 spec, section 10.5 says:
 *
 *<indent>"blah blah blah
 *<indent> blah blah blah."
 */

> +    * OpenGL 4.5 spec. section 10.4:
> +    * "An INVALID_OPERATION error is generated if  zero is bound to
> +    * DRAW_INDIRECT_BUFFER, or if  no element array buffer is bound"
> +    */

Same thing (and same thing on the other patches as well!)