Re: [PATCH v9 0/2] MR2778: mfmediaengine: Add support for video effects.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> - {
> -
>    if (FAILED(hr = IMFSample_SetSampleDuration(samples[0].pSample, duration)))
> -
>        goto done;
> - }
> -
> - if (SUCCEEDED(hr = IMFSample_GetSampleTime(impl->sample_buf, &time)))
> - {
> -
>    if (FAILED(hr = IMFSample_SetSampleTime(samples[0].pSample, time)))
> -
>        goto done;
> - }
> -
> - if (SUCCEEDED(hr = IMFSample_ConvertToContiguousBuffer(samples[0].pSample, &buffer)))
> - {
> -
>    if (FAILED(hr = IMFSample_CopyToBuffer(impl->sample_buf, buffer)))
> -
>        goto done;
> - }

Instead of all that, could you simply return original sample object?