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
Quoted #2463
Quoted #2463
Conversation
chapters/lexicalstructure.tex
Outdated
The single quotes are part of the identifier, i.e., \lstinline!'x'! and \lstinline!x! | ||
are distinct identifiers, but the redundant escapes (\lstinline!'\?'! and \lstinline!'\"'!) are the same as the corresponding non-escaped | ||
variants (\lstinline!'?'! and \lstinline!'"'!). A full BNF definition of the Modelica syntax and | ||
variants (\lstinline!'?'! and \lstinline!'"'!), and outside of Modelica source code the redundant escapes shall not be used. |
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.
Might be easier to grasp if that long sentence is split in two or three parts.
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.
Agreed, and I tried to do that.
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 agree with intentions. Just some comments on wording.
Try to clarify single-quoted identifiers.
Closes #2408
Since this is a straightforward but non-trivial change, I want that in addition to the review that you vote thumbs up to accept or thumbs down if you don't think it should be accepted.
The poll is running to 2019-12-19.