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

[Rémi Bernon (＠rbernon)]

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;

```suggestion:-9+0
        }

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.