Re: [PATCH v6 0/5] MR3572: mf: Make session_get_node_object() more robust.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c:
>  IMFSampleGrabberSinkCallback_Release(grabber_callback);
>
> }
>
> +/* create a media session with the specified source and sink */
> +static void create_media_session(IMFMediaSource *source, IMFActivate *sink_activate,
> -
>    IMFMediaSession **session)
>
> +{

Having a helper for that is fine, but it's better to call it create_test_topology() and make it do just that, instead of creating session, topology, and setting topology.