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:
> -
>        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.