Re: [PATCH v3 0/2] MR7230: dmime: Fix loading and default soundfont collection.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/dmime/midi.c:
> for (i = 0;; i++)
>      {
> -        BYTE magic[4] = {0}, last_status = 0;
> +        BYTE magic[4], last_status = 0;
>          DWORD length_be;
>          ULONG length;
>          ULONG read = 0;
>          struct midi_event event = {0};
>
> TRACE("Start parsing track %u\n", i);
> -        if ((hr = IStream_Read(parser->stream, magic, sizeof(magic), &read)) != S_OK) break;
> -        if (read < sizeof(magic)) break;
> +        hr = IStream_Read(parser->stream, magic, sizeof(magic), &read);
> +        if (read < sizeof(magic)) {

You're missing a line break before the brace.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7230#note_94005