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

2013-05-16 Thread Dmitry Timoshkov
Ken Thomases wrote: > > 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

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

2013-05-16 Thread Ken Thomases
On May 17, 2013, at 12:21 AM, Dmitry Timoshkov wrote: > Ken Thomases 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 t

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

2013-05-16 Thread Dmitry Timoshkov
Ken Thomases wrote: > dlls/user32/tests/menu.c | 21 - Please add a message test instead, it would much more clearer show what is going on. > +{MSG msg; while (PeekMessage(&msg, 0, 0, 0, PM_REMOVE)) > DispatchMessage(&msg);} Please avoid creating such embedded block