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.

Sign up for GitHub

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

Jump to bottom

[cmp.categories.pre] nonequal and nonequivalent are non-used #3541

Closed
StephanTLavavej opened this issue Dec 13, 2019 · 0 comments · Fixed by #3542
Closed

[cmp.categories.pre] nonequal and nonequivalent are non-used #3541

StephanTLavavej opened this issue Dec 13, 2019 · 0 comments · Fixed by #3542
Assignees
@CaseyCarter

Comments

@StephanTLavavej
Copy link
Contributor

After P1959R0 "Remove std::weak_equality and std::strong_equality" was adopted, N4842 17.11.2.1 [cmp.categories.pre]/1 says:

Each is specified in terms of an exposition-only data member named value whose value typically corresponds to that of an enumerator from one of the following exposition-only enumerations:
enum class eq { equal = 0, equivalent = equal, nonequal = 1, nonequivalent = nonequal }; // exposition only

However, nonequal and nonequivalent are now unused. As they're exposition-only enumerators, they should be editorially removed.
CaseyCarter added a commit to CaseyCarter/draft that referenced this issue Dec 13, 2019
@CaseyCarter
[cmp.categories.pre] Remove unused enumerators
091fe2b 
Fixes cplusplus#3541.
@CaseyCarter CaseyCarter mentioned this issue Dec 13, 2019
Merged
zygoloid pushed a commit that referenced this issue Sep 18, 2020
@CaseyCarter @zygoloid
[cmp.categories.pre] Remove unused enumerators
3cc8595 
Fixes #3541.
zygoloid pushed a commit that referenced this issue Sep 18, 2020
@CaseyCarter @zygoloid
[cmp.categories.pre] Remove unused enumerators
9740a66 
Fixes #3541.
tkoeppe pushed a commit that referenced this issue Sep 22, 2020
@CaseyCarter @tkoeppe
[cmp.categories.pre] Remove unused enumerators
94914fd 
Fixes #3541.
tkoeppe pushed a commit that referenced this issue Sep 24, 2020
@CaseyCarter @tkoeppe
[cmp.categories.pre] Remove unused enumerators
6449dfa 
Fixes #3541.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CaseyCarter CaseyCarter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[cmp.categories.pre] Remove unused enumerators CaseyCarter/draft
2 participants
@CaseyCarter @StephanTLavavej