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

Log SDL keyboard events as part of RUNTIME_DEBUG. #22918

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Nov 13, 2024

Also, test_sdl_key_test.c to test_interactive.py. This test was not being run anywhere else and serves as simple way to test/debug keyboard events.

Also, only set the RUNTIME_DEBUG default if the user does not explicitly set it themselves. This allows for -sSOCKETS_DEBUG -sRUNTIM_DEBUG=0 to opt out of the RUNTIME_DEBUG channel.

Finally, disable the RUNTIME_DEBUG traces that occur during turns of the event loop unless RUNTIME_DEBUG is set to 2. This avoids flooding the console will messages in applications that are using the event loop.

@sbc100 sbc100 requested a review from kripken November 13, 2024 19:14
@sbc100
Copy link
Collaborator Author

sbc100 commented Nov 13, 2024

I admit this is kind of 3 changes in one, but they are all related.

@sbc100 sbc100 force-pushed the test_sdl_key_test branch 4 times, most recently from e016451 to 0c8a7cf Compare November 13, 2024 19:48
@sbc100 sbc100 changed the title Log SDL keybaord events as part of RUNTIME_DEBUG. Log SDL keyboard events as part of RUNTIME_DEBUG. Nov 13, 2024
@sbc100 sbc100 requested a review from dschuff November 13, 2024 20:01
site/source/docs/tools_reference/settings_reference.rst Outdated Show resolved Hide resolved
site/source/docs/tools_reference/settings_reference.rst Outdated Show resolved Hide resolved
src/library.js Outdated Show resolved Hide resolved
src/runtime_stack_check.js Outdated Show resolved Hide resolved
src/settings.js Outdated Show resolved Hide resolved
Also, test_sdl_key_test.c to test_interactive.py.  This test was not
being run anywhere else and serves as simple way to test/debug keyboard
events.

Also, only set the `RUNTIME_DEBUG` default if the user does not
explicitly set it themselves.  This allows for `-sSOCKETS_DEBUG
-sRUNTIM_DEBUG=0` to opt out of the `RUNTIME_DEBUG` channel.

Finally, disable the `RUNTIME_DEBUG` traces that occur during turns of
the event loop unless `RUNTIME_DEBUG` is set to 2.  This avoids flooding
the console will messages in applications that are using the event loop.
@sbc100 sbc100 merged commit 475eeda into emscripten-core:main Nov 13, 2024
2 of 13 checks passed
@sbc100 sbc100 deleted the test_sdl_key_test branch November 13, 2024 21:11
uysalibov pushed a commit to uysalibov/emscripten that referenced this pull request Nov 15, 2024
)

Also, test_sdl_key_test.c to test_interactive.py. This test was not
being run anywhere else and serves as simple way to test/debug keyboard
events.

Also, only set the `RUNTIME_DEBUG` default if the user does not
explicitly set it themselves. This allows for `-sSOCKETS_DEBUG
-sRUNTIM_DEBUG=0` to opt out of the `RUNTIME_DEBUG` channel.

Finally, disable the `RUNTIME_DEBUG` traces that occur during turns of
the event loop unless `RUNTIME_DEBUG` is set to 2. This avoids flooding
the console will messages in applications that are using the event loop.
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