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

2016-03-09 Thread Matt Turner
On Wed, Mar 9, 2016 at 1:37 PM, Francisco Jerez wrote: > Iago Toral writes: > >> 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 i

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

2016-03-09 Thread Francisco Jerez
Iago Toral writes: > 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

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

2016-03-09 Thread Iago Toral
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

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

2016-03-08 Thread Francisco Jerez
brw_cfg.h already has include guards, remove the "#pragma once" which is redundant and non-standard. --- 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 -