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
Rewrite of Syntax-Identifier section #2336
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I (kind of) see most of the comments have been addressed. Please take a look at these, but by all means do not close the PR, just add new commits and make comments on how you are addressing them. You've done great work, thanks for the contribution.
doc/Language/syntax.pod6
Outdated
for more details. | ||
In the C<x²> example above, since the Unicode superscript numeral C<²> is not | ||
alphanumeric, it cannot be a part of an ordinary identifier. Still, it may form | ||
part of a valid expression as an operator. The expression C<$x²>, for instance, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as it's clear it's not alphanumeric, the rest of the paragraph is probably not needed in a section about identifiers, not even as a teaser for the other section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And since it is the first comment, I would really have appreciated if you had not closed the other PR, added new commits addressing (or not) my comments, which you apparently have to a extent. Please do so this time; just push new commits, and comment responding to these comments.
doc/Language/syntax.pod6
Outdated
party:$a # ...or $a | ||
=end code | ||
N<As the examples illustrate, the C<key> I<or> the quoted C<value> is optional. | ||
Colon pairs with a null key are normally used for naming operators, as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should have said so in the description above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see my note about :skip-test
The problem
Identifier-section was perceived as a bit messy.
Solution provided
Restructured section along the lines of the different types of identifiers: