[3/3] winex11.drv: Use the old method to delete the desktop window.

Alexandre Julliard julliard at winehq.org
Fri Dec 11 10:35:26 CST 2009


Vincent Povirk <madewokherd+8cd9 at gmail.com> writes:

> On Fri, Dec 11, 2009 at 4:55 AM, Alexandre Julliard <julliard at winehq.org> wrote:
>> "Vincent Povirk" <vincent at codeweavers.com> writes:
>>
>>> @@ -516,6 +516,14 @@ static void handle_wm_protocols( HWND hwnd, XClientMessageEvent *event )
>>>              HMENU hSysMenu;
>>>              POINT pt;
>>>
>>> +            if (hwnd == GetDesktopWindow())
>>> +            {
>>> +                /* The desktop window does not have a close button that we can
>>> +                 * pretend to click. Therefore, we simply send it a close command. */
>>> +                PostMessageW( hwnd, WM_X11DRV_DELETE_WINDOW, 0, 0 );
>>> +                return;
>>> +            }
>>
>> You could just as well send the WM_SYSCOMMAND directly.
>
> Out of curiosity, when was WM_X11DRV_DELETE_WINDOW needed, and why
> isn't it needed here?

It was needed because some apps didn't expect to receive a sent
WM_SYSCOMMAND at that point. The desktop doesn't care.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list