OutpostUniverse / op2ext

Outpost 2 extension module loader
1 stars 0 forks source link

Misc fixes #283

Closed DanRStevens closed 5 years ago

DanRStevens commented 5 years ago

Fixed a few things reported by increased warning options from Mingw.

DanRStevens commented 5 years ago

There does need to be a semicolon after the closing brace of a class declaration. Same with structs and unions too. The code here though is the closing brace of the constructor and class method implementation in the .cpp file.

DanRStevens commented 5 years ago

I added one more flag for warning checks to the Linux build. Now that the debug logger change to always log for all configurations has been merged into master, that code will no longer trigger a warning about an unused parameter for non-debug builds.

The flag change is a Linux only change.