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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/dmime/midi.c:
> -
>    hr = stream_read_accumulate(stream, &byte, 1, total_bytes_read);
> -
>    if (hr != S_OK) return hr;
> -
>    event->type = byte;
> -
> -
>    hr = read_variable_length_number(stream, &length, total_bytes_read);
> -
>    if (hr != S_OK) return hr;
> -
> -
>    if (is_event_fixed_length(event->status, event->type))
> -
>    {
> -
>        if (length > sizeof(event->data.fixed.byte))
> -
>        {
> -
>            WARN("Invalid MIDI meta event length %lu for type %#02x\n", length, event->type);
> -
>            return E_FAIL;
> -
>        }
> -
>        hr = stream_read_accumulate(stream, &event->data.fixed.byte[0], length, total_bytes_read);
> -
>        if (hr == S_OK)

```suggestion:-1+0
            hr = stream_read_accumulate(stream, &event->data.fixed.byte[0], length, total_bytes_read);
            if (hr != S_OK) return hr;
```