Skip to content
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.

Sign up for GitHub

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

Jump to bottom

P2585R1 Improve Default Container Formatting #2941

Closed
StephanTLavavej opened this issue Jul 25, 2022 · 3 comments
Closed

P2585R1 Improve Default Container Formatting #2941

StephanTLavavej opened this issue Jul 25, 2022 · 3 comments
Labels
duplicate This issue or pull request already exists

Comments

@StephanTLavavej
Copy link
Member

P2585R1 Improve Default Container Formatting

Increased feature-test macro (expected):
Also updated by P2286R8 Formatting Ranges
Also updated by P2419R2 Clarify Handling Of Encodings In Localized Formatting Of chrono Types
Also updated by P2508R1 basic_format_string, format_string, wformat_string

#define __cpp_lib_format 202207L

This is reportedly strongly coupled to P2286R8 Formatting Ranges and they should probably be implemented simultaneously.
@StephanTLavavej StephanTLavavej added the cxx23 C++23 feature label Jul 25, 2022
@CaseyCarter
Copy link
Contributor

CaseyCarter commented Jul 25, 2022 •
edited by StephanTLavavej
Loading

P2585R1 Improve Default Container Formatting

Increased feature-test macro (expected): Also updated by P2286R8 Formatting Ranges Also updated by P2419R2 Clarify Handling Of Encodings In Localized Formatting Of chrono Types Also updated by P2508R1 basic_format_string, format_string, wformat_string

#define __cpp_lib_format 202207L

This is reportedly strongly coupled to P2286R8 Formatting Ranges and they should probably be implemented simultaneously.

Yes, this is a patch paper for P2286.

@StephanTLavavej
Copy link
Member Author

Thanks, I will update the issues accordingly.

@StephanTLavavej StephanTLavavej added duplicate This issue or pull request already exists and removed cxx23 C++23 feature labels Jul 25, 2022
@StephanTLavavej
Copy link
Member Author

Fused into #2919. I have removed the cxx23 label so that this doesn't affect the Status Chart.

@StephanTLavavej StephanTLavavej closed this as not planned Won't fix, can't repro, duplicate, stale Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@CaseyCarter @StephanTLavavej