Re: Initializing an object with auto pointers

From:

Alex Blekhman <tkfx.REMOVE@yahoo.com>

Newsgroups:

microsoft.public.vc.language

Date:

Mon, 12 Apr 2010 14:03:23 +0300

Message-ID:

<#WfmG$i2KHA.5212@TK2MSFTNGP04.phx.gbl>

On 12-Apr-10 13:48, Jack wrote:

std::auto_ptr<BFThread> thread1;

case WM_CREATE:
thread1 = new BFThread(1, hEvent);
break;

////////////
I know I should have used BFThread*
But how do I take advantage of auto_ptr in this example?
I put a direct assignment inside the switch-case statement, but
it results in "Error 1 error C2360: initialization of 'thread1' is

skipped by 'case' label"
It was the original statement I coped from, and I wanted to keep it intact.
Anyway, I need to break it up into 2 statements inside wndproc.
What is the correct statement in this case?

I don't think any statement that involves std::auto_ptr in your case
will be correct. The moment std::auto_ptr instance goes out of scope the
BFThread will be deleted. May be this is what you want, I don't know,
but it seems that you'd like to preserve BFThread instance for the
lifetime of the running thread.

So, if my assumption is right, then you cannot use std::auto_ptr here.
You need to store a pointer to BFThread object somewhere until your
thread exits and only then delete the object. Maybe
std::tr1::shared_pointer, which is stored outside of the function is
more appropriate here.

Alex