Investigate CI to avoid nameclashes.

Open 1uc opened this issue 2 years ago • 1 comments

Since we recently had an issue where we allowed names reserved for macros to be used in HighFive, e.g. DEBUG or INFO, it would make sense to investigate if there's any static analysis we can set up, e.g. clang-tidy to ban UPPERCASE names for anything that isn't a macro.

See #716 for the type of issue we want to detect.

Apr 03 '23 10:04 1uc

There's this: https://clang.llvm.org/extra/clang-tidy/checks/readability/identifier-naming.html

Apr 04 '23 05:04 1uc