fix: replace curl_error_map with switch to fix Static Initialization Order Fiasco#1296
fix: replace curl_error_map with switch to fix Static Initialization Order Fiasco#1296COM8 merged 1 commit intolibcpr:masterfrom
Conversation
a1ae4f4 to
e340073
Compare
e340073 to
fcd8920
Compare
COM8
left a comment
There was a problem hiding this comment.
@MarkRakhmatov thanks for contributing and fixing this! How did you debug the cause there? I also noticed this issue in the NuGet debug build but only when using msvc and I was unable identify it.
You are welcome :) I saw this issue after refactoring tests and moving them from the main function to global test suites. I know that such strange lifetime bugs usually occur when you access object that is not initialized yet, so first assumption was Static Initialization Order Fiasco By the way, linter was suppressed for |
Closes #1295