[PATCH 2/2] user32/tests: Test that sending WM_CANCELMODE cancels tracking in window's menu bar.

Ken Thomases ken at codeweavers.com
Fri May 17 00:38:17 CDT 2013


On May 17, 2013, at 12:21 AM, Dmitry Timoshkov wrote:

> Ken Thomases <ken at codeweavers.com> wrote:
> 
>> dlls/user32/tests/menu.c |   21 ++++++++++++++++++++-
> 
> Please add a message test instead, it would much more clearer show what is
> going on.

I added to the existing tests of WM_CANCELMODE because that's what this new test is.  It's just like the existing tests except with a different way of initiating menu tracking.


>> +    {MSG msg;   while (PeekMessage(&msg, 0, 0, 0, PM_REMOVE)) DispatchMessage(&msg);}
> 
> Please avoid creating such embedded blocks.

Again, that's just directly copied from the earlier code in the same test function.  I suppose I could break that up, if you prefer.

-Ken




More information about the wine-devel mailing list