Re: [PATCH 0/1] MR10850: msado15: Correctly increase the cnt value

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msado15/command.c:
> free(para);
>              break;
>          }
> +        cnt++;

While the change is correct how about improving the function further. Here's a list of issues with current implementation:

* validate index, it's enough to print FIXME message if `V_VT(&index) != VT_I4` and return `E_NOTIMPL` 
* you can use `LIST_FOR_EACH` instead of `LIST_FOR_EACH_ENTRY_SAFE`
* the function should return `MAKE_ADO_HRESULT(adErrItemNotFound)` if the item doesn't exist

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10850#note_139363