[Mesa-dev] [PATCH 06/12] i965: Add parentheses around usage of macro arguments

Iago Toral itoral at igalia.com
Fri Sep 29 11:59:59 UTC 2017


Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>

On Thu, 2017-09-28 at 23:05 -0700, Matt Turner wrote:
> Otherwise I cannot use this macro in test_eu_validate.cpp
> ---
>  src/intel/common/gen_device_info.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/intel/common/gen_device_info.h
> b/src/intel/common/gen_device_info.h
> index ede4915e28..59b345e949 100644
> --- a/src/intel/common/gen_device_info.h
> +++ b/src/intel/common/gen_device_info.h
> @@ -193,7 +193,7 @@ struct gen_device_info
>  };
>  
>  #define gen_device_info_is_9lp(devinfo) \
> -   (devinfo->is_broxton || devinfo->is_geminilake)
> +   ((devinfo)->is_broxton || (devinfo)->is_geminilake)
>  
>  bool gen_get_device_info(int devid, struct gen_device_info
> *devinfo);
>  const char *gen_get_device_name(int devid);


More information about the mesa-dev mailing list