Skip to content

Loadso's Unload returns bool to indicate success/failure#15918

Open
CowKeyMan wants to merge 1 commit into
libsdl-org:mainfrom
CowKeyMan:main
Open

Loadso's Unload returns bool to indicate success/failure#15918
CowKeyMan wants to merge 1 commit into
libsdl-org:mainfrom
CowKeyMan:main

Conversation

@CowKeyMan

Copy link
Copy Markdown
  • I confirm that I am the author of this code and release it to the SDL project under the Zlib license. This contribution does not contain code from other sources, including code generated by a Large Language Model ("AI").

Description

While debugging loading dynamic libraries, and in this case, while debugging unloading them, it would be good to have the necessary debug tools. This PR allows the user to rule out any possibility that it is loadso's unloading that is failing, but rather something else in the user's code. Or if it actually is failing, then the user will know of the failure and also get an appropriate error message from the OS.

Existing Issue(s)

No existing issues

@CowKeyMan CowKeyMan marked this pull request as draft July 1, 2026 22:22
@CowKeyMan CowKeyMan marked this pull request as ready for review July 1, 2026 22:32
@slouken

slouken commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

I'll throw this in the 4.x milestone, since changing a function's return value changes the ABI.

@slouken slouken added this to the 4.x milestone Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants