Re: [PATCH v2 0/3] MR3536: mfmediaengine: Implement media_engine_GetSeekable().

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> -
>    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
>    ok(start == 0, "Unexpected start %lf.\n", start);
> -
>    hr = IMFMediaTimeRange_GetEnd(time_range, 0, &end);
> -
>    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
>    duration = IMFMediaEngineEx_GetDuration(media_engine);
> -
>    ok(end == duration, "Unexpected end %lf.\n", end);
> -
>    IMFMediaTimeRange_Release(time_range);
> - }
> -
> - /* Media engine is shut down */
> - IMFMediaEngineEx_Shutdown(media_engine);
> -
> - hr = IMFMediaEngineEx_GetSeekable(media_engine, &time_range);
> - ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#lx.\n", hr);
> -
> - IMFMediaEngineEx_Release(media_engine);

Please add return value check for Shutdown(), out pointer check for GetSeekable(), and a refcount check for Release().