🚀 go-pugleaf

"Paw Suddergaard" <paw@top-house.dk> wrote in message
news:4517ed96$1@newsgroups.borland.com...

> I have an abort button that calls terminate:
>
> void __fastcall TEasyMail::IdThreadComponent1Terminate(
>        TIdCustomThreadComponent *Sender)
> {
>   // Does nothing:  IdSMTP->Socket->Close();
>   // Does nothing: IdSMTP->Disconnect();
> }

That is the completely wrong place to put that code.  The thread is blocked
while Send() is running.  The OnTerminate event is triggered when the thread
is terminated (thus its name).  Calling Terminate() DOES NOT actually
terminate the thread immediately.  All it does is sets the Terminated
property to true and nothing else.  That signals to the thread that it
should terminate itself at its earliest convenience, which it can't do until
after Send() returns.

You need to call Disconnect() inside your button OnClick event directly, not
in the thread's OnTerminate event, in order to abort the sending.  You
should also use DisconnectSocket() instead of Disconnect(), since you are
closing the connecting from a different thread than the one that established
the connection.


Gambit