Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/toolbar.c:
>
> return 0;
> }
> + case TEST_WM_NOTIFY_CODE:
> + if (expecting_test_WM_NOTIFY)
> + {
> + ok(hdr == &test_WM_NOTIFY_nmhdr, "Got unexpected header.\n");
> + ok(wParam == TEST_WM_NOTIFY_IDFROM, "Got unexpected wParam 0x%Ix.\n", wParam);
> + }
> + else
This `else` branch should never get executed because only the test_WM_NOTIFY() test sends a WM_NOTIFY with TEST_WM_NOTIFY_CODE. So you can delete this branch.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/6737#note_86459