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:
>  ok(r, "Failed to get item.\n");
> -
>
> todo_wine ok(item.iImage == 500, "Unexpected iImage value %d.\n", item.iImage);
>
> ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "get image dispinfo 2", FALSE);
> -
>
> r = SendMessageA(hwnd, LVM_SETEXTENDEDLISTVIEWSTYLE, 0, LVS_EX_SUBITEMIMAGES);
> -
>
> ok(!r, "Unexpected return value %d.\n", r);
> -
> -
>
> memset(&item, 0, sizeof(item));
> -
>
> item.mask = LVIF_IMAGE;
> -
>
> item.iSubItem = 1;
> -
>
> item.iImage = 500;
> -
>
> r = SendMessageA(hwnd, LVM_GETITEMA, 0, (LPARAM)&item);
> -
>
> ok(r, "Failed to get item.\n");
> -
>
> ok(item.iImage == I_IMAGECALLBACK, "Unexpected iImage value %d.\n", item.iImage);
> -

The test looks fine, but I think using some value that's not I_IMAGECALLBACK would be better.