Skip to content

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
@jensmaurer jensmaurer force-pushed the motions-2022-02-cwg-1 branch from 45b8ed7 to e941a31 Compare February 15, 2022 21:45
@jensmaurer jensmaurer requested a review from tkoeppe February 15, 2022 21:46
@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