bgame icon indicating copy to clipboard operation
bgame copied to clipboard

There are a TON of warnings

Open moon-chilled opened this issue 7 years ago • 3 comments

If you want, I can try to clean some of them up and make a PR, but there are a TON of them

moon-chilled avatar Mar 17 '17 00:03 moon-chilled

You're absolutely right - there are a ton of warnings (depending upon which compiler you use, there are even more - GCC gives a lot more than Clang). My goal is to do a big cleanup on that (and some global variable issues) soon, but I want to stabilize some architecture decisions first. Thanks!

thebracket avatar Mar 17 '17 13:03 thebracket

Two changes to RLTK just went up that massively reduced the number of warnings (especially on Clang). I'd omitted some braces in an initializer list (it still worked, oddly), in each and each_if - which are used everywhere. That's a good start! (I'll have the RLTK link updated in the main source shortly)

thebracket avatar Apr 06 '17 20:04 thebracket

I just went on a big warning hunt, and removed a LOT of warnings from Clang builds. I'll have to do the same for GCC, but not today.

thebracket avatar Apr 06 '17 21:04 thebracket