[Mesa-dev] [PATCH] i915: add parens to silence operator precedence warning

Kenneth Graunke kenneth at whitecape.org
Tue Mar 10 20:52:19 PDT 2015


On Tuesday, March 10, 2015 05:41:28 PM Brian Paul wrote:
> ---
>  src/mesa/drivers/dri/i915/i915_debug_fp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i915/i915_debug_fp.c b/src/mesa/drivers/dri/i915/i915_debug_fp.c
> index 3f09902..d107c5a 100644
> --- a/src/mesa/drivers/dri/i915/i915_debug_fp.c
> +++ b/src/mesa/drivers/dri/i915/i915_debug_fp.c
> @@ -303,7 +303,7 @@ i915_disassemble_program(const GLuint * program, GLuint sz)
>  
>     printf("\t\tBEGIN\n");
>  
> -   assert(program[0] & 0x1ff + 2 == sz);
> +   assert((program[0] & 0x1ff) + 2 == sz);
>  
>     program++;
>     for (i = 1; i < sz; i += 3, program += 3) {
> 

LGTM, thanks

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


More information about the mesa-dev mailing list