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.

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

P0472R3 Put std::monostate in <utility> #7700

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Eisenwave
Copy link
Contributor

@Eisenwave Eisenwave commented Feb 19, 2025

Fixes #7663.
Fixes cplusplus/papers#1993.

New Wording

image

Paper Wording

image

@jwakely
Copy link
Member

jwakely commented Feb 19, 2025

I think there should be an index entry.

@Eisenwave
Copy link
Contributor Author

I think there should be an index entry.

An index entry for what specifically? monostate already has index entries for the class, comparison operators, and the hash specialization. These are found in [variant.monostate] etc.

Do you mean that there should be redundant index entries that also point to these declarations in [utility.syn]? Do we normally index the synopsis instead of the place of definition?

@jwakely
Copy link
Member

jwakely commented Feb 19, 2025

We don't, but it's not redundant here because it is a new occurrence of the name in a different place.

If the class is declared in two different places, I would expect two index entries.

@Eisenwave Eisenwave force-pushed the motions-2025-02-lwg-4 branch from 563a74d to 67e5510 Compare February 19, 2025 13:41
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.

[2025-02 LWG Motion 4] P0472R3 Put std::monostate in ⟨utility⟩ P0472 R3 Put std::monostate in <utility>
2 participants