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

Marko Nikolic grkoma at gmail.com
Wed Oct 5 09:43:25 CDT 2011


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

diff --git a/dlls/comctl32/tests/datetime.c b/dlls/comctl32/tests/datetime.c
index d60ad54..323ec58 100644
--- a/dlls/comctl32/tests/datetime.c
+++ b/dlls/comctl32/tests/datetime.c
@@ -204,22 +204,22 @@ 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;
+    //LRESULT r;
+    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)
-- 
1.7.4.1




More information about the wine-patches mailing list