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

[util.smartptr.shared] misuses italics #533

Closed
jwakely opened this issue Aug 25, 2015 · 2 comments
Closed

[util.smartptr.shared] misuses italics #533

jwakely opened this issue Aug 25, 2015 · 2 comments
Assignees
@jwakely @jensmaurer

Comments

@jwakely
Copy link
Member

jwakely commented Aug 25, 2015

Throughout [util.smartptr.shared] terms are italicised on each use, not only when defined, e.g. [util.smartptr.shared.const]/1 uses empty which is defined [util.smartptr.shared]/1

Effects: Constructs an empty shared_ptr object.
@jwakely jwakely self-assigned this Aug 25, 2015
@jensmaurer
Copy link
Member

@jwakely, are you actively working on this?

@jensmaurer jensmaurer self-assigned this Dec 1, 2016
jensmaurer added a commit to jensmaurer/draft that referenced this issue Dec 1, 2016
@jensmaurer
[util.smartptr] Cleanse of italics infestation.
a130c75 
Technical terms should not be in italics everywhere they are referenced.
Common terms such as 'owns' or 'empty' that are used as technical terms
in this subsection only are not italicized or entered into the index.

Fixes cplusplus#533.
@jensmaurer jensmaurer mentioned this issue Dec 1, 2016
Merged
@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 2, 2016 •
edited

To repeat @zygoloid's suggestion here for reference:

We can change "Foo is bar if quz" to "Foo is said to be bar if quz", so that it's clear that a term is being defined ("is said to") without the need to distinguish the defined term typographically.

jensmaurer added a commit to jensmaurer/draft that referenced this issue Dec 2, 2016
@jensmaurer
[util.smartptr] Cleanse of italics infestation.
14c806f 
Technical terms should not be in italics everywhere they are referenced.
Common terms such as 'owns' or 'empty' that are used as technical terms
in this subsection only are not italicized or entered into the index.

Fixes cplusplus#533.
tkoeppe pushed a commit that referenced this issue Dec 2, 2016
@jensmaurer @tkoeppe
[util.smartptr] Cleanse text of italics infestation. (#1161)
afef023 
Technical terms should not be in italics everywhere they are referenced.
Common terms such as 'owns' or 'empty' that are used as technical terms
in this subsection only are not italicized or entered into the index.

Fixes #533.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jwakely jwakely

@jensmaurer jensmaurer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jwakely @tkoeppe @jensmaurer