Skip to content

Implement P0472R3 - Put monostate in <utility> - #5382

Merged
Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
yehudamber:p0472
Apr 10, 2025
Merged

Implement P0472R3 - Put monostate in <utility>#5382
Stephan T. Lavavej (StephanTLavavej) merged 3 commits into
microsoft:mainfrom
yehudamber:p0472

Conversation

@yehudamber

Copy link
Copy Markdown
Contributor

Move monostate (from <xutility>) and related definitions (from <variant>) to <utility>.
<utility> is included by both headers, so no code is broken.

resolves #5294

@yehudamber

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks! I pushed an additional commit to follow our convention of listing all implemented features in <yvals_core.h>.

This change is sufficiently simple and resistant to accidental damage that no test coverage is necessary.

We merge PRs simultaneously to our GitHub and MSVC-internal repos in a semi-manual process, batched up to save time. Your PR will be part of the next batch, likely either this week or next week. I'll post comments here as I prepare your PR for merging!

Comment thread stl/inc/utility
@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 41d6925 into microsoft:main Apr 10, 2025
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Apr 10, 2025
@StephanTLavavej

Stephan T. Lavavej (StephanTLavavej) commented Apr 10, 2025

Copy link
Copy Markdown
Member

Thanks for implementing this C++26 feature, and congratulations on your first microsoft/STL commit! 😻 🚀 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cxx26 C++26 feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

P0472R3 Put monostate In <utility>

3 participants