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

[Bernhard Kölbl (＠besentv)]

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.