Lovrom8 / RSMods

383 stars 34 forks source link

Fixed a bunch of compiler warnings (now down to 143 warnings on VS2022) #119

Closed tjwrona closed 2 years ago

tjwrona commented 2 years ago

Fixed a decent amount of warnings. Still needs a decent amount more work. If I have more time later I will look into more of them.

tjwrona commented 2 years ago

Now down to just 33 warnings and the game still runs but some of these were caused by type conversions that seemed a bit sketchy. Many of the warnings were fixed with "static_cast" telling the compiler that those type conversions were intentional. But its likely there is still some undefined behavior going on here and we may want to take a closer look at some of these before officially merging.