Skip to content
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.

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

Added Webassembly support and fixed GLES2 (somewhat) #169

Merged
merged 91 commits into from
Aug 11, 2024
Merged

Added Webassembly support and fixed GLES2 (somewhat) #169

merged 91 commits into from
Aug 11, 2024

Conversation

Lockl00p
Copy link
Contributor

Webassembly version doesn’t work online and screen flashes on GLES2 are opaque.

Lockl00p and others added 30 commits April 8, 2024 21:35
(For some reason, make exits with an error after building emscripten-release. Don't know how to fix it, please don't ask)
Change DP_LINK_JPEG back to dlopen
Remove duplicate parameters
Create index.html as an example of how to use the Emscripten build.
Make game resize on SDL resize events.
Create autoexec.cfg to set some game settings.
Update gitignore.
Update readme with instructions on how to server the wasm/index.html
Revert the change that broke rendering in regular SDL builds
Clean up makefiles
Revert shader change which seems to do nothing
Update path in index.html
Change WASM HTML page title
Lockl00p and others added 7 commits May 15, 2024 15:11
Fixes frame abort (it did an immediate unclean shutdown on all
platforms).
Fixes vid_vsync 0, cl_maxfps, etc.
Fixes clean shutdown (config saving on `quit` works again).
Removes code duplication and platform-specific ifdefs in host.c

Signed-off-by: bones_was_here <[email protected]>
@Cloudwalk9
Copy link
Contributor

Cloudwalk9 commented Jul 4, 2024

This is going to be merged one way or another eventually but we're moving at below glacial pace due to the nature of volunteer work (irl things taking precedence) and the complexity of implementing support for a new platform. General reminder not to feel discouraged.

@Cloudwalk9 Cloudwalk9 added the enhancement New feature or request label Jul 4, 2024
Copy link
Contributor

@bones-was-here bones-was-here left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you fix all the compiler warnings in sys_wasm.c please? New code shouldn't have any warns.
Don't worry about the warns in other files, they're not caused by your code and are harmless (mostly unused variables due to compiling GLES, we can clean up GLES in another PR).

@Lockl00p Lockl00p requested a review from bones-was-here August 7, 2024 23:56
Signed-off-by: bones_was_here <[email protected]>
stringToNewUTF8() mallocs memory which must be free'd, and which doesn't
need to be const (more convenient here).

Signed-off-by: bones_was_here <[email protected]>
Most of these don't need to be declared for all compilation units.
They're all FS commands.

Signed-off-by: bones_was_here <[email protected]>
@bones-was-here
Copy link
Contributor

I have a bit more tidying and testing todo but then I think it'll be good to merge, planning to do that Sunday (unless someone finds an issue)

@bones-was-here bones-was-here merged commit 558f167 into DarkPlacesEngine:master Aug 11, 2024
2 checks passed
@Cloudwalk9
Copy link
Contributor

Great work guys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants