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

[mdspan.extents.expo] remove redundant // \expos #5907

Merged
merged 1 commit into from Nov 8, 2022

Conversation

hewillk
Copy link
Contributor

@hewillk hewillk commented Oct 18, 2022

No description provided.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 7, 2022

I don't think this is the only place where we say // \expos in an itemdecl; why do you want to remove it just in these few cases? I think we might want a more general discussion/decision if we want to make any changes here, but I also don't consider that terribly pressing.

@tkoeppe tkoeppe added the decision-required A decision of the editorial group (or the Project Editor) is required. label Nov 7, 2022
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 8, 2022

Editorial meeting consensus: See also #5498, we seem to have agreed to have \expos comments only in the synopsis, and never later on. We should record that in the wiki and clean up. This PR is a good part of that.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 8, 2022

Added to the wiki.

@tkoeppe tkoeppe removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Nov 8, 2022
@tkoeppe tkoeppe merged commit 65c494b into cplusplus:main Nov 8, 2022
@hewillk hewillk deleted the main-mdspan branch November 8, 2022 07:52
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.

None yet

2 participants