Skip to content

P2533R0 Core Language Working Group "ready" - #5287

Merged
tkoeppe merged 7 commits into
mainfrom
motions-2022-02-cwg-1
Feb 16, 2022
Merged

tkoeppe merged 7 commits into
mainfrom
motions-2022-02-cwg-1

Conversation

@jensmaurer

@jensmaurer jensmaurer commented Feb 10, 2022 •

Copy link
Copy Markdown
Member

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
Comment thread source/classes.tex Outdated
Comment thread source/classes.tex
Comment thread source/expressions.tex Outdated
@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
Comment thread source/classes.tex
\end{itemize}

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

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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