Skip to content

[cmp, comparisons] Move description of compare_three_way. #3670

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

Merged
merged 1 commit into from
Feb 14, 2020

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Feb 11, 2020

Moves the definition and specification of class compare_three_way from
[cmp, 17.11] to [comparisons, 20.14.7].

This is a partial response to NB GB 175 (C++20 CD).

Fixes cplusplus/nbballot#173.

@tkoeppe tkoeppe requested a review from jensmaurer February 11, 2020 15:18
Moves the definition and specification of class compare_three_way from
[cmp, 17.11] to [comparisons, 20.14.7]. Also adds a declaration of this
class to the synopsis of <functional> and removes a paragraph that says
(now redundantly) that the class is available also via inclusion of
<functional>.

This is a partial response to NB GB 175 (C++20 CD).
@tkoeppe
Copy link
Contributor Author

tkoeppe commented Feb 13, 2020

@jensmaurer: Would you like to review this PR?

@jensmaurer jensmaurer added this to the C++20 milestone Feb 13, 2020
@tkoeppe
Copy link
Contributor Author

tkoeppe commented Feb 13, 2020

Thanks! Do you think we can merge this now? I don't see any upcoming core issues for this section?

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Feb 13, 2020

@jensmaurer, @zygoloid, @mclow, @jwakely: in LWG review it emerged that the pattern of "In addition to ... available when the header <foo> is included" is fairly widely used (as opposed to repeating declarations in synopses). We may like to consider a review of the situation.

@zygoloid zygoloid merged commit 7918bc2 into cplusplus:master Feb 14, 2020
@tkoeppe tkoeppe deleted the cmp3 branch February 14, 2020 10:06
@tkoeppe
Copy link
Contributor Author

tkoeppe commented Feb 14, 2020

Note to selves: the other places that use "in addition..." wording are:

tuple.helper p6, p8
iterator.range p1
range.access p1

@jwakely
Copy link
Member

jwakely commented Feb 15, 2020

Feel free to assign an issue to me, asking to unify how we say that.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Feb 15, 2020

@jwakely: done, this is now #3683.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GB175 17.11.06 Move [cmp.object] to [comparisons]
4 participants