Francois Gouget : user32/tests: Use the caller line number for open_clipboard() & co.

Alexandre Julliard julliard at winehq.org
Fri Jun 11 15:31:30 CDT 2021


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Thu Jun 10 12:11:16 2021 +0200

user32/tests: Use the caller line number for open_clipboard() & co.

In particular this stops the traces in open_clipboard() and
has_no_open_wnd() from clobbering the line number when used in
expressions such as ok(has_no_open_wnd(), ...).

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/user32/tests/clipboard.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/dlls/user32/tests/clipboard.c b/dlls/user32/tests/clipboard.c
index 5951fc3247f..c8de860f8cc 100644
--- a/dlls/user32/tests/clipboard.c
+++ b/dlls/user32/tests/clipboard.c
@@ -33,7 +33,8 @@ static BOOL (WINAPI *pGetUpdatedClipboardFormats)( UINT *formats, UINT count, UI
 static int thread_from_line;
 static char *argv0;
 
-static BOOL open_clipboard(HWND hwnd)
+#define open_clipboard(hwnd) open_clipboard_(__LINE__, hwnd)
+static BOOL open_clipboard_(int line, HWND hwnd)
 {
     DWORD start = GetTickCount();
     while (1)
@@ -54,7 +55,7 @@ static BOOL open_clipboard(HWND hwnd)
              *   response to a native application.
              */
             GetClassNameA(clipwnd, classname, ARRAY_SIZE(classname));
-            trace("%p (%s) opened the clipboard\n", clipwnd, classname);
+            trace_(__FILE__, line)("%p (%s) opened the clipboard\n", clipwnd, classname);
             SetLastError(le);
             return ret;
         }
@@ -62,7 +63,8 @@ static BOOL open_clipboard(HWND hwnd)
     }
 }
 
-static BOOL has_no_open_wnd(void)
+#define has_no_open_wnd() has_no_open_wnd_(__LINE__)
+static BOOL has_no_open_wnd_(int line)
 {
     DWORD start = GetTickCount();
     DWORD le = GetLastError();
@@ -76,7 +78,7 @@ static BOOL has_no_open_wnd(void)
             le = GetLastError();
             /* See open_clipboard() */
             GetClassNameA(clipwnd, classname, ARRAY_SIZE(classname));
-            trace("%p (%s) opened the clipboard\n", clipwnd, classname);
+            trace_(__FILE__, line)("%p (%s) opened the clipboard\n", clipwnd, classname);
             SetLastError(le);
             return FALSE;
         }




More information about the wine-cvs mailing list