Re: [PATCH 0/4] MR4852: Add support from AudioPathConfig

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/dmime/audiopath.c:
> +                TRACE("Unfo list in PortConfig is ignored\n");
> +            else
> +                WARN("Unknown %s\n", debugstr_chunk(&chunk));
> +            break;
> +        default:
> +            WARN("Unknown %s\n", debugstr_chunk(&chunk));
> +        }
> +    }
> +
> +done:
> +    if (FAILED(hr)) {
> +        free(port_config);
> +        return hr;
> +    }
> +    list_add_tail(&This->port_config_entries, &port_config->entry);
> +    return S_OK;
        }

        if (FAILED(hr)) break;
    }

    if (FAILED(hr)) free(port_config);
    else list_add_tail(&This->port_config_entries, &port_config->entry);
    return hr;

Like it's done in other recent dmusic changes. I don't really mind either but I prefer a consistent code style.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4852#note_57672