[Mesa-dev] [PATCH 1/2] mesa: fix the format of glEdgeFlagPointer

[Brian Paul]

On 03/02/2014 06:25 PM, Marek Olšák wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> Softpipe expects a float in the vertex shader, which is what glEdgeFlag
> generates.
>
> This fixes piglit/gl-2.0-edgeflag.
>
> Cc: mesa-stable at lists.freedesktop.org
> ---
>   src/mesa/main/varray.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
> index 44db2cb..b4b6fa9 100644
> --- a/src/mesa/main/varray.c
> +++ b/src/mesa/main/varray.c
> @@ -569,8 +569,8 @@ void GLAPIENTRY
>   _mesa_EdgeFlagPointer(GLsizei stride, const GLvoid *ptr)
>   {
>      const GLbitfield legalTypes = UNSIGNED_BYTE_BIT;
> -   /* see table 2.4 edits in GL_EXT_gpu_shader4 spec: */
> -   const GLboolean integer = GL_TRUE;
> +   /* this is the same type that glEdgeFlag uses */
> +   const GLboolean integer = GL_FALSE;
>      GET_CURRENT_CONTEXT(ctx);
>
>      FLUSH_VERTICES(ctx, 0);
>

For both 1 & 2, Reviewed-by: Brian Paul <brianp at vmware.com>