Re: [PATCH v9 0/10] MR4982: Load and play MIDI

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/dmime/midi.c:
> +static inline struct midi_sequence_track *impl_from_IDirectMusicTrack8(IDirectMusicTrack8 *iface)
> +{
> - return CONTAINING_RECORD(iface, struct midi_sequence_track, IDirectMusicTrack8_iface);
>
> +}
> -
>
> +static HRESULT WINAPI midi_sequence_track_QueryInterface(IDirectMusicTrack8 *iface, REFIID riid, void **ret_iface)
> +{
> - struct midi_sequence_track *This = impl_from_IDirectMusicTrack8(iface);
> -
> - TRACE("(%p, %s, %p)\n", This, debugstr_dmguid(riid), ret_iface);
> -
> - if (IsEqualIID(riid, &IID_IUnknown) || IsEqualIID(riid, &IID_IDirectMusicTrack) ||
> -
>        IsEqualIID(riid, &IID_IDirectMusicTrack8))
> -
>    *ret_iface = &This->IDirectMusicTrack8_iface;
> - else if (IsEqualIID(riid, &IID_IPersistStream)) *ret_iface = &This->dmobj.IPersistStream_iface;
> - else

```suggestion:-4+0
    if (IsEqualIID(riid, &IID_IUnknown) ||
        IsEqualIID(riid, &IID_IDirectMusicTrack) ||
        IsEqualIID(riid, &IID_IDirectMusicTrack8))
        *ret_iface = &This->IDirectMusicTrack8_iface;
    else if (IsEqualIID(riid, &IID_IPersistStream))
        *ret_iface = &This->dmobj.IPersistStream_iface;
    else
```