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.