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-021-013 5.3p5.2 [lex.charset] Codepoint names in identifiers #423

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

FR-021-013 5.3p5.2 [lex.charset] Codepoint names in identifiers #423

wg21bot opened this issue Oct 23, 2022 · 9 comments · Fixed by cplusplus/draft#6126
Labels
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Oct 23, 2022

The XID_Start and XID_Continue properties that form the identifier grammar are extracted from Unicode. But character names are extracted from ISO 10646.
Because Unicode and ISO 10646 do not have the same release schedule, not all characters valid in identifiers can be spelled with the \N{} syntax.
This inconsistency could be avoided by referring to Unicode directly in the specification of the \N{} escape sequences.

Please refer to Unicode consistently for all Unicode properties, using a consistent set of references pertaining to the same version of Unicode.

@wg21bot wg21bot added the CWG Core label Oct 23, 2022
@wg21bot wg21bot added this to the CD C++23 milestone Oct 23, 2022
@wg21bot wg21bot changed the title FR 5.3 [lex.charset] Codepoint names in identifiers FR 5.3p5.2 [lex.charset] Codepoint names in identifiers Oct 23, 2022
@tahonermann tahonermann added the SG16 Unicode label Oct 25, 2022
@jensmaurer jensmaurer changed the title FR 5.3p5.2 [lex.charset] Codepoint names in identifiers FR-021-013 5.3p5.2 [lex.charset] Codepoint names in identifiers Nov 3, 2022
@tahonermann
Copy link
Collaborator

SG16 reviewed this issue along with FR-010-133 during its 2022-11-02 telecon.

The problem tracked by this issue is a specific consequence of the problem described by FR-010-133. Resolving FR-010-133 such that the standard no longer directly or indirectly references multiple versions of the Unicode Standard will resolve this issue. See the comments in FR-010-133 for more details.

I'm going to retain the SG16 label for now. This issue should be considered blocked on FR-010-133.

@jensmaurer jensmaurer removed the CWG Core label Nov 8, 2022
@tahonermann
Copy link
Collaborator

P2736R0 seeks to address this issue (as well as FR-010-133).

@tahonermann
Copy link
Collaborator

SG16 discussed a draft of P2736R0 during its 2022-12-14 telecon. No polls were taken, discussion will continue at the next SG16 telecon scheduled for 2023-01-11. I'm retaining the SG16 label for now.

@tahonermann
Copy link
Collaborator

SG16 completed its review of a draft of P2736R1 during its 2023-01-25 telecon. The following poll was taken:

  • Poll 1.2: Forward P2736R1, amended as discussed, to CWG and LWG as the recommended resolution of NB comments FR-010-133 and FR-021-013.
    • Attendees: 10 (2 abstentions)
      SF F N A SA
      7 2 0 0 0
    • Unanimous consent.

I'm removing the SG16 label; this NB comment is ready for CWG review.

@tahonermann tahonermann added CWG Core and removed SG16 Unicode labels Jan 26, 2023
@jensmaurer jensmaurer added the LWG Library label Jan 26, 2023
@jensmaurer
Copy link
Member

Adding LWG per the poll in SG16.

@tahonermann
Copy link
Collaborator

The SG16 poll is also recorded in the issue for P2736 which is the paper that SG16 actually polled forwarding (and does address CWG and LWG). I think the particular NB comment this issue tracks is only relevant for CWG though.

@jensmaurer jensmaurer removed the LWG Library label Jan 26, 2023
@jensmaurer
Copy link
Member

Except that LWG won't review a paper in Issaquah (with priority) unless it clearly has relevance for an NB comment. But I notice that #412 is also assigned to LWG, so we're fine.

@JeffGarland
Copy link
Member

LWG is planning to look at #412 in our pre-Issaquah telecom on 2023-02-01

@jensmaurer
Copy link
Member

Accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants