[PATCH 1/3] comctl32/tests: Print id value as decimal. (Resend)

[Dmitry Timoshkov]

All the notification codes (printed as an id) have negative base value
(SBN_, UDN_, HDN_, BCN_, TBN_, TTN_, RBN_, TRBN_, PGN_, TVN_, LVN_, TCN_,
CBEN_, IPN_, MCN_, DTN_), and printing them as decimal helps to quickly
identify the corresponding notification value in cases when an id in
the message sequence doesn't match. Custom ids in the message sequences
have small numbers (0, 1, 2, ...), so they also get printed correctly.

Signed-off-by: Dmitry Timoshkov 
---
 dlls/comctl32/tests/msg.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/comctl32/tests/msg.h b/dlls/comctl32/tests/msg.h
index 7337c724a5c..8ba2efddb0a 100644
--- a/dlls/comctl32/tests/msg.h
+++ b/dlls/comctl32/tests/msg.h
@@ -261,14 +261,14 @@ static void ok_sequence_(struct msg_sequence **seq, int sequence_index,
                         failcount++;
                         dump++;
                         ok_(file, line) (FALSE,
-                            "%s: in msg 0x%04x expecting id 0x%x got 0x%x\n",
+                            "%s: in msg 0x%04x expecting id %d got %d\n",
                             context, expected->message, expected->id, actual->id);
                     }
                 }
                 else
                 {
                     ok_(file, line) (expected->id == actual->id,
-                        "%s: in msg 0x%04x expecting id 0x%x got 0x%x\n",
+                        "%s: in msg 0x%04x expecting id %d got %d\n",
                         context, expected->message, expected->id, actual->id);
                     if (expected->id != actual->id) dump++;
                 }

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=105915

Your paranoid android.

=== w10pro64_ar (64 bit report) ===

comctl32:
datetime.c:798: Test failed: Expected 25/01/2022, got 2022/01/25

=== w864 (testbot log) ===

WineRunTask.pl:error: The previous 1 run(s) terminated abnormally

[Dmitry Timoshkov]

Hello,

it would be nice to have some feedback of this.

[Dmitry Timoshkov]

Dmitry Timoshkov  wrote:
> Hello,
>
> it would be nice to have some feedback of this.

Please find another reviewer to these patches.