Re: [PATCH 0/3] MR8561: msado15 check all interfaces in hierarchy

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msado15/recordset.c:
> struct fields *fields = impl_from_Fields( iface );
>      TRACE( "%p, %s, %p\n", iface, debugstr_guid(riid), obj );
>
> -    if (IsEqualGUID( riid, &IID_Fields ) || IsEqualGUID( riid, &IID_IDispatch ) ||
> +    if (IsEqualGUID( riid, &IID_Fields ) ||
> +        IsEqualGUID( riid, &IID_Fields20 ) ||
> +        IsEqualGUID( riid, &IID_Fields15 ) ||
> +        IsEqualGUID( riid, &IID_Fields ) ||

You're checking for `IID_Fields` twice.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8561#note_109744