[Mesa-dev] [PATCH] i965: Separate gen < 8 and gen >= 8 paths explicitly in wrap_mode()

[Kenneth Graunke]

On Monday, June 26, 2017 11:07:11 AM PDT Topi Pohjolainen wrote:
> Makes coverity happier.
> 
> Fix indentation in gen >= 8 block while at it.
> 
> CID: 1413020
> CC: Rafael Antognolli <rafael.antognolli at intel.com>
> Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
> ---
>  src/mesa/drivers/dri/i965/genX_state_upload.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index 3f8a7265db..d65b468863 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
> @@ -4455,9 +4455,8 @@ translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
>         *
>         * Gen8+ supports this natively.
>         */
> -         return TCM_HALF_BORDER;
> -#endif
> -
> +      return TCM_HALF_BORDER;
> +#else
>        /* On Gen4-7.5, we clamp the coordinates in the fragment shader
>         * and set clamp_border here, which gets the result desired.
>         * We just use clamp(_to_edge) for nearest, because for nearest
> @@ -4468,6 +4467,7 @@ translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
>           return TCM_CLAMP;
>        else
>           return TCM_CLAMP_BORDER;
> +#endif
>     case GL_CLAMP_TO_EDGE:
>        return TCM_CLAMP;
>     case GL_CLAMP_TO_BORDER:
> 

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170626/c0d40ba1/attachment.sig>