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

[lex.string,dcl.init.string] Use 'code unit' for initialization from a string-literal #5118

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jensmaurer
Copy link
Member

[lex.string] defines a sequence of code unit values
for a string-literal. Use that sequence for the
initialization of an array by a string-literal.

Fixes #5103

…a string-literal

[lex.string] defines a sequence of code unit values
for a string-literal.  Use that sequence for the
initialization of an array by a string-literal.
characters of the
value of the \grammarterm{string-literal}
initialize the elements of the array.
The elements of the array are initialized
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jensmaurer Please look at this comment. values of the sequence of code unit values sounds a bit vague; what value is that?

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Jan 23, 2022
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Aug 19, 2022
@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 22, 2022

Is this change still useful after the recent encoding-related papers?

@jensmaurer
Copy link
Member Author

I think there is still value in a bit of wording enhancement here, but the words need a bit of work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Changes to the wording or approach have been requested and not yet applied. needs rebase The pull request needs a git rebase to resolve merge conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[dcl.init.string] p1 Successive characters initializing array elements make nosense after P2314
4 participants