Re: [PATCH 0/4] MR8330: comctl32/listview: Miscellaneous fixes for uninitialized and unnecessarily initialized variables.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/comctl32/tests/listview.c:
> expect(1, res);
>      res = SendMessageA(hwnd, LVM_GETITEMCOUNT, 0, 0);
>      expect(1, res);
> +    res = SendMessageA(hwnd, LVM_GETITEMSTATE, 0, 0xff);
> +    todo_wine expect(2, res);
>      DestroyWindow(hwnd);

If this is specific to LVS_OWNERDATA path, we need more tests with callback mask. If it's not specific to LVS_OWNERDATA, the test should be placed elsewhere.

Note that return value is not an index, so please compare with defined state constants, and without expect(), just plain ok().

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8330#note_106926