[Mesa-dev] [PATCH] mesa: use sizeof on the correct type

[Brian Paul]

On 02/22/2016 03:58 PM, Thomas Hindoe Paaboel Andersen wrote:
> Before the luminance stride was based on the size of GL_FLOAT
> which is just the type constant (0x1406). Change it to use the
> size of GLfloat.
> ---
>   src/mesa/main/readpix.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/readpix.c b/src/mesa/main/readpix.c
> index 470182a..882d863 100644
> --- a/src/mesa/main/readpix.c
> +++ b/src/mesa/main/readpix.c
> @@ -582,7 +582,7 @@ read_rgba_pixels( struct gl_context *ctx,
>         void *luminance;
>         uint32_t luminance_format;
>
> -      luminance_stride = width * sizeof(GL_FLOAT);
> +      luminance_stride = width * sizeof(GLfloat);
>         if (format == GL_LUMINANCE_ALPHA)
>            luminance_stride *= 2;
>         luminance_bytes = height * luminance_stride;
>

Thanks.

Reviewed-by: Brian Paul <brianp at vmware.com>

Need me to push this for you?

-Brian