envire / envire-envire_core

Core part for the Environment Representation library
BSD 2-Clause "Simplified" License
7 stars 13 forks source link

Switch to #pragma once #4

Closed arneboe closed 8 years ago

arneboe commented 8 years ago

Hey guys,

should we switch from using #ifdef include guards to using "#pragma once"?

pragma once is widely supported (https://en.wikipedia.org/wiki/Pragma_once#Portability)

and less error prone (as you don't have to manually change the include guard when renaming files). For some compilers it improves performance (although not for gcc and clang as they have ifdef guard optimizations).

jhidalgocarrio commented 8 years ago

pragma once is not part of the C++ standard, isn't?

Anyway, it is fine to me, I am not particularly against it. +1

jmachowinski commented 8 years ago

+1