Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[conv.general, expr.static.cast] Remove inappropriate "temporary" #6646

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
Loading
from

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Nov 8, 2023

It's not clear in which sense the invented variables in question are "temporary". Since C++ already contains a notion of a "temporary (object)", and since variables are precisely not temporary objects, it seems unnecessary to use the words "temporary variable" in the present sense.

See also http://thecodelesscode.com/case/234.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Nov 8, 2023

@opensdh: could you please PTAL a look?

It's not clear in which sense the invented variables in question are
"temporary". Since C++ already contains a notion of a "temporary
(object)", and since variables are precisely _not_ temporary objects,
it seems unnecessary to use the words "temporary variable" in the
present sense.
Copy link
Contributor

@opensdh opensdh left a comment

Choose a reason for hiding this comment

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

You should probably check with more than one person here, though.

source/expressions.tex Outdated Show resolved Hide resolved
@tkoeppe
Copy link
Contributor Author

tkoeppe commented Nov 9, 2023

@jensmaurer: Could I interest you/CWG in this?

@jensmaurer
Copy link
Member

CWG 2023-11-10: We want [class.temporary] p6 to refer to "invented temporary variable" as one of the roots of "temporary expression" and say this is a "temporary object". Other cases (where we don't mean "temporary") should use "invented variable". This needs a core issue.

@tkoeppe tkoeppe added cwg Issue must be reviewed by CWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. labels Nov 10, 2023
@languagelawyer
Copy link
Contributor

OP should say it addresses #2215

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cwg Issue must be reviewed by CWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.