search

eth-cscs / DLA-Future

DLA-Future
https://eth-cscs.github.io/DLA-Future/master/
BSD 3-Clause "New" or "Revised" License
64 stars 14 forks source link

Check for include guards in all header files in repo, not only in `include` directory #1117

Closed msimberg closed 5 months ago

msimberg commented 6 months ago

Fixes missing #pragma once in a couple of headers.

msimberg commented 6 months ago

cscs-ci run

msimberg commented 6 months ago

cscs-ci run

rasolca commented 5 months ago

Note: the C++ Core Guidelines suggest not to use pragma once (but it is less error prone, and easier to check).

We discussed it when we decided to use it. Most of the compilers support this feature (https://en.wikipedia.org/wiki/Pragma_once) so it is not a problem.