On 7 October 2015 at 12:27, Boyan Ding <boyan.j.d...@gmail.com> wrote: > 2015-10-07 18:11 GMT+08:00 Emil Velikov <emil.l.veli...@gmail.com>: >> We will reuse it in later commits, to check if other headers are >> included from within nir.h. >> >> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> >> --- >> src/glsl/nir/nir.h | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h >> index bde9f49..b017be2 100644 >> --- a/src/glsl/nir/nir.h >> +++ b/src/glsl/nir/nir.h >> @@ -26,6 +26,8 @@ >> */ >> >> #pragma once >> +#ifndef NIR_H >> +#define NIR_H > > The '#pragma once' above will become redundant after the #ifdef compile > guard is added. No idea whether it should be removed, though. > Indeed it would. There was a discussion a while back on the topic pragma once and/or ifndef [1]. Don't think we ever reached a consensus though. I can drop the pragma, If there are no objections.
Thanks for bringing this up :) Emil [1] http://lists.freedesktop.org/archives/mesa-dev/2013-September/045377.html _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev