Skip to content

Fix run win32 console reader in bare thread#15724

Merged
straight-shoota merged 1 commit intocrystal-lang:masterfrom
ysbaddaden:fix/start-win32-console-reader-loop-in-bare-thread
Apr 28, 2025
Merged

Fix run win32 console reader in bare thread#15724
straight-shoota merged 1 commit intocrystal-lang:masterfrom
ysbaddaden:fix/start-win32-console-reader-loop-in-bare-thread

Conversation

@ysbaddaden
Copy link
Collaborator

@ysbaddaden ysbaddaden commented Apr 28, 2025

Simpler alternative to #15714 that focuses on dropping the isolated context that depends on the default context (not yet initialized at this point) and we don't need the isolated context bubble wrap anyway.

Closes #15714

@ysbaddaden ysbaddaden self-assigned this Apr 28, 2025
@straight-shoota straight-shoota added kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:windows Windows support based on the MSVC toolchain / Win32 API topic:stdlib:runtime labels Apr 28, 2025
@straight-shoota straight-shoota added this to the 1.16.2 milestone Apr 28, 2025
@straight-shoota straight-shoota merged commit fc41c28 into crystal-lang:master Apr 28, 2025
38 checks passed
@crysbot
Copy link
Collaborator

crysbot commented Apr 28, 2025

Server Error

created backport PR for release/1.16:

straight-shoota pushed a commit that referenced this pull request Apr 28, 2025
Co-authored-by: Julien Portalier <julien@portalier.com>
@ysbaddaden ysbaddaden deleted the fix/start-win32-console-reader-loop-in-bare-thread branch April 29, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport release/1.16 kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:windows Windows support based on the MSVC toolchain / Win32 API topic:stdlib:runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants