The ScanDirectory method of FileScanner allocates pointers with new and returns raw pointers. It should use unique_ptr instead.

Note: Wait for #13444 resolution before working on this.

This function needs a unit test.

Guide to how to contribute to an open source project on GitHub.

Test stub: https://github.com/pygame/pygame/blob/main/test/display_test.py#L255-L274
Docs: https://www.pygame.org/docs/ref/display.html#pygame.display.gl_set_attribute
Code to be tested: https://github.com/pygame/pygame/bl

SuperTux version: Newest nightly (began around when the DIscord RPC was implemented)
System information: (details about your kernel, operating system, or distribution) Windows 10 (?)

Expected behavior

Upon installing, SuperTux should open as usual.

Actual behavior

SuperTux does not open and crashes upon startup, giving me the message: "Libcurl.dll was not found"

My English sucks

So, you know

Thanks.

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

Via playtesting, the compass arrows (#29) were ignored, likely because they are right at the edge and outside the player's attention. If they are moved slightly inwards, they may become more noticeable.

Games that have these arrows also tend to draw them closer to the player rather than near the edge; e.g.

• GTA 1 & 2
• Fire Fight
• Crazy Taxi

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 ;

Help messages for various built-in commands are annoying mish-mash of styles and formatting. Plenty of documentation is simply missing (and users usually don't know where to find our manual file).

In #608 and #577 we improved the help messages for mount and imgmount. In the process, we established a nice, readable style for our help commands - now it's time to expand help for all remaining