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

[Rémi Bernon]

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;
        }
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1262#note_13330