Cursor-Custodian icon indicating copy to clipboard operation
Cursor-Custodian copied to clipboard

Added CMake, Fixed Memory Leak, Fixed Score

Open TheMrPoseidon opened this issue 4 years ago • 3 comments

In order to work properly on mac I've changed the build system to cmake. Additionally, I've fixed two minor problems a) memory leak in WindowsRenderer that increase the memeroy usage over time and b) fixed the output of distance and best values.

TheMrPoseidon avatar Dec 28 '20 00:12 TheMrPoseidon

@TheMrPoseidon why did you change SDL2/sdl to just sdl? I think the first one is used in case sdl(1) is also installed

mandar1jn avatar Jun 02 '21 13:06 mandar1jn

@TheMrPoseidon why did you change SDL2/sdl to just sdl? I think the first one is used in case sdl(1) is also installed

I'm not a pro in how cross plattform development should be done with sdl2, but as far as I know on unix system (linux and mac) you should use sdl2/sdl.h, but for windows and as for emscripten if I remember correctly you should use the include without the namespace upfront. Hope this answers your question, if not I'm happy to test my code (It's been a while) and come back to you.

TheMrPoseidon avatar Jun 02 '21 14:06 TheMrPoseidon

@mandar1jn Forget what I said, the change from #include <SDL2/SDL.h> to #include <SDL.h> was done to work with the SDL2 cmake scripts.

TheMrPoseidon avatar Jun 02 '21 16:06 TheMrPoseidon