Michael Stefaniuc : comctl32/tests: IDI_APPLICATION is already an INTRESOURCE (PVS-Studio).

Alexandre Julliard julliard at wine.codeweavers.com
Tue Oct 13 10:53:47 CDT 2015


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

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Tue Oct 13 00:32:36 2015 +0200

comctl32/tests: IDI_APPLICATION is already an INTRESOURCE (PVS-Studio).

Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/comctl32/tests/misc.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/comctl32/tests/misc.c b/dlls/comctl32/tests/misc.c
index ffd0dca..9a4b902 100644
--- a/dlls/comctl32/tests/misc.c
+++ b/dlls/comctl32/tests/misc.c
@@ -267,7 +267,7 @@ static void test_LoadIconWithScaleDown(void)
 
     /* invalid arguments */
     icon = (HICON)0x1234;
-    hr = pLoadIconMetric(NULL, MAKEINTRESOURCEW(IDI_APPLICATION), 0x100, &icon);
+    hr = pLoadIconMetric(NULL, (LPWSTR)IDI_APPLICATION, 0x100, &icon);
     ok(hr == E_INVALIDARG, "Expected E_INVALIDARG, got %x\n", hr);
     ok(icon == NULL, "Expected NULL, got %p\n", icon);
 
@@ -302,7 +302,7 @@ static void test_LoadIconWithScaleDown(void)
        "Expected HRESULT_FROM_WIN32(ERROR_RESOURCE_TYPE_NOT_FOUND), got %x\n", hr);
 
     /* load icon using predefined identifier */
-    hr = pLoadIconMetric(NULL, MAKEINTRESOURCEW(IDI_APPLICATION), LIM_SMALL, &icon);
+    hr = pLoadIconMetric(NULL, (LPWSTR)IDI_APPLICATION, LIM_SMALL, &icon);
     ok(hr == S_OK, "Expected S_OK, got %x\n", hr);
     res = GetIconInfo(icon, &info);
     ok(res, "Failed to get icon info, error %u\n", GetLastError());
@@ -312,7 +312,7 @@ static void test_LoadIconWithScaleDown(void)
     ok(bmp.bmHeight == GetSystemMetrics(SM_CYSMICON), "Wrong icon height %d\n", bmp.bmHeight);
     DestroyIcon(icon);
 
-    hr = pLoadIconMetric(NULL, MAKEINTRESOURCEW(IDI_APPLICATION), LIM_LARGE, &icon);
+    hr = pLoadIconMetric(NULL, (LPWSTR)IDI_APPLICATION, LIM_LARGE, &icon);
     ok(hr == S_OK, "Expected S_OK, got %x\n", hr);
     res = GetIconInfo(icon, &info);
     ok(res, "Failed to get icon info, error %u\n", GetLastError());
@@ -322,7 +322,7 @@ static void test_LoadIconWithScaleDown(void)
     ok(bmp.bmHeight == GetSystemMetrics(SM_CYICON), "Wrong icon height %d\n", bmp.bmHeight);
     DestroyIcon(icon);
 
-    hr = pLoadIconWithScaleDown(NULL, MAKEINTRESOURCEW(IDI_APPLICATION), 42, 42, &icon);
+    hr = pLoadIconWithScaleDown(NULL, (LPWSTR)IDI_APPLICATION, 42, 42, &icon);
     ok(hr == S_OK, "Expected S_OK, got %x\n", hr);
     res = GetIconInfo(icon, &info);
     ok(res, "Failed to get icon info, error %u\n", GetLastError());




More information about the wine-cvs mailing list