Marko Nikolic : comctl32/tests: Removed sign comparison warning in datetime tests.

Alexandre Julliard julliard at winehq.org
Thu Oct 6 17:24:30 CDT 2011


Module: wine
Branch: master
Commit: 154d088eb99e7a31e9f365fc88278a001c96a063
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=154d088eb99e7a31e9f365fc88278a001c96a063

Author: Marko Nikolic <grkoma at gmail.com>
Date:   Thu Oct  6 12:18:02 2011 +0200

comctl32/tests: Removed sign comparison warning in datetime tests.

---

 dlls/comctl32/tests/datetime.c |   19 +++++++++----------
 1 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/dlls/comctl32/tests/datetime.c b/dlls/comctl32/tests/datetime.c
index d60ad54..171da5c 100644
--- a/dlls/comctl32/tests/datetime.c
+++ b/dlls/comctl32/tests/datetime.c
@@ -204,22 +204,21 @@ static void test_dtm_set_format(void)
 
 static void test_mccolor_types(HWND hWndDateTime, int mccolor_type, const char* mccolor_name)
 {
-    LRESULT r;
-    COLORREF theColor, prevColor;
+    COLORREF theColor, prevColor, crColor;
 
     theColor=RGB(0,0,0);
-    r = SendMessage(hWndDateTime, DTM_SETMCCOLOR, mccolor_type, theColor);
-    ok(r != -1, "%s: Set RGB(0,0,0): Expected COLORREF of previous value, got %ld\n", mccolor_name, r);
+    crColor = SendMessage(hWndDateTime, DTM_SETMCCOLOR, mccolor_type, theColor);
+    ok(crColor != ~0u, "%s: Set RGB(0,0,0): Expected COLORREF of previous value, got %d\n", mccolor_name, crColor);
     prevColor=theColor;
     theColor=RGB(255,255,255);
-    r = SendMessage(hWndDateTime, DTM_SETMCCOLOR, mccolor_type, theColor);
-    ok(r==prevColor, "%s: Set RGB(255,255,255): Expected COLORREF of previous value, got %ld\n", mccolor_name, r);
+    crColor = SendMessage(hWndDateTime, DTM_SETMCCOLOR, mccolor_type, theColor);
+    ok(crColor==prevColor, "%s: Set RGB(255,255,255): Expected COLORREF of previous value, got %d\n", mccolor_name, crColor);
     prevColor=theColor;
     theColor=RGB(100,180,220);
-    r = SendMessage(hWndDateTime, DTM_SETMCCOLOR, mccolor_type, theColor);
-    ok(r==prevColor, "%s: Set RGB(100,180,220): Expected COLORREF of previous value, got %ld\n", mccolor_name, r);
-    r = SendMessage(hWndDateTime, DTM_GETMCCOLOR, mccolor_type, 0);
-    ok(r==theColor, "%s: GETMCCOLOR: Expected %d, got %ld\n", mccolor_name, theColor, r);
+    crColor = SendMessage(hWndDateTime, DTM_SETMCCOLOR, mccolor_type, theColor);
+    ok(crColor==prevColor, "%s: Set RGB(100,180,220): Expected COLORREF of previous value, got %d\n", mccolor_name, crColor);
+    crColor = SendMessage(hWndDateTime, DTM_GETMCCOLOR, mccolor_type, 0);
+    ok(crColor==theColor, "%s: GETMCCOLOR: Expected %d, got %d\n", mccolor_name, theColor, crColor);
 }
 
 static void test_dtm_set_and_get_mccolor(void)




More information about the wine-cvs mailing list