Skip to content

Move to SDL3 #6735

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Move to SDL3 #6735

wants to merge 4 commits into from

Conversation

notimaginative
Copy link
Contributor

Make the move from SDL2 to SDL3 along with fixes and improvements made along the way. (long-term draft PR)

@notimaginative notimaginative added refactor A cleanup/restructure of a feature for speed, simplicity, and/or maintainability library A feature or issue related to third-party libraries labels May 27, 2025
@notimaginative
Copy link
Contributor Author

All existing features should be in working order now, though only limited testing was done to confirm that. Additional tweaks are likely. The main exception is setting of gamma, which has been removed from SDL3. This will need to be handled by shaders instead.

Look for TODO: SDL3 comments in the code for any notable issue or change that may need to be addressed in the future.

This PR will also see bigger refactors to make use of SDL3's new audio and filesystem code. Proper gamepad support and working multi voice chat are also likely. Those changes should be self-contained feature commits which may eventually be split into separate PRs or just dropped entirely.

WARNING: This PR will be rebased, squashed, and reordered many times before it's done. So if you follow it locally then please plan accordingly.

@notimaginative notimaginative force-pushed the move_to_sdl3 branch 4 times, most recently from 0a4b830 to a4c9284 Compare May 28, 2025 17:58
@notimaginative
first pass at move to SDL3
11be04d 
Also updates imgui and related addons to newest versions
@notimaginative
clean up cfile
1ef7a70 
 - replace a lot of platform specific code with SDL filesystem functions
 - remove support for memory-mapped files (not actually used)
 - only do special case-sensitive filesystem handling if required
 - support case-sensitive roots on Windows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@z64555 z64555 Awaiting requested review from z64555 z64555 will be requested when the pull request is marked ready for review z64555 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
library A feature or issue related to third-party libraries refactor A cleanup/restructure of a feature for speed, simplicity, and/or maintainability
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@notimaginative