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?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2778#note_36957