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

[time.duration.nonmember] is missing formatting #916

Closed
zygoloid opened this issue Sep 6, 2016 · 8 comments
Closed

[time.duration.nonmember] is missing formatting #916

zygoloid opened this issue Sep 6, 2016 · 8 comments
Assignees
@jensmaurer

Comments

@zygoloid
Copy link
Member

zygoloid commented Sep 6, 2016

More than half of the Returns: clauses in this subclause are in Roman rather than Teletype.

We also have inconsistency between this subclause and the next one: in [time.duration.comparisons](and in [time.point.nonmember]), we use italic teletype for the placeholder CT, but in this subclause (and in [time.duration.cast]) we use non-italic teletype for the placeholders CD and CR.
@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 25, 2016 •
edited

@zygoloid: To clarify the complaint: Some itemdescrs are using \effects Returns rather than \returns, right? So the \effects should go away, too.

@zygoloid
Copy link
Member Author

I'm not seeing any \effects in this section. Can you point to a paragraph where you see that issue? So far I'm only concerned by the issues I pointed out in the first comment.

@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 26, 2016

Never mind, I misunderstood!

@tkoeppe tkoeppe mentioned this issue Sep 26, 2016
Merged
@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 26, 2016

I'm unsure on how to proceed regarding placeholders. I am currently of the opinion that none of the use cases in utilities.tex should use placeholders, and the only legitimate placeholders I can think of are the F in do_F in locales and the key in fetch_key in atomics, where the placeholder actually stands for part of an identifier. By contrast, all the uses in utilities.tex are merely type aliases, usually introduced by something like "Let T be the type of...".

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Nov 20, 2016
@jensmaurer
Copy link
Member

Editorial meeting consensus: Do not italicize "CD" or "CT", these are actual types, not meta-placeholders.

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Mar 2, 2017
@tkoeppe
Copy link
Contributor

tkoeppe commented Mar 2, 2017

As a further example of what makes a placeholder, there is now the X in XTypes in [tuple.creation].

@jensmaurer
Copy link
Member

I agree that CD represents a specific type (and thus should not be \placeholder), but CR(A,B) is a type meta-function that isn't actually C++, yet we use \tcode. Maybe we want to \placeholder CR here?

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Dec 1, 2017
@jensmaurer jensmaurer mentioned this issue Dec 1, 2017
Merged
@zygoloid
Copy link
Member Author

Expand the CR macro in its four uses.

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Mar 18, 2018
@jensmaurer jensmaurer assigned jensmaurer and unassigned tkoeppe Mar 18, 2018
@jensmaurer jensmaurer mentioned this issue Mar 25, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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