Re: [PATCH v2] user32: Also remove WM_CHAR from queue when a menu gets VK_ESCAPE and add tests

[Alexandre Julliard]

Fabian Maurer  writes:
> @@ -3186,8 +3186,12 @@ static BOOL MENU_TrackMenu( HMENU hmenu, UINT wFlags, INT x, INT y,
>    	    break;
> case VK_ESCAPE:
> -
>            {
> -
>                MSG msg_dummy;
>                fEndMenu = MENU_KeyEscape(&mt, wFlags);
> -
>     break;
> -
>                PeekMessageW(&msg_dummy, 0, WM_CHAR, WM_CHAR, PM_REMOVE);
> -
>                break;
> -
>            }

It would be cleaner to avoid TranslateMessage instead of removing the
message (possibly getting the wrong one).