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

[container.adapters] The flat_* container adapters need to index their members #6221

Open
AlisdairM opened this issue Apr 4, 2023 · 2 comments
Open

[container.adapters] The flat_* container adapters need to index their members #6221

AlisdairM opened this issue Apr 4, 2023 · 2 comments

Comments

@AlisdairM
Copy link
Contributor

A bulk of the specification for the flat_* container adapters is in the container requirements clauses, as they are both containers and adapters. However, they are not listed for indexing in the clauses for containers, reversible containers, or associative containers.

I quickly hacked up a patch to do exactly that, but there were too many double-references where some members DO have "otherwise" documentation, so filing an issue as I do not have time to clean up properly right now.
@Quuxplusone
Copy link
Contributor

I'd be willing to take this on as part of P2767, since it'll merge-conflict heavily no matter what.

@jensmaurer
Copy link
Member

Since indexing usually doesn't show up in any papers, I'd prefer to keep the indexing fixes as an editorial pull request. I don't think there will be much conflict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@AlisdairM @Quuxplusone @jensmaurer