SDL3 port, easy to merge for forks (first commit is code, next two are updating windows vendored libraries) #321
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Notes:
- Only three meaningful changes were needed:
* adding a call to SDL_StartTextInput() in main.c
* updating and unifying the get_scale() DPI code (CAUTION: cannot test)
* move window event blocks in f_poll_event() in system.c
- Linux: works with system or user-provided SDL3 libraries
Linking SDL3 statically is possible with small modifications to build.sh,
by adding custom include and libdirs in cflags and ldflags, and
changing linking for "-Wl,-Bstatic -lSDL3 -Wl,-Bdynamic ..."
A statically-linked SDL3 can still use dynamic dlopen() to load the platform libs
- Windows: vendored SDL3 headers and libs are NOT UPDATED in this commit.
These files will be updated in the next two commits.
If using this commit, bring your own SDL3.dll and headers
into winlib/SDL3-3.2.16/x86_64-w64-mingw32/{include/,lib/,bin/}
…iginal lite with SDL2 This is only for keeping the same structure as in the original lite ; we'll create another repo without any vendoring (Lua or SDL), to have an actual lightweight repository.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This SDL3 minimal patch may be useful to the many forks of rxi/lite.
Rationale:
This change is split into 3 logical commits: