[Mesa-dev] [PATCH] mesa: fix Z32_FLOAT -> uint conversion functions

[Jose Fonseca]

Looks good.

Jose

----- Original Message -----
> The IROUND converted all arguments to 0 or 1.  That's not what we
> wanted.
> 
> NOTE: This is a candidate for the 8.0 branch.
> ---
>  src/mesa/main/format_unpack.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/mesa/main/format_unpack.c
> b/src/mesa/main/format_unpack.c
> index b00e012..c42bac1 100644
> --- a/src/mesa/main/format_unpack.c
> +++ b/src/mesa/main/format_unpack.c
> @@ -2929,7 +2929,7 @@ unpack_uint_z_Z32_FLOAT(const void *src, GLuint
> *dst, GLuint n)
>     const float *s = (const float *)src;
>     GLuint i;
>     for (i = 0; i < n; i++) {
> -      dst[i] = FLOAT_TO_UINT(IROUND(CLAMP((s[i]), 0.0F, 1.0F)));
> +      dst[i] = FLOAT_TO_UINT(CLAMP(s[i], 0.0F, 1.0F));
>     }
>  }
>  
> @@ -2940,7 +2940,7 @@ unpack_uint_z_Z32_FLOAT_X24S8(const void *src,
> GLuint *dst, GLuint n)
>     GLuint i;
>  
>     for (i = 0; i < n; i++) {
> -      dst[i] = FLOAT_TO_UINT(IROUND(CLAMP((s[i].z), 0.0F, 1.0F)));
> +      dst[i] = FLOAT_TO_UINT(CLAMP(s[i].z, 0.0F, 1.0F));
>     }
>  }
>  
> --
> 1.7.3.4
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>