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

Some exposition-only types are not formatted as such in the index #5832

Closed
jwakely opened this issue Sep 14, 2022 · 8 comments · Fixed by #5849
Closed

Some exposition-only types are not formatted as such in the index #5832

jwakely opened this issue Sep 14, 2022 · 8 comments · Fixed by #5849
Assignees

Comments

@jwakely
Copy link
Member

jwakely commented Sep 14, 2022

See chunk_view's inner-iterator and outer-iterator and sentinel:

image

And adjacent_transform_view and adjacent_view's iterator and sentinel:

image

And the index entries for their base() member of various iterator and sentinel types:

image
image

And so on ...

image

image

image

image

image

@jensmaurer
Copy link
Member

We don't want to index exposition-only names at all, I think, because those are not names that could appear in user programs.

So, exposition-only member functions should not be indexed.

For exposition-only classes, we do want to index them, if/when they contain user-visible member functions (e.g. operators in case of views).

@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 24, 2022

@jensmaurer What about exposition-only members such as move_only_function::is-callable-from?

And namespace-scope variable templates such as is-vector-bool-reference?

@jensmaurer
Copy link
Member

All kebab-case names should be in exposition-italics. If you notice any that aren't, we should fix them. But those exposition-only names cannot ever be called by users, so they shouldn't appear in the index of library names.

@W-E-Brown
Copy link
Contributor

W-E-Brown commented Sep 24, 2022 via email

@jensmaurer
Copy link
Member

@tkoeppe, we need a new issue here that we can mark "decision-required". I do agree the exposition-only names should be in some index (or several?); the question is which one. I'm not strongly opposed to the library names index if we add some intro paragraph explaining the typeface.

@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 24, 2022

We can open such an issue, but for now we have four straggling exposition-only names that remain in the library index, in the wrong place (namely right at the start), which doesn't seem all that useful, either.

Maybe we could put all the exposition-only names into the existing library index, but into a separate section?

@jensmaurer
Copy link
Member

We should sort them in the proper alphabetical order; anything else amounts to a special index that nobody wants.

@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 24, 2022

The library index also serves as a list of reserved names (e.g. for the purpose of defining macros) so maybe there's some value in not "polluting" that list with names that don't affect actual programs, but only exist for the purpose of specification?

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 a pull request may close this issue.

4 participants