MFC: Need help with WinodwPlacement

Hi all!,

i've generated a standard MFC application using the Wizzard under VC++6.0.

I want to save the WINDOWPLACEMENT when the window closes and load the data when it opens.

Code:

---


/////////////////////////////////////////////////////////////////////////////
// CMainFrame message handlers


void CMainFrame::OnShowWindow(BOOL bShow, UINT nStatus)
{
        CFrameWnd::OnShowWindow(bShow, nStatus);
       
        if (bShow && !IsWindowVisible())
        {
                WINDOWPLACEMENT *wp;
                UINT                    nByte;
                if (AfxGetApp()->GetProfileBinary ("Window", "WP", (LPBYTE*)&wp, &nByte))
                {
                        wp->length = sizeof(WINDOWPLACEMENT);
                        SetWindowPlacement(wp);
                        delete [] wp;
                }
        }
        // TODO: Add your message handler code here
       
}

void CMainFrame::OnClose()
{
        // TODO: Add your message handler code here and/or call default
        WINDOWPLACEMENT wp;

        wp.length = sizeof(WINDOWPLACEMENT);
        GetWindowPlacement(&wp);
        AfxGetApp()->WriteProfileBinary ("Window", "WP", (BYTE *)&wp, sizeof(WINDOWPLACEMENT));

        CFrameWnd::OnClose();
}

---

A Stack Overflow within the OnShowWindow appears when it gots started.

Please Help!

Regards,
Robert

i've added some TRACE lines ...

Code:

---

                if (AfxGetApp()->GetProfileBinary (_T("Window"), _T("WP"),  (LPBYTE *)&wp, &nByte))
                {
                        TRACE("WINDOWPLACEMENT:\n");
                        TRACE("Flags:  %ld\n", wp->flags );
                        TRACE("ShowCMD: %ld\n", wp->showCmd );

                        SetWindowPlacement(wp);

---

... the message
WINDOWPLACEMENT:
Flags: 0
ShowCMD: 1

appears in the debug window for 348 times and the it crashes.

Any idea what's wrong ?

-
Robert

I think mainframe's PreCreateWindow(CREATESTRUCT& cs) virtual function is best place to positioned your application window.

thanks , but i'd like to use the SetWindowPlacement function instead of manipulation the CREATESTRUCT

Meanwhile i've moved the code

Code:

---

        WINDOWPLACEMENT *wp;
        UINT                    nByte=0;
        if (AfxGetApp()->GetProfileBinary ("Window", "WP",  (LPBYTE *)&wp, &nByte))
        {
                m_pMainWnd->SetWindowPlacement (wp);
                delete [] wp;
        }

---

to the CMyApp::InitInstance() method just infornt of

Code:

---

        // The one and only window has been initialized, so show and update it.
        m_pMainWnd->ShowWindow(SW_SHOW);
        m_pMainWnd->UpdateWindow();

---

and the window gets restores correctly.

-
Robert