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

[over.call.object] Reference postfix-expression in call syntax correctly #1683

Conversation

hubert-reinterpretcast
Copy link
Contributor

The grammar production corresponding to the call syntax has postfix-expression as the callee operand. [over.call.object] appears to refer to this operand as the primary-expression in the syntax.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 25, 2017

Please fix to make tests pass.

@hubert-reinterpretcast
Copy link
Contributor Author

Only the check-whitespace test was failing, and it seemed that the fix was already pending. In any case, I incorporated the fix in my branch.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 25, 2017

Yes, the fix is to rebase on master.

@hubert-reinterpretcast
Copy link
Contributor Author

Okay; I don't recall having tried force pushing to a branch with a pull request active before. First time for everything.

The grammar production corresponding to the call syntax has
_postfix-expression_ as the callee operand. [over.call.object] appears
to refer to this operand as the _primary-expression_ in the syntax.
@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 25, 2017

Oh boy, get used to force pushing to PR branches all the time as you start interacting with the OCD nitpick master editors over here :-)

@hubert-reinterpretcast
Copy link
Contributor Author

Is that (OCD nitpick)+(master) editors, (OCD)+(nitpick master) editors, ((OCD nitpick) master) editors, something else, all, or some? :-)

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 25, 2017

Absolutely all of the above!

@zygoloid zygoloid merged commit 10e4c74 into cplusplus:master Jul 30, 2017
@hubert-reinterpretcast hubert-reinterpretcast deleted the over_call_object_p1_grammar_ref_fix branch July 30, 2017 20:59
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

3 participants