🚀 go-pugleaf

Thanks again... no problem now...

"Remy Lebeau (TeamB)" <no.spam@no.spam.com> wrote in message
news:421e1b91$1@newsgroups.borland.com...
>
> "roger" <rogersmail@ntlworld.com> wrote in message
> news:421dc13d$1@newsgroups.borland.com...
>
> > extern "C" {
> >     bool __stdcall __declspec(dllexport) InstallHook(void);
> >     bool __stdcall __declspec(dllexport) RemoveHook(void);
> > }
>
> The form should be importing the functions, not exporting them.
>
> >         char *buffer = new char[cds->cbData];
> >         memcpy(buffer, cds->lpData, cds->cbData);
> >         Memo1->Lines->Add(buffer);
>
> You are leaking the memory you allocated as you are never freeing it.  You
> don't need to allocate a new buffer anyway, just use the existing buffer
in
> the message:
>
>     Memo1->Lines->Add(AnsiString(cds->lpData, cds->cbData));
>
> > void __fastcall TForm1::FormClose(TObject *Sender, TCloseAction &Action)
> > {
> >     RemoveHook();
> > }
>
> I would suggest moving the call to RemoveHook() into the form's destructor
> instead.  You used the constructor to install the hook, so the destructor
is
> logical to remove it.  Also, OnClose is not guaranteed to always be called
> in all situations, but the destructor is.
>
>
> Gambit
>
>