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

[format.formattable] Add the second template argument for basic_format_context LWG3925 #6264

Closed
wants to merge 1 commit into from

Conversation

cpplearner
Copy link
Contributor

basic_format_context has two template parameters:

template<class Out, class charT> class basic_format_context;

But there's only one template argument in basic_format_context<fmt-iter-for<charT>>.

This seems to be a typo in the resolution of LWG3631.

@Dani-Hub
Copy link
Member

Dani-Hub commented Jun 1, 2023

Please note that this is LWG 3925

@jensmaurer jensmaurer changed the title [format.formattable] Add the second template argument for basic_format_context [format.formattable] Add the second template argument for basic_format_context LWG3925 Jun 1, 2023
@jensmaurer jensmaurer added the lwg Issue must be reviewed by LWG. label Jun 1, 2023
@jensmaurer
Copy link
Member

Since this is handled by a "Tentatively Ready" LWG issue, let's not bother with a duplicate editorial issue.

@jensmaurer jensmaurer closed this Jun 1, 2023
@cpplearner cpplearner deleted the patch-2 branch June 1, 2023 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lwg Issue must be reviewed by LWG.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants