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

[gram] Change "syntax" to "grammar". #790

Merged
merged 1 commit into from Nov 12, 2016
Merged

Conversation

FrankHB
Copy link
Contributor

@FrankHB FrankHB commented Jul 3, 2016

Annex A is a summary of grammar, but not only syntactic grammar. Even in contexts without need of grammatical disambiguation, pure syntactically handling (reduction merely based on parsing of the token stream without semantic information) of several context-sensitive constructs (e.g. constant-expression and several kinds of type-id) has already been insufficient. Since the remained difference is acknowledged in the same paragraph, I think my change is also editorial. (On the contrary, ISO C may have more problems because it uses "syntax" everywhere.)

However, I have no idea to change \indextext{summary!syntax} ...

@zygoloid zygoloid force-pushed the master branch 2 times, most recently from 6f5c913 to bbb0244 Compare July 7, 2016 23:35
@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 9, 2016

@FrankHB: Can you please rebase this PR?

@FrankHB FrankHB force-pushed the patch-12 branch 2 times, most recently from c219931 to a3537ea Compare July 9, 2016 12:38
@@ -5,7 +5,7 @@
\pnum
\indextext{grammar}%
\indextext{summary!syntax}%
This summary of \Cpp\ syntax is intended to be an aid to comprehension.
This summary of \Cpp\ grammar is intended to be an aid to comprehension.
Copy link
Contributor

@tkoeppe tkoeppe Jul 11, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, this is actually in the wrong file, since this one is auto-generated from tools/grambase.tex.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, didn't realize that...

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 18, 2016

(This would need to be rebased now.)

Annex A is a summary of grammar, but not only syntactic grammar. Even in contexts without need of grammatical disambiguation, pure syntactically handling (reduction merely based on parsing of the token stream without semantic information) of several context-sensitive constructs (e.g. constant-expression and several kinds of type-id) has already been insufficient. Since the remained difference is acknowledged in the same paragraph, I think my change is also editorial. (On the contrary, ISO C may have more problems because it uses "syntax" everywhere.)
@FrankHB
Copy link
Contributor Author

FrankHB commented Aug 2, 2016

Done. Now the original change is in the right place?

@tkoeppe tkoeppe merged commit fa639c4 into cplusplus:master Nov 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants