[Mesa-dev] [PATCH] i965/cfg: Remove redundant #pragma once.

Iago Toral itoral at igalia.com
Wed Mar 9 15:15:28 UTC 2016


On Tue, 2016-03-08 at 17:42 -0800, Francisco Jerez wrote:
> brw_cfg.h already has include guards, remove the "#pragma once" which
> is redundant and non-standard.

FWIW, I think using both #pragma once and include guards is a way to
keep portability while still getting the performance advantage of
#pragma once where it is supported.

Also it seems that we do the same thing in many other files...

> ---
>  src/mesa/drivers/dri/i965/brw_cfg.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h b/src/mesa/drivers/dri/i965/brw_cfg.h
> index 405020b..a2ca6b1 100644
> --- a/src/mesa/drivers/dri/i965/brw_cfg.h
> +++ b/src/mesa/drivers/dri/i965/brw_cfg.h
> @@ -25,7 +25,6 @@
>   *
>   */
>  
> -#pragma once
>  #ifndef BRW_CFG_H
>  #define BRW_CFG_H
>  




More information about the mesa-dev mailing list