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

[ostreambuf.iter.ops] not divided into subsections as [istreambuf.iter.ops] #1429

Closed
lynnboy opened this issue Feb 5, 2017 · 11 comments
Closed
Assignees

Comments

@lynnboy
Copy link

lynnboy commented Feb 5, 2017

there are several [istreambuf.iterator::op...] sections under [istreambuf.iterator],
but there is only a combined [ostreambuf.iter.ops] section under [ostreambuf.iterator].
This is inconsistent.

Consider split it into subsections.

Also: consider change :: into . and * into star in section labels

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 5, 2017

I don't think we are going to change stable names.

@lynnboy
Copy link
Author

lynnboy commented Feb 5, 2017

But we've already made :: -> . for [string.xxx].

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 5, 2017

@lynnboy: Wasn't that for new content post-C++14? Things that were added in this revision can still be changed, indeed.

@cpplearner
Copy link
Contributor

The change from [string::xxx] to [string.xxx] is made in 0b5bcd5.

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 5, 2017

I see. I guess we do change stable names then...

@jensmaurer
Copy link
Member

jensmaurer commented Feb 8, 2017

The stable names are really bad, and those single-paragraph descriptions should never have gotten separate sections to start with.

jensmaurer added a commit to jensmaurer/draft that referenced this issue Feb 8, 2017
There was one subsection for every operator, yet everything
fits on half a page.

Fixes cplusplus#1429.
@jensmaurer jensmaurer self-assigned this Feb 8, 2017
@Eelis
Copy link
Contributor

Eelis commented Feb 10, 2017

[reverse.iter.ops] also has many single-paragraph subsections for individual operators. So possibly that one could be cleaned up as well.

@jensmaurer
Copy link
Member

This seems to be a pervasive issue in that area; see [front.insert.iter.ops] and [back.insert.iter.ops]. The [reverse.iter.ops] that you mentioned are more than half a page, though, so some kind of subdivision (e.g. "comparison operators", "arithmetic") might be worthwhile.

In short, all of [predef.iterators] is affected, and we should come up with a common approach for that subsection instead of fixing individual issues.

@zygoloid, what do you think?

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Feb 10, 2017
@jensmaurer
Copy link
Member

Editorial committee consensus: Fold mini-sections for operators and rename "::" in stable names.

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Mar 2, 2017
jensmaurer added a commit to jensmaurer/draft that referenced this issue Mar 18, 2017
There was one subsection for every operator, yet everything
fits on half a page.

Fixes cplusplus#1429.
jensmaurer added a commit to jensmaurer/draft that referenced this issue Mar 20, 2017
There was one subsection for every operator, yet everything
fits on half a page.

Fixes cplusplus#1429.
@tkoeppe
Copy link
Contributor

tkoeppe commented Mar 20, 2017

@jensmaurer: Did we mean to close this issue? Didn't you say there were other aspects to it not yet dealt with by pull requests?

@jensmaurer jensmaurer reopened this Mar 20, 2017
@jensmaurer
Copy link
Member

In short, all of [predef.iterators] is affected, and we should come up with a common approach for that subsection instead of fixing individual issues.

Editorial committee consensus: Fold mini-sections for operators and rename "::" in stable names.

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

No branches or pull requests

5 participants