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

P2533R0 Core Language Working Group "ready" #5287

Merged
merged 7 commits into from Feb 16, 2022
Merged

Conversation

jensmaurer
Copy link
Member

@jensmaurer jensmaurer commented Feb 10, 2022

Fixes #5257
Fixes #4843
Fixes #2338
Fixes cplusplus/papers#1196

@jensmaurer jensmaurer added this to the post-2022-02 milestone Feb 10, 2022
@jensmaurer jensmaurer changed the title P2533R0 (Core Language Working Group "ready" P2533R0 Core Language Working Group "ready" Feb 11, 2022
source/classes.tex Outdated Show resolved Hide resolved
source/classes.tex Show resolved Hide resolved
source/expressions.tex Outdated Show resolved Hide resolved
@tkoeppe tkoeppe merged commit 21dc0d3 into main Feb 16, 2022
\end{itemize}

\pnum
A conversion function shall have no non-object parameters and
Copy link
Contributor

Choose a reason for hiding this comment

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

The core issue doesn't include "non-object".

I suppose you added it to account for explicit object parameters.

Is that really enough? No need to restrict it to an explicit object parameter, but any amount of non-object parameters is OK? Is there wording elsewhere that make these cases ill-formed, and thus saying "non-object" is enough?

How about the function type specified below? Does it need a revision, too?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think the paper misrepresents the status quo? So tihs seems like a "plausible merge".

Copy link
Member Author

Choose a reason for hiding this comment

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

A conversion function with an explicit this parameter seems plausible, but there must not be other parameters. Further cleanup (if any) should be in a separate editorial or core issue.

@jensmaurer jensmaurer deleted the motions-2022-02-cwg-1 branch July 29, 2022 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants