🚀 go-pugleaf

Oh, thanks.  I didn't even realize their was a ParentWindow property.

I've noted the additional exception handling as well.  I had a TODO on that 
(to look up common E_ return values).

Thanks again.
-- 
Malcolm Smith
MJ Freelancing
http://www.mjfreelancing.com

Associate Editor
C++Builder Developers Journal
http://www.bcbjournal.com


"Remy Lebeau (TeamB)" <no.spam@no.spam.com> wrote in message 
news:446260ad$1@newsgroups.borland.com...
>
> "Malcolm Smith" <mjfreelancing@optusnet.com.au> wrote in message
> news:446259de@newsgroups.borland.com...
>
>> The HWND is coming into my DLL from the Cleanup Manager, not going out.
>
> Exactly.  You have to use the HWND that is passed into ShowProperties() as
> the parent window for any windows that ShowProperties() displays.
>
>> we are caching the form in case the user shows it more than once
>
> You need to change the form's ParentWindow property, ie:
>
>    HRESULT STDMETHODCALLTYPE TDevCleanCtrlImpl::ShowProperties(HWND hwnd)
>    {
>        try
>        {
>            if( !frmFileDetails )
>                frmFileDetails = new TfrmFileDetails( NULL, FVolume,
> FFileSearch, FFileStates );
>        }
>        catch(const Exception &)
>        {
>            return E_OUTOFMEMORY;
>        }
>
>        try
>        {
>            frmFileDetails->ParentWindow = hwnd;
>            try {
>                frmFileDetails->ShowModal();
>            }
>            __finally {
>                frmFileDetails->ParentWindow = NULL;
>            }
>        }
>        catch(const Exception &)
>        {
>            return E_UNEXPECTED;
>        }
>
>        return S_OK;
>    }
>
>
> Gambit
>
>