omf2097 / openomf

One Must Fall 2097 Remake
http://www.openomf.org
MIT License
362 stars 35 forks source link

Improve clang-tidy options #447

Closed katajakasa closed 2 years ago

katajakasa commented 2 years ago

This is on top of clang-format stuff.

I dropped the secure function (_s) checks, as they seem to be rather pointless, and are marked optional in the spec. Let's not bother.

Otherwise, work-in-progress. This should be expanded to also add fixes for all tidy warnings, and then set WarningsAsErrors: '*' in .clang-tidy to make sure problems crash the build. Also, set -DUSE_TIDY=1 in tests builds.