-
Notifications
You must be signed in to change notification settings - Fork 5
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
Explicitly disallow unnamed Unicode codepoints in http://eel.is/c++draft/lex.charset#2 #8
Comments
After some review, I have the feeling this stems from an outdated 10646 version. The current version doesn't have a "short name" concept (well, except for jamo short names, but that's clearly not the intention here). There's a "short identifier" concept (in 6.5) but that cannot be more than six digits (the familiar U+ syntax). I propose the following.
(Ab)use of the C-word notwithstanding. |
Also, IIUC, all short identifiers are valid even if they refer to a reserved codepoint. So you may be indicating something meaningless, but well defined meaningless. |
At Rapperswil I discussed this issue with people from Core and it was agreed that the current wording with "short name" will be fixed editorially. We didn't discuss whether we should explicitly make those without such a short identifier illformed. |
That's good. I think the right process is to submit a PR to https://github.com/cplusplus/draft. Anyone want to volunteer to do so? |
I can do that. |
This PR cplusplus/draft#2201 will be merged fixing the "short name" issue editorially. As discussed with Jens, I will write a short paper to clean up the wording there and clarify the nonexistent code point case. |
@rmartinho I think this issue is complete, yes? |
Ah, no, this isn't complete yet. The accepted editorial PR only addressed the "short name" vs "short identifier" terminology issue. Martinho's draft D1139 (which I think is yet to be submitted to a mailing) addresses the concern tracked by this issue. |
Closing this issue as resolved following the adoption of P1139R2 in Kona. |
http://eel.is/c++draft/lex.charset#2
Says:
Should it also say:
Or is that implied?
The text was updated successfully, but these errors were encountered: