I had to disable -Werror in the default compiler settings for -DCMAKE_BUILD_TYPE=RelWithDebInfo to build with clang16, and I disliked that. So, I fixed all the showstoppers. None of them were critical, but overall, they increase readability, reduce probability of future bugs. The struct vs class fwd declaration might be problematic on MSVC, anyways, fixed that, no reason not to.
Also, moved the CMake version check in front of the project definition.
CMake: Check for minimum version before setting the project name
core/organisefmt: use same visibility for fwd decl as in def
internet services: consistently use 'override'
library: use boolean, not bitwise, operator on bools
I had to disable
-Werror
in the default compiler settings for-DCMAKE_BUILD_TYPE=RelWithDebInfo
to build with clang16, and I disliked that. So, I fixed all the showstoppers. None of them were critical, but overall, they increase readability, reduce probability of future bugs. The struct vs class fwd declaration might be problematic on MSVC, anyways, fixed that, no reason not to.Also, moved the CMake version check in front of the project definition.