search

bvschaik / julius

An open source re-implementation of Caesar III
GNU Affero General Public License v3.0
2.79k stars 311 forks source link

Increase warnings for julius #694

Closed crudelios closed 1 year ago

crudelios commented 1 year ago

This adds some GCC configurations to display more warnings.

These extra warnings helped me fix some bugs in Augustus, maybe they'll be useful for Julius as well.

Note: after enabling the new warnings, a full compilation triggers 93 warnings on mingw, some of them related to our external libs.

QuakeIV commented 1 year ago

normally you flag external targets to ignore them

crudelios commented 1 year ago

They are flagged on cmake, but apparently that only considers include files, not source files, which we actually compile...

If there's a better way to do this I'm all for it.