Re: Reading Serial Port

From:

"Alexander Grigoriev" <alegr@earthlink.net>

Newsgroups:

microsoft.public.vc.mfc

Date:

Mon, 4 Feb 2008 18:53:58 -0800

Message-ID:

<ObCgbJ6ZIHA.6140@TK2MSFTNGP02.phx.gbl>

You've got the class destructor wrong. You should NOT do 'delete this' in
the destructor.

In your Stop() function, you need to signal an event, then wait for the
thread handle, not for that event (which you just signaled). You cannot
simply close handles in the destructor, while they may be in use in the
thread, and only then wait for the thread completion.

"clinisbut" <clinisbut@gmail.com> wrote in message
news:76012220-ac65-4870-9126-e322cbba1b82@n20g2000hsh.googlegroups.com...

And this is what I write to kill my thread in his destructor:
First I stop the reader thread:

MySerial::Stop()
{

SetEvent( ShutdownEvent );
WaitForSingleObject( ShutdownEvent, INFINITE );

}

Then I close the app and this should be executed:
MySerial::~MySerial()
{
CloseHandle(ShutdownEvent);
CloseHandle(ReadEvent);
CloseHandle(WriteEvent);

// Wait for the thread to exit before deleting
WaitForSingleObject( this->m_hThread, INFINITE );

// delete this; //wrong

}