Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/listview.c:
+ sprintf(buffer, "%d", rand() % 100); + SendMessageA(list_view, LVM_INSERTITEMA, 0, (LPARAM) &lvi); + + lvi.iSubItem = 1; + sprintf(buffer, "%d", rand() % 100); + SendMessageA(list_view, LVM_SETITEMTEXTA, i, (LPARAM) &lvi); + } + + SendMessageA(list_view, LVM_SORTITEMSEX, (WPARAM) list_view, (LPARAM) test_CallBackCompareEx); + + for (int i = 1; i < 100; i++) + { + ListView_GetItemTextA(list_view, i - 1, 0, buffer, sizeof(buffer)); + prev_value = atoi(buffer); + + ListView_GetItemTextA(list_view, i, 0, buffer, sizeof(buffer)); There are two spaces before sizeof(buffer).
Otherwise, it looks good. Thanks for your work. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6160#note_77522