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

[C++17 DIS comment 018] Hanging paragraphs #1730

Closed
zygoloid opened this issue Sep 6, 2017 · 5 comments · Fixed by #4172
Closed

[C++17 DIS comment 018] Hanging paragraphs #1730

zygoloid opened this issue Sep 6, 2017 · 5 comments · Fixed by #4172
Labels
ballot-comment Response to an NB or ISO comment on a ballot
Milestone
C++23

Comments

@zygoloid
Copy link
Member

zygoloid commented Sep 6, 2017

Perhaps we should see how hard ISO pushes for this one. It's probably not too hard to script up the addition of a bunch of "General" headings if we need to, but I would worry that such a heading would be inappropriate in many cases.
@zygoloid zygoloid added the ballot-comment Response to an NB or ISO comment on a ballot label Sep 6, 2017
@zygoloid zygoloid modified the milestone: C++17 IS Sep 6, 2017
@jensmaurer jensmaurer modified the milestones: C++20, C++17 IS Sep 8, 2017
@jensmaurer
Copy link
Member

Agreed: Fixing these for real (e.g. clause [expr]) means more work than adding a bunch of trivial subsection headings. Postpone to C++20.

@zygoloid
Copy link
Member Author

Response: REJECTED. We will consider this change for the next revision of the document, but adding meaningful headings for these paragraphs would require significant reorganization of the text and potentially updating of cross-references, which introduces too much risk for this stage of the standardization process.

@zygoloid zygoloid modified the milestones: C++17 IS, C++20 Sep 15, 2017
@zygoloid
Copy link
Member Author

Leaving this open for consideration post-C++17 IS.

@jensmaurer
Copy link
Member

See also #574 and #1771.

@jensmaurer jensmaurer mentioned this issue Nov 1, 2017
Open
@jensmaurer jensmaurer mentioned this issue Nov 11, 2017
Closed
jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 11, 2017
@jensmaurer
[expr] Add subclauses and adjust cross-references to [expr].
b9033e9 
Partially addresses cplusplus#1730.
jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 12, 2017
@jensmaurer
[expr] Add subclauses and adjust cross-references to [expr].
3e675ce 
Partially addresses cplusplus#1730.
jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 12, 2017
@jensmaurer
[expr] Add subclauses and adjust cross-references to [expr].
0e5f4d1 
Partially addresses cplusplus#1730.
jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 12, 2017
@jensmaurer
[expr] Add subclauses and adjust cross-references to [expr].
5c2a8a8 
Partially addresses cplusplus#1730.
@jensmaurer jensmaurer mentioned this issue Jul 6, 2018
Merged
@jensmaurer jensmaurer mentioned this issue Jun 6, 2019
Merged
@zygoloid zygoloid modified the milestones: C++20, C++23 Feb 10, 2020
@zygoloid
Copy link
Member Author

We have addressed a lot of issues here for C++20, but there's more work to do. Retargeting at C++23.

@jensmaurer jensmaurer mentioned this issue Feb 10, 2020
Closed
21 tasks
@JohelEGP JohelEGP mentioned this issue Sep 9, 2020
Merged
@zygoloid zygoloid mentioned this issue Sep 9, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ballot-comment Response to an NB or ISO comment on a ballot
Projects
None yet
Milestone
C++23
Development

Successfully merging a pull request may close this issue.

Fix all hanging paragraphs by adding "General" sections. zygoloid/cplusplus
2 participants
@zygoloid @jensmaurer