Jeff Smith : user32/tests: Change misleading variable name and type.

Alexandre Julliard julliard at winehq.org
Tue Nov 17 15:04:08 CST 2020


Module: wine
Branch: master
Commit: e92cc29d6c520ff44e73a6dc7f2807bbe78c2746
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=e92cc29d6c520ff44e73a6dc7f2807bbe78c2746

Author: Jeff Smith <whydoubt at gmail.com>
Date:   Mon Nov 16 08:45:28 2020 -0600

user32/tests: Change misleading variable name and type.

Signed-off-by: Jeff Smith <whydoubt at gmail.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/user32/tests/msg.c | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c
index bf20c5c7f20..1e616e3d061 100644
--- a/dlls/user32/tests/msg.c
+++ b/dlls/user32/tests/msg.c
@@ -15964,15 +15964,15 @@ static void test_PostMessage(void)
     flush_events();
 }
 
-static LPARAM g_broadcast_lparam;
+static WPARAM g_broadcast_wparam;
 static LRESULT WINAPI broadcast_test_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
 {
     WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA);
 
     if (wParam == 0xbaadbeef)
-        g_broadcast_lparam = wParam;
+        g_broadcast_wparam = wParam;
     else
-        g_broadcast_lparam = 0;
+        g_broadcast_wparam = 0;
 
     return CallWindowProcA(oldproc, hwnd, message, wParam, lParam);
 }
@@ -16040,7 +16040,7 @@ static void test_broadcast(void)
         }
 
         /* send, broadcast */
-        g_broadcast_lparam = 0xdead;
+        g_broadcast_wparam = 0xdead;
         ret = SendMessageTimeoutA(HWND_BROADCAST, messages[i], 0xbaadbeef, 0, SMTO_NORMAL, 2000, NULL);
         if (!ret && GetLastError() == ERROR_TIMEOUT)
             win_skip("broadcasting test %d, timeout\n", i);
@@ -16048,18 +16048,18 @@ static void test_broadcast(void)
         {
             if (messages[i] < WM_USER || messages[i] >= 0xc000)
             {
-                ok(g_broadcast_lparam == 0xbaadbeef, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
-                    g_broadcast_lparam, GetLastError());
+                ok(g_broadcast_wparam == 0xbaadbeef, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
+                    g_broadcast_wparam, GetLastError());
             }
             else
             {
-                ok(g_broadcast_lparam == 0xdead, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
-                    g_broadcast_lparam, GetLastError());
+                ok(g_broadcast_wparam == 0xdead, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
+                    g_broadcast_wparam, GetLastError());
             }
         }
 
         /* send, topmost */
-        g_broadcast_lparam = 0xdead;
+        g_broadcast_wparam = 0xdead;
         ret = SendMessageTimeoutA(HWND_TOPMOST, messages[i], 0xbaadbeef, 0, SMTO_NORMAL, 2000, NULL);
         if (!ret && GetLastError() == ERROR_TIMEOUT)
             win_skip("broadcasting test %d, timeout\n", i);
@@ -16067,13 +16067,13 @@ static void test_broadcast(void)
         {
             if (messages[i] < WM_USER || messages[i] >= 0xc000)
             {
-                ok(g_broadcast_lparam == 0xbaadbeef, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
-                    g_broadcast_lparam, GetLastError());
+                ok(g_broadcast_wparam == 0xbaadbeef, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
+                    g_broadcast_wparam, GetLastError());
             }
             else
             {
-                ok(g_broadcast_lparam == 0xdead, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
-                    g_broadcast_lparam, GetLastError());
+                ok(g_broadcast_wparam == 0xdead, "%d: message %04x, got %#lx, error %d\n", i, messages[i],
+                    g_broadcast_wparam, GetLastError());
             }
         }
     }




More information about the wine-cvs mailing list