[class.copy.assign] Remove a superfluous note. #6619

Quuxplusone · 2023-10-12T22:27:36Z

Alternatively we could have added the word "non-object"; or changed it to say "An overloaded assignment operator must be a member function"; but it doesn't seem like it needs to be here at all.

(Clang, EDG, MSVC all agree that B& operator=(this B, int); is legal C++23.)

Alternatively we could have added the word "non-object"; or changed it to say "An overloaded assignment operator must be a member function"; but it doesn't seem like it needs to be here at all.

jensmaurer

This looks fine; the intent seems obvious and we already say "non-object parameter" when we define "copy assignment operator", so explicit object parameters were in scope (and intended to be accepted) when the words were last changed.

Quuxplusone added 2 commits October 12, 2023 18:24

[class.copy.assign] Remove a superfluous note.

8459943

Alternatively we could have added the word "non-object"; or changed it to say "An overloaded assignment operator must be a member function"; but it doesn't seem like it needs to be here at all.

[class.copy.assign] Add some missing "non-object"s

2cf8e6a

jensmaurer approved these changes Oct 15, 2023

View reviewed changes

jensmaurer assigned tkoeppe Oct 15, 2023

tkoeppe merged commit c9c69dc into cplusplus:main Nov 8, 2023
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[class.copy.assign] Remove a superfluous note. #6619

[class.copy.assign] Remove a superfluous note. #6619

Quuxplusone commented Oct 12, 2023

jensmaurer left a comment •

edited

[class.copy.assign] Remove a superfluous note. #6619

[class.copy.assign] Remove a superfluous note. #6619

Conversation

Quuxplusone commented Oct 12, 2023

jensmaurer left a comment • edited

Choose a reason for hiding this comment

jensmaurer left a comment •

edited