Repository navigation
: Fix the default constructor of error_category conditionally - #3139
Stephan T. Lavavej (StephanTLavavej) merged 10 commits into
Conversation
std/diagnostics/syserr/syserr.errcat/syserr.errcat.nonvirtuals/default_ctor.pass.cpp
nicole mazzuca (strega-nil-ms)
left a comment
There was a problem hiding this comment.
I don't love that we have to do this to keep ABI, but this is certainly better than the alternative where this isn't constexpr.
|
Thanks, this is awesome! 😻 I think we can regard #1116 as completely fixed by this ( I pushed a conflict-free and no-reformatting-needed merge with |
|
I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed. |
|
I've pushed an additional commit to silence a warning found by internal testing. |
|
Thanks for fixing this bug we thought was unfixable! 🐞 🛠️ 😻 |
Fixes #3119.
Addresses #1116 (fixes it except for
/clr:pure) and unblocks one libcxx test.This PR would complicate Vinnie Falco (@vinniefalco)'s #3044 if it gets merged.