Skip to content

Implement LWG-4061 Should std::basic_format_context be default-constructible/copyable/movable? - #4758

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
frederick-vs-ja:lwg-4061
Jul 5, 2024
Merged

Implement LWG-4061 Should std::basic_format_context be default-constructible/copyable/movable?#4758
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
frederick-vs-ja:lwg-4061

Conversation

@frederick-vs-ja

@frederick-vs-ja A. Jiang (frederick-vs-ja) commented Jun 30, 2024

Copy link
Copy Markdown
Contributor

See LWG-4061 and WG21-P3341R0.

Blocked libcxx test

  • std/utilities/format/format.functions/bug_81590.compile.pass.cpp

@frederick-vs-ja
A. Jiang (frederick-vs-ja) requested a review from a team as a code owner June 30, 2024 15:38
std/ranges/range.adaptors/range.lazy.split/range.lazy.split.outer.value/ctor.iter.pass.cpp FAIL

# libc++ doesn't implement LWG-4061
std/utilities/format/format.functions/bug_81590.compile.pass.cpp FAIL

@frederick-vs-ja A. Jiang (frederick-vs-ja) Jun 30, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test is weird. It has a formatter specialization whose format function passes basic_format_context by value, which is unexpected.

Comment thread stl/inc/format
@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 7e24bc4 into microsoft:main Jul 5, 2024
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for implementing this LWG issue resolution! Resolve all the things! 🪄 🎉 🐈

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

Labels

format C++20/23 format LWG Library Working Group issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants