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:
> +            IMFTopologyNode_Release(node);
> +            continue;
> +        }
> +        if (FAILED(hr = IMFTopologyNode_GetInputCount(node, &input_count)))
> +        {
> +            IMFTopologyNode_Release(node);
> +            continue;
> +        }
> +
> +        for (j = 0; j < input_count; ++j)
> +        {
> +            IMFMediaTypeHandler *handler;
> +            IMFTopologyNode *up_node;
> +            DWORD input;
> +
> +            if (SUCCEEDED(hr = IMFTopologyNode_GetInput(node, j, &up_node, &input)))
> I think this should be named `up_output` or `up_index` instead, it's not the input index of the current node, but rather the output index of the upstream node.

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