KEMBAR78
Implement LWG-4061 Should `std::basic_format_context` be default-constructible/copyable/movable? by frederick-vs-ja · Pull Request #4758 · microsoft/STL · GitHub
Skip to content

Conversation

frederick-vs-ja
Copy link
Contributor

@frederick-vs-ja frederick-vs-ja commented Jun 30, 2024

See LWG-4061 and WG21-P3341R0.

Blocked libcxx test

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

@frederick-vs-ja 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
Copy link
Contributor Author

@frederick-vs-ja frederick-vs-ja Jun 30, 2024

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.

@StephanTLavavej StephanTLavavej added LWG Library Working Group issue format C++20/23 format labels Jun 30, 2024
@StephanTLavavej StephanTLavavej self-assigned this Jun 30, 2024
@StephanTLavavej StephanTLavavej removed their assignment Jul 3, 2024
@StephanTLavavej StephanTLavavej self-assigned this Jul 4, 2024
@StephanTLavavej
Copy link
Member

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

@StephanTLavavej StephanTLavavej merged commit 7e24bc4 into microsoft:main Jul 5, 2024
@StephanTLavavej
Copy link
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

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants