Re: [PATCH v5 1/6] wmp: Implement playback via quartz

[Jacek Caban]

Hi Anton,

On 03/28/2018 06:00 AM, Anton Romanov wrote:
> +    if (!This->filter_graph) {
> +        return NS_S_WMPCORE_COMMAND_NOT_AVAILABLE;
> +    }
> +    if (This->media_control) {
> +        hres = IMediaControl_Stop(This->media_control);
> +        IMediaControl_Release(This->media_control);
> +    }
> +    if (!This->filter_graph) {

This doesn't look right, did you mean if (This->filter_graph)? Note that
you return earlier if this is NULL, do you probably don't need that check.
> +        IGraphBuilder_Release(This->filter_graph);
> +    }

Thanks,

Jacek