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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/dmime/audiopath.c:
> -
>        break;
> -
>    default:
> -
>        WARN("Unknown %s\n", debugstr_chunk(&chunk));
> -
>        break;
> -
>    }
> - }
>
> +done:
> - if (FAILED(hr)) {
> -
>    if (pchannel_to_buffer)
> -
>    {
> -
>        free(pchannel_to_buffer->guids);
> -
>        free(pchannel_to_buffer);
> -
>    }
> -
>    return hr;
> - }
> - return S_OK;

You can combine the ifs with && and return hr. Also with a single alloc you could even get rid of the `if (pchannel_to_buffer)`.