comctl32/tests: make sure to use return value (LLVM/Clang) (2/2)
Austin English
austinenglish at gmail.com
Fri Feb 4 18:08:42 CST 2011
Not really a series, typo in the email subject, but didn't want to
screw up testbot.
--
-Austin
-------------- next part --------------
diff --git a/dlls/comctl32/tests/imagelist.c b/dlls/comctl32/tests/imagelist.c
index bbb7d85..28b21a4 100644
--- a/dlls/comctl32/tests/imagelist.c
+++ b/dlls/comctl32/tests/imagelist.c
@@ -751,7 +751,6 @@ static INT DIB_GetWidthBytes( int width, int bpp )
case 32: words = width; break;
default:
- words=0;
trace("Unknown depth %d, please report.\n", bpp );
assert(0);
break;
@@ -1616,7 +1615,7 @@ static void test_IImageList_Draw(void)
if (0)
{
/* crashes on native */
- hr = IImageList_Draw(imgl, NULL);
+ IImageList_Draw(imgl, NULL);
}
memset(&imldp, 0, sizeof (imldp));
@@ -1691,8 +1690,8 @@ static void test_IImageList_Merge(void)
if (0)
{
/* null cases that crash on native */
- hr = IImageList_Merge(imgl1, -1, NULL, 0, 0, 0, &IID_IImageList, (void**)&merge);
- hr = IImageList_Merge(imgl1, -1, (IUnknown*) imgl2, 0, 0, 0, &IID_IImageList, NULL);
+ IImageList_Merge(imgl1, -1, NULL, 0, 0, 0, &IID_IImageList, (void**)&merge);
+ IImageList_Merge(imgl1, -1, (IUnknown*) imgl2, 0, 0, 0, &IID_IImageList, NULL);
}
/* If himl1 has no images, merge still succeeds */
@@ -1805,7 +1804,7 @@ static void test_IImageList_Clone(void)
if (0)
{
/* crashes on native */
- hr = IImageList_Clone(imgl, &IID_IImageList, NULL);
+ IImageList_Clone(imgl, &IID_IImageList, NULL);
}
hr = IImageList_Clone(imgl, &IID_IImageList, (void**)&imgl2);
@@ -1829,7 +1828,7 @@ static void test_IImageList_GetBkColor(void)
if (0)
{
/* crashes on native */
- hr = IImageList_GetBkColor(imgl, NULL);
+ IImageList_GetBkColor(imgl, NULL);
}
hr = IImageList_GetBkColor(imgl, &color);
@@ -1851,7 +1850,7 @@ static void test_IImageList_SetBkColor(void)
if (0)
{
/* crashes on native */
- hr = IImageList_SetBkColor(imgl, RGB(0, 0, 0), NULL);
+ IImageList_SetBkColor(imgl, RGB(0, 0, 0), NULL);
}
hr = IImageList_SetBkColor(imgl, CLR_NONE, &color);
@@ -1881,7 +1880,7 @@ static void test_IImageList_GetImageCount(void)
if (0)
{
/* crashes on native */
- hr = IImageList_GetImageCount(imgl, NULL);
+ IImageList_GetImageCount(imgl, NULL);
}
count = -1;
More information about the wine-patches
mailing list