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.

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

FR-008-011 22.14 [format] Support formatting of thread::id #410

Closed
wg21bot opened this issue Oct 23, 2022 · 5 comments · Fixed by cplusplus/draft#6147
Closed

FR-008-011 22.14 [format] Support formatting of thread::id #410

wg21bot opened this issue Oct 23, 2022 · 5 comments · Fixed by cplusplus/draft#6147

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Oct 23, 2022

std::format is not fully integrated with the rest of the standard library.

Please consider adopting P1636R2 so that format and print can be used with standard library types, especially thread::id which is often used for logging purposes. (Integration with filesystem::path has unresolved complexity and should probably be deferred)

@wg21bot wg21bot added the LEWG Library Evolution label Oct 23, 2022
@wg21bot wg21bot added this to the CD C++23 milestone Oct 23, 2022
@jensmaurer jensmaurer changed the title FR 22.14 [format] Support formatting of thread::id FR-008-011 22.14 [format] Support formatting of thread::id Nov 3, 2022
@brycelelbach
Copy link

The proposed resolution is to adopt P1636 cplusplus/papers#425, which is somewhat stalled, but I think we can look at it anyways and cut it down to just the parts we want.

@brycelelbach
Copy link

The updated paper for this NB comment is cplusplus/papers#1358

@tahonermann tahonermann added the SG16 Unicode label Nov 29, 2022
@tahonermann
Copy link
Collaborator

SG16 reviewed P2693R0 (Formatting thread::id and stacktrace) during its 2022-11-30 telecon. The following poll was taken.

  • Poll 2: P2693R0: Forward to LEWG as the recommended resolution of FR-008-011.
    • Attendees: 10
    • No objection to unanimous consent.

@brycelelbach, I am removing the SG16 label; this issue is ready for LEWG review.

@tahonermann tahonermann removed the SG16 Unicode label Dec 20, 2022
@brycelelbach brycelelbach added LWG Library and removed LEWG Library Evolution labels Jan 24, 2023
@JeffGarland
Copy link
Member

Reviewed and approved at Issaquah -- see paper issue for details.

@jwakely
Copy link
Member

jwakely commented Mar 23, 2023

Accepted with modification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Ready
Development

Successfully merging a pull request may close this issue.

5 participants