tags:

views:

305

answers:

4

When creating a new instance of a MyClass as an argument to a function like so:

class MyClass
{
  MyClass(int a);
};    

myFunction(MyClass(42));

does the standard make any grantees on the timing of the destructor?
Specifically, can I assume that the it is going to be called before the next statement after the call to myFunction() ?

+22  A: 

Temporary objects are destroyed at the end of the full expression they're part of.

A full expression is an expression that isn't a sub-expression of some other expression. Usually this means it ends at the ; (or ) for if, while, switch etc.) denoting the end of the statement. In your example, it's the end of the function call.

Note that you can extend the lifetime of temporaries by binding them to a const reference. Doing so extends their lifetime to the reference's lifetime:

MyClass getMyClass();

{
  const MyClass& r = getMyClass(); // full expression ends here
  ...
} // object returned by getMyClass() is destroyed here

If you don't plan to change the returned object, then this is a nice trick to safe a copy constructor call (compared to MyClass obj = getMyClass();) which unfortunately isn't very well known. (I suppose C++11's move semantics will render it less useful, though.)

sbi
Informative answer. +1
Konrad
+1 nice answer. Tho i think you mean ) instead of } for if, while, switch etc? `if(f(string())) { ... }` surely won't make the string survive until `}` :)
Johannes Schaub - litb
@Johannes: No doesn't it? I thought it would.
sbi
@sbi the full expression ends at the `)`. The whole thing might be a "full statement" but not a "full expression". Same in `for(iterator i = begin(); i<end; i++) { ... }` the compiler is not required to keep the temporary iterators resulting from `i++` alive until the end of the for statement, of course. The temporaries will live only until after its full expression's end, which is the `)` in this case too.
Johannes Schaub - litb
You said it yourself correctly: "A full expression is an expression that isn't a sub-expression of some other expression." and in the while, if, switch cases we have a full expression between the `(` and `)` because the statement they appear in is not an expression by any means. In your interpretation, the end of `;` would not be a full-expression-end at all, because there is a larger "expression" around it if you are within a while/if or switch. I think it's easy to see that this makes no sense :)
Johannes Schaub - litb
@Johannes: I changed it. <sigh> After so many years of C++ you still learn something every week.
sbi
+8  A: 

The standard does indeed offer guarantees - from section 12.2/5:

A temporary bound to a reference parameter in a function call (5.2.2) persists until the completion of the full expression containing the call

However, in your code it is not clear if the parameter is being passed by reference or by value, though at some point a copy constructor which does take a reference will be used.

anon
+1  A: 

In section 12.2, Temporary Objects, clause 3, the ANSI/ISO C standard states: "... Temporary objects are destroyed as the last step in evaluating the full-expression that (lexically) contains the point where they were created."

This is closely related to the concept of Sequence Points. When a sequence point is reached, all side-effects of expressions are guaranteed to have taken place.

Mike Mueller
+4  A: 

Everyone has rightly cited 12.2/3 or similar:

Temporary objects are destroyed as the last step in evaluating the full-expression that (lexically) contains the point where they were created.

I find it amusing that over the next page in my printing of the standard, 12.2/4 says:

There are two contexts in which temporaries are destroyed at a different point than the end of the full-expression.

Neither of them applies to your example, they both relate to the use of temporaries in initializers. But it does go to show that you have to keep your wits about you when dealing with a tricky beast like the C++ standard.

Steve Jessop