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.arg,filebuf.members] Invalid uses of NTCTS #4185

Open
JohelEGP opened this issue Sep 10, 2020 · 4 comments
Open

[format.arg,filebuf.members] Invalid uses of NTCTS #4185

JohelEGP opened this issue Sep 10, 2020 · 4 comments
Labels
lwg Issue must be reviewed by LWG.

Comments

@JohelEGP
Copy link
Contributor

For [format.arg], when replacing NTCTS with its definition, there's no charT in context.
For [filebuf.members], there's a charT in context, but its type might differ from the character type of s.

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Feb 7, 2021
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 19, 2021

We agree that the current definition of "NTCTS" could be improved. We need some analysis where it comes from and what it should have meant, and improve the wording.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 19, 2021

Notes:

  • We need to spell out what the abbreviation means.
  • Need to be clear that the term depends on a template parameter.
  • Can we use small-caps somehow?

@jensmaurer
Copy link
Member

Move the definition to [character.seq] as a separate subsection. Spend more than a half-sentence on the definition; avoid use of charT(), which is not in scope anyway. Null characters are encoded as the 0 integer value anyway, so we don't need to jump through extra hoops. Use small-caps.

@jensmaurer jensmaurer added lwg Issue must be reviewed by LWG. and removed decision-required A decision of the editorial group (or the Project Editor) is required. labels Nov 19, 2021
@jwakely
Copy link
Member

jwakely commented Nov 19, 2021

I think the term originated in the Filesystem TS, if I recall correctly.

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

No branches or pull requests

4 participants