comctl32/tests: Check for the outcome of the SendMessage calls in test_monthcal_size.

Gerald Pfeifer gerald at pfeifer.com
Thu May 13 16:44:25 CDT 2010


Nikolay suggest that instead of dropping res here, it wouldn't hurt 
testing for the result of SendMessage.  So, that's what this patch does.

https://testbot.winehq.org/JobDetails.pl?Key=2055 is fine with it, too
(just W7PROX64 still running).

Gerald

---
 dlls/comctl32/tests/monthcal.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/dlls/comctl32/tests/monthcal.c b/dlls/comctl32/tests/monthcal.c
index d1fa8de..30f682f 100644
--- a/dlls/comctl32/tests/monthcal.c
+++ b/dlls/comctl32/tests/monthcal.c
@@ -1503,10 +1503,12 @@ static void test_monthcal_size(void)
     /* initialize to a font we can compare against */
     SendMessage(hwnd, WM_SETFONT, (WPARAM)hFont1, 0);
     res = SendMessage(hwnd, MCM_GETMINREQRECT, 0, (LPARAM)&r1);
+    ok(res, "SendMessage(MCM_GETMINREQRECT) failed\n");
 
     /* check that setting a larger font results in an larger rect */
     SendMessage(hwnd, WM_SETFONT, (WPARAM)hFont2, 0);
     res = SendMessage(hwnd, MCM_GETMINREQRECT, 0, (LPARAM)&r2);
+    ok(res, "SendMessage(MCM_GETMINREQRECT) failed\n");
 
     OffsetRect(&r1, -r1.left, -r1.top);
     OffsetRect(&r2, -r2.left, -r2.top);
-- 
1.6.6.2



More information about the wine-patches mailing list