We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 3459 - SDL_assert doesn't work in emscripten
Summary: SDL_assert doesn't work in emscripten
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: *don't know* (show other bugs)
Version: 2.0.4
Hardware: Other Other
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-17 13:02 UTC by hef
Modified: 2017-05-19 18:53 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description hef 2016-10-17 13:02:20 UTC 
SDL_assert doesn't work in emscripten.

really, SDL_assert doesn't work in platforms that have turned off thread support, because SDL_ReportAssertion() attempts to create a mutex, and if it fails, returns SDL_ASSERTION_IGNORE instead of doing something useful.
Comment 1 Ryan C. Gordon 2017-05-19 16:55:02 UTC 
The lack of thread support is fixed now in https://hg.libsdl.org/SDL/rev/b00c4088f687 ...still need to make this suck less on Emscripten, though, so I'm leaving this bug open a little longer.

--ryan.
Comment 2 Ryan C. Gordon 2017-05-19 18:53:11 UTC 
(In reply to Ryan C. Gordon from comment #1)
> The lack of thread support is fixed now in
> https://hg.libsdl.org/SDL/rev/b00c4088f687 ...still need to make this suck
> less on Emscripten, though, so I'm leaving this bug open a little longer.

Ok, now it's ugly but largely functions as one would expect on Emscripten, as of https://hg.libsdl.org/SDL/rev/411bf9f7f908 ...marking this bug resolved.

--ryan.