Re: [PATCH v7 0/2] MR6160: comctl32: Fix sorting for listview.

[Zhiyi Zhang (＠zhiyi)]

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.