masm/bnf-grammar: Format some literal characters correctly#4456
masm/bnf-grammar: Format some literal characters correctly#4456TylerMSFT merged 1 commit intoMicrosoftDocs:mainfrom
Conversation
A couple of character literal terminals don't follow consistent formatting. Fix that.
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
4 similar comments
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit b3dc4de: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
TylerMSFT
left a comment
TylerMSFT
left a comment
There was a problem hiding this comment.
I'm on the fence about the value of these changes. One of the reasons we do code escape is to prevent machine translation of identifiers. But that's not an issue with parenthesis, etc.
I'll take the change since the work is done, but I wouldn't necessarily encourage further work like this unless it is to prevent machine translation from incorrectly modifying a code statement.
|
#sign-off |
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
1 similar comment
|
@lkundrak : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
TylerMSFT
left a comment
TylerMSFT
left a comment
There was a problem hiding this comment.
Hi, thanks for the changes.
2 participants
A couple of character literal terminals don't follow consistent formatting. Fix that.