Skip to content

Protect constant initializers with mutex on Windows#15134

Merged
straight-shoota merged 2 commits intocrystal-lang:masterfrom
HertzDevil:bug/win32-const-mt
Oct 31, 2024
Merged

Protect constant initializers with mutex on Windows#15134
straight-shoota merged 2 commits intocrystal-lang:masterfrom
HertzDevil:bug/win32-const-mt

Conversation

@HertzDevil
Copy link
Contributor

Fixes #15029.

@HertzDevil HertzDevil 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 kind:regression Something that used to correctly work but no longer works labels Oct 29, 2024
@straight-shoota straight-shoota added this to the 1.15.0 milestone Oct 29, 2024
@straight-shoota straight-shoota merged commit 4aac6f2 into crystal-lang:master Oct 31, 2024
@HertzDevil HertzDevil deleted the bug/win32-const-mt branch October 31, 2024 11:03
CTC97 pushed a commit to CTC97/crystal that referenced this pull request Nov 9, 2024
`Crystal::System::FileDescriptor#@@reader_thread` is initialized before `Crystal::System::Fiber::RESERVED_STACK_SIZE` which creates a race condition.
Regression from crystal-lang#14947
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug A bug in the code. Does not apply to documentation, specs, etc. kind:regression Something that used to correctly work but no longer works 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.

CI failure: Recursion while initializing class variables and/or constants

2 participants