This function needs a unit test.

Guide to how to contribute to an open source project on GitHub.
Draft guide on how to write a unit test for pygame

Test stub: https://github.com/pygame/pygame/blob/master/test/surface_test.py#L1017-L1026
Docs: https

(As this is an enhancement not a bug, I don't think i need to provide too much information.)
Although enemies in Super Mario don't have such animation either, as a more elegant platform game, it is better to add animations like kicking Tux when Tux collides them, because it can make the enemies more vivid.

From the comment in SDL_filesystem.h:
Please call SDL_free() on the pointer when you are done with it

When building the project with MSVC, I ran into some debug heap asserts early into the program startup.
The culprit appears to be the code in m_config.c that does free(prefdir) as changing these to SDL_free(prefdir) resolved the issue.

I'm not sure what to do about `GetDefaultConfi

Suppose you have weapon A and B, A being equipped. Now you attempt to pick up weapon B. What happens is:

• Weapons are switched, which is the best option since there is no dual wielding.
• However, the drop/pickup code acts as if you now have two B weapons, turning the weapon on the floor into weapon A.

Basically, you have turned a B weapon into an A weapon.

EDIT: Reported for 0.9.1

The reasons why this header file lives outside of game_logic don't apply anymore. It would make things simpler if it was merged into the other "global" file.

The file dialogs should get the ability to store bookmarks and maybe even to show the default folders of the OS (like Downloads, Pictures, ...)

The custom bookmarks should just be stored as a Cvar (see core/Var.h, or in-game-console command cvarlist) that is separated with ;