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

Use "requirement" in grammar font in expr.prim.req #3648

Merged
merged 2 commits into from Mar 4, 2020

Conversation

akrzemi1
Copy link
Contributor

[expr.prim.req] uses grammarterm requirement in paragraph 5, but normal font in paragraph 6. Term "requirement" is never defined in this clause, but it has a different meaning in the library.

source/expressions.tex Outdated Show resolved Hide resolved
source/expressions.tex Outdated Show resolved Hide resolved
Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

See above.

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Jan 23, 2020
@jensmaurer
Copy link
Member

You change some "s" in places that weren't previously in scope for this pull request (ok), but you didn't change the "s" things for the stuff that you originally changed here. That's not really progress.

@akrzemi1
Copy link
Contributor Author

Sorry. Learning git pull requests and tex at the same time...

@jensmaurer jensmaurer removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Jan 24, 2020
Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

Ok, looks good now.

Note to committer: Needs squashing and fixing of the commit message.

[expr.prim.req] uses grammarterm <em><code>requirement</code></em> in paragraph 5, but normal font in paragraph 6. Term "requirement" is never defined in this clause, but it has a different meaning in the library.
@zygoloid
Copy link
Member

zygoloid commented Mar 4, 2020

There's no need for braces around or near the ss at all. Will remove those now...

@zygoloid zygoloid merged commit b87e1be into cplusplus:master Mar 4, 2020
@jensmaurer
Copy link
Member

(Braces were required in former times due to LaTeX-isms.)

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

3 participants