-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
<system_error>
: Fix the default constructor of error_category
conditionally
#3139
<system_error>
: Fix the default constructor of error_category
conditionally
#3139
Conversation
std/diagnostics/syserr/syserr.errcat/syserr.errcat.nonvirtuals/default_ctor.pass.cpp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
tests/std/tests/GH_003119_error_category_ctor/test.compile.pass.cpp
Outdated
Show resolved
Hide resolved
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 @vinniefalco's #3044 if it gets merged.