Tobias Markus
Tobias Markus
That is a lot of code duplication. Can this be done in other ways so we only have the same code once?
How about moving this to a parent ~~GameObject~~ BadGuy class method?
There's various compiler errors: ``` D:\a\supertux\supertux\src\video\canvas.cpp(340,21): error C2641: cannot deduce template arguments for 'std::array' [D:\a\supertux\supertux\build\supertux2_lib.vcxproj] D:\a\supertux\supertux\src\video\canvas.cpp(340,21): error C2780: 'std::array std::array(void)': expects 0 arguments - 6 provided [D:\a\supertux\supertux\build\supertux2_lib.vcxproj] C:\Program Files\Microsoft Visual...
Looks good, I guess. The drawing methods I didn't look at in detail because *urgh, code* but from what I can tell this is good to merge.
This good to merge?
Difficult to review. From my preliminary look, it seems like the code looks good. But let's avoid disasters and test this thoroughly. I like how this reduces complexity in the...
Maybe add a border around that thing so it isn't too "out of place"?
Can you rebase this, please?
I'm afraid people would be less likely to find the option if it existed. Should we ask them at the first start of the game?
How about "Shift"? Easily accessible as well and doesn't interfere with anything.