[PATCH v6 0/2] MR10452: comctl32/tests: Print id field of logged/expected messages on failure

[Piotr Pawłowski (＠DEATH)]

Makes debugging of WM_NOTIFY message sequence errors much easier.

--
  v6: uxtheme/tests: Print id field of logged/expected messages on failure
      comctl32/tests: Print id field of logged/expected messages on failure
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10452

[PATCH v6 1/2] comctl32/tests: Print id field of logged/expected messages on failure

[Piotr Pawłowski]

From: Piotr Pawłowski <[e-mail]>

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

diff --git a/dlls/comctl32/tests/msg.h b/dlls/comctl32/tests/msg.h
index 76bf5450b2f..a9c150e5da8 100644
--- a/dlls/comctl32/tests/msg.h
+++ b/dlls/comctl32/tests/msg.h
@@ -115,8 +115,8 @@ static void dump_sequence( struct msg_sequence **seq, int sequence_index,
     trace_(file, line)("Failed sequence %s:\n", context );
     while (expected->message && actual->message)
     {
-        trace_(file, line)( "  %u: expected: %04x - actual: %04x wp %08Ix lp %08Ix\n",
-                            count, expected->message, actual->message, actual->wParam, actual->lParam );
+        trace_(file, line)( "  %u: expected: %04x id %u - actual: %04x wp %08Ix lp %08Ix id %u\n",
+                            count, expected->message, expected->id, actual->message, actual->wParam, actual->lParam, actual->id );
 
 	if (expected->message == actual->message)
 	{
@@ -157,8 +157,8 @@ static void dump_sequence( struct msg_sequence **seq, int sequence_index,
 
     while (actual->message)
     {
-        trace_(file, line)( "  %u: expected: nothing - actual: %04x wp %08Ix lp %08Ix\n",
-                            count, actual->message, actual->wParam, actual->lParam );
+        trace_(file, line)( "  %u: expected: nothing - actual: %04x wp %08Ix lp %08Ix id %u\n",
+                            count, actual->message, actual->wParam, actual->lParam, actual->id );
         actual++;
         count++;
     }
-- 
GitLab


https://gitlab.winehq.org/wine/wine/-/merge_requests/10452

[PATCH v6 2/2] uxtheme/tests: Print id field of logged/expected messages on failure

[Piotr Pawłowski]

From: Piotr Pawłowski <[e-mail]>

---
 dlls/uxtheme/tests/msg.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/uxtheme/tests/msg.h b/dlls/uxtheme/tests/msg.h
index f967c39d091..219142f7867 100644
--- a/dlls/uxtheme/tests/msg.h
+++ b/dlls/uxtheme/tests/msg.h
@@ -113,8 +113,8 @@ static void dump_sequence( struct msg_sequence **seq, int sequence_index,
     trace_(file, line)("Failed sequence %s:\n", context );
     while (expected->message && actual->message)
     {
-        trace_(file, line)( "  %u: expected: %04x - actual: %04x wp %08Ix lp %08Ix\n",
-                            count, expected->message, actual->message, actual->wParam, actual->lParam );
+        trace_(file, line)( "  %u: expected: %04x id %u - actual: %04x wp %08Ix lp %08Ix id %u\n",
+                            count, expected->message, expected->id, actual->message, actual->wParam, actual->lParam, actual->id );
 
         if (expected->message == actual->message)
         {
@@ -154,8 +154,8 @@ static void dump_sequence( struct msg_sequence **seq, int sequence_index,
 
     while (actual->message)
     {
-        trace_(file, line)( "  %u: expected: nothing - actual: %04x wp %08Ix lp %08Ix\n",
-                            count, actual->message, actual->wParam, actual->lParam );
+        trace_(file, line)( "  %u: expected: nothing - actual: %04x wp %08Ix lp %08Ix id %u\n",
+                            count, actual->message, actual->wParam, actual->lParam, actual->id );
         actual++;
         count++;
     }
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/10452

[PATCH v6 0/2] MR10452: comctl32/tests: Print id field of logged/expected messages on failure - approved

[Zhiyi Zhang (＠zhiyi)]

This merge request was approved by Zhiyi Zhang.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10452