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

[temp.deduct.call] Don't nest paragraphs inside eachother. #882

Merged
merged 1 commit into from Aug 5, 2016

Conversation

Eelis
Copy link
Contributor

@Eelis Eelis commented Aug 5, 2016

In this section, paragraphs 7, 8 and 9 are nested /inside/ paragraph 6 (they are part of the second item of the enumeration in paragraph 6). As far as I know, this is the only instance of nested paragraphs in the document.

LaTeX seems to allows this, and so technically there is no problem, but in my HTML conversion of the draft (http://eel.is/c++draft), this curiosity is very hard to deal with, to the point where I've resorted to branching the document and moving the paragraphs outside of the enumeration.

So this is a pull request for the odd chance that you guys either want to support the HTML version, and/or agree with me that nesting paragraphs inside eachother is a very dubious construct of limited value to begin with, to be avoided in any case. :)

Cheers!

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 5, 2016

Would you happen to have a before/after screenshot?

@Eelis
Copy link
Contributor Author

Eelis commented Aug 5, 2016

I believe the only visible difference in the PDF is indentation, but I'll do a diff to be sure.

@Eelis
Copy link
Contributor Author

Eelis commented Aug 5, 2016

Yep, just indentation: diffs

@Eelis
Copy link
Contributor Author

Eelis commented Aug 5, 2016

Btw, an alternative resolution that would work equally well for me is to keep the examples in the enumeration, but make them no longer be paragraphs.

@tkoeppe tkoeppe merged commit 912f5f2 into cplusplus:master Aug 5, 2016
@Eelis Eelis deleted the sanitized branch August 5, 2016 11:54
@Eelis
Copy link
Contributor Author

Eelis commented Aug 5, 2016

Thanks!

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 5, 2016

No problem! Thanks for spotting this.

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