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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/dmime/midi.c:
> -
> - if (!ref) free(This);
> -
> - return ref;
>
> +}
> -
>
> +static HRESULT WINAPI midi_sequence_track_Init(IDirectMusicTrack8 *iface, IDirectMusicSegment *pSegment)
> +{
> - struct midi_sequence_track *This = impl_from_IDirectMusicTrack8(iface);
> - FIXME("(%p, %p): stub\n", This, pSegment);
> - return S_OK;
>
> +}
> -
>
> +static HRESULT WINAPI midi_sequence_track_InitPlay(IDirectMusicTrack8 *iface,
> -
>    IDirectMusicSegmentState *pSegmentState, IDirectMusicPerformance *pPerformance,
> -
>    void **ppStateData, DWORD dwVirtualTrack8ID, DWORD dwFlags)

Please get rid of the hungarian notation. Same comment for the other functions here.