Re: Fix SetMenu after CreateWindow for invisible window

[Dmitry Timoshkov]

"Michal Janusz Miroslaw"  wrote:
> +/**********************************************************************
> -
> -
>     SetMenu    (USER32.@)
> - */
>
> +BOOL WINAPI SetMenu( HWND hWnd, HMENU hMenu )
> +{
> - TRACE("(%p, %p);\n", hWnd, hMenu);
> -
> - if (GetWindowLongA( hWnd, GWL_STYLE ) & WS_CHILD) return FALSE;

A better fix is to remove the above check altogether.

Fix SetMenu after CreateWindow for invisible window

[Michal Janusz Miroslaw]

On Fri, Sep 19, 2003 at 11:11:30PM +0900, Dmitry Timoshkov wrote:
> "Michal Janusz Miroslaw"  wrote:
> > +/**********************************************************************
> > -
> > -
> >     SetMenu    (USER32.@)
> > - */
> >
> > +BOOL WINAPI SetMenu( HWND hWnd, HMENU hMenu )
> > +{
> > - TRACE("(%p, %p);\n", hWnd, hMenu);
> > -
> > - if (GetWindowLongA( hWnd, GWL_STYLE ) & WS_CHILD) return FALSE;
>
> A better fix is to remove the above check altogether.

I tried it, but SetMenu is called from WIN_CreateWindowEx before initialising
X11DRV's structures (wndPtr->pDriverData is bogus), and it didn't work for
windows with hMenu != 0 parameter to CreateWindow*. The real fix, was to
remove 'if (IsWindowVisible(hWnd))' before SetWindowPos, but that didn't
work in the middle of CreateWindow.

-- Michał Mirosław