On May 17, 2013, at 12:21 AM, Dmitry Timoshkov wrote:
> Ken Thomases <k...@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
