Re: [PATCH 0/4] MR789: mf: Set source and output nodes media types in the media session.

[Bernhard Kölbl]

Bernhard Kölbl (@besentv) commented about dlls/mf/session.c:
> +    {
> +        if (FAILED(hr = IMFTopology_GetNode(topology, i, &node)))
> +            break;
> +
> +        if (FAILED(hr = IMFTopologyNode_GetNodeType(node, &node_type))
> +                || node_type != MF_TOPOLOGY_OUTPUT_NODE)
> +        {
> +            IMFTopologyNode_Release(node);
> +            continue;
> +        }
> +        if (FAILED(hr = IMFTopologyNode_GetInputCount(node, &input_count)))
> +        {
> +            IMFTopologyNode_Release(node);
> +            continue;
> +        }
> +        if (FAILED(hr = IMFTopologyNode_GetOutputCount(node, &output_count)))
> Just a few comments about things that I came across while debugging this commit:

Output nodes don't have outputs, so iterating over them doesn't seem necessary here.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/789#note_12740