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

[dcl.decl] Turn very large footnote into ordinary note. #1121

Merged
merged 1 commit into from Nov 28, 2016

Conversation

Eelis
Copy link
Contributor

@Eelis Eelis commented Nov 24, 2016

This footnote

  • is very large
  • has very dense vertical layout in the pdf (as a result of being a footnote)
  • uses unusual commands for its code formatting (\indent for indentation, \textrm for comments)
  • forgets to escape curly brackets in its tcode fragments, so they're missing in the pdf
  • uses unusual commands for its prose formatting (\noindent)
  • has unusual code formatting in the pdf (non-italic comments)

Some or all of these issues might be addressable individually, but the easiest way to fix them all in one fell swoop is to just turn this weird creature into a normal note that needs no special treatment or commands. :)

diff

(I tried diffpdf's other comparison modes, but they produced even less useful coloring.)

T D1; T D2; ... T Dn;
\end{codeblock}
where \tcode{T} is a \grammarterm{decl-specifier-seq}
and each \tcode{Di} is an \grammarterm{init-declarator}.
An exception occurs when a name introduced by one of the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we're promoting this to a note, the bar for clarity and correctness is a little higher, and we shouldn't be saying "an exception occurs when" unless we mean that an exception is thrown. How about "One exception is when" here and "Another exception is when" below?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good! Patch amended.

@zygoloid zygoloid merged commit 7ce7c17 into cplusplus:master Nov 28, 2016
@Eelis Eelis deleted the footnote branch November 28, 2016 18:32
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