Use compiler warnings

[tagniam]

Use compiler warnings

[tagniam]

... the whole project's is completely mixed on tabs vs spaces. IMO as a separate issue this should be fixed.

... identifiers starting with an underscore and then an upper case character (like _Level, or _Player) are reserved for implementations, and using them is undefined behavior

I will soon be pushing out a code style change that will finally fix these issues once and for all. :) I previously opened an issue (#7) that tried to address this but I closed it due to inactivity.

Feel free to make a PR that fixes all the warnings you listed. As an extension, I would like to use these compiler warnings as part our Travis CI pipeline, let me know if you feel up to the task - otherwise, I'll create a separate issue for it.

[bodand]

OK, I'll create the PR in a few minutes.

For the Travis CI pipeline, I have no idea as I never worked with Travis nor any CI really. In the past, I once tried using it, but it didn't really work out, so I can't say I'm up to that task.
Also, I'd look into the new GitHub actions instead, because then you can keep development completely inside GitHub. But again, I never really tried either of them. Read below

[bodand]

Oh, we already have a CI pipeline. I completely missed that. Then the CMake build should work with the warnings. As for detecting them, I have no idea.