Re: [PATCH v2 0/6] MR1262: mf: Set source and output nodes media types in the media session.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/mf/session.c:
> - for (i = 0; i < node_count; ++i)
> - {
> -
>    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;
> -
>    }

I didn't do it for dubious style reasons but I think we could merge these two, into something like:

```c
        if (FAILED(hr = IMFTopologyNode_GetInputCount(node, &input_count))
                || FAILED(hr = IMFTopologyNode_GetNodeType(node, &node_type))
                || node_type != MF_TOPOLOGY_OUTPUT_NODE)
        {
            IMFTopologyNode_Release(node);
            continue;
        }
```