Re: [PATCH v6 0/2] MR7796: winegstreamer: Allow caller to allocate samples in MF_XVP_PLAYBACK_MODE.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/winegstreamer/video_processor.c:
>  if (FAILED(hr = IMFTransform_GetOutputStreamInfo(iface, 0, &info)))
>      return hr;
>
> - if (impl->output_info.dwFlags & MFT_OUTPUT_STREAM_PROVIDES_SAMPLES)
> - if (FAILED(IMFAttributes_GetUINT32(impl->attributes, &MF_XVP_PLAYBACK_MODE, (UINT32 *) &use_caller_samples)))
> -
>    use_caller_samples = FALSE;
> -
> - use_caller_samples = use_caller_samples || !(impl->output_info.dwFlags & MFT_OUTPUT_STREAM_PROVIDES_SAMPLES);
> -
> - if (!use_caller_samples)

I would rather use non-negated condition, like "if (provides_samples)", but it's not important.