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().