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.match.oper] Add a note for conversions on synthesized candidates. #1927

Merged
merged 1 commit into from Apr 2, 2018

Conversation

jensmaurer
Copy link
Member

Fixes #1926.

@hsutter
Copy link

hsutter commented Feb 15, 2018

👍 And this seems editorial.

@zygoloid
Copy link
Member

[over.match.funcs]p3: "Since arguments and parameters are associated by position within their respective lists, the convention is that the implicit object parameter, if present, is always the first parameter and the implied object argument, if present, is always the first argument."

That's normative wording, and contradicts this note. The note reflects the obvious intent of the wording, though, and the lack of an update to the above wording is just an oversight, so I'm OK with changing this editorially if there's no CWG opposition. Here's what I suggest:

  • Update the wording to also change the above paragraph.
  • Add the CWG label to request that CWG check they're OK with us changing this editorially.

@jensmaurer
Copy link
Member Author

I think we can simply say that a member function gets an extra first parameter when it becomes a candidate. Then, the synthesized candidate is based on that member function candidate and the reversal happens.

@jensmaurer jensmaurer added the cwg Issue must be reviewed by CWG. label Feb 16, 2018
@jensmaurer jensmaurer added this to the C++20 milestone Feb 16, 2018
@jensmaurer
Copy link
Member Author

CWG in Jacksonville: approved.

@jensmaurer jensmaurer removed the cwg Issue must be reviewed by CWG. label Mar 15, 2018
@tkoeppe tkoeppe added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Mar 30, 2018
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Mar 30, 2018
@jensmaurer
Copy link
Member Author

Rebased.

@jensmaurer jensmaurer added the after-motions Pull request is to be applied after the pending edits from WG21 straw polls have been applied. label Apr 1, 2018
@zygoloid zygoloid merged commit 1d50d2d into cplusplus:master Apr 2, 2018
@jensmaurer jensmaurer deleted the b18 branch April 2, 2018 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
after-motions Pull request is to be applied after the pending edits from WG21 straw polls have been applied.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants