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

Can we find a better name for "promoted arithmetic type"? #2665

Closed
timsong-cpp opened this issue Feb 9, 2019 · 3 comments · Fixed by #3591
Closed

Can we find a better name for "promoted arithmetic type"? #2665

timsong-cpp opened this issue Feb 9, 2019 · 3 comments · Fixed by #3591
Assignees

Comments

@timsong-cpp
Copy link
Contributor

The term "promoted arithmetic type" is defined by [over.built]p2 to mean "promoted integral types and all floating-point types". It seems suboptimal to say that float is a "promoted arithmetic type" when it is subject to (floating-point) promotion to double.

@jensmaurer
Copy link
Member

jensmaurer commented Feb 10, 2019

Is this sufficiently editorial? Does it matter sufficiently (we're talking about a local definition in [over.built])? What's the new name?

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Feb 10, 2019
@timsong-cpp
Copy link
Contributor Author

I don't have a great alternative name right now (or I'd have suggested it). Given that the term appears to be only used four times and all in this subclause, perhaps we can just substitute the definition in those uses and remove the term altogether?

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Jun 5, 2019
@jensmaurer
Copy link
Member

jensmaurer commented Jun 5, 2019

Editorial meeting: Not enthusiastic about a change in this area. Go with Tim's expansion suggestion for now.

p13: "L and R are each...", also p27

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 a pull request may close this issue.

2 participants