[FFmpeg-devel] [PATCH 1/3] vaapi_encode_h264: Fix setting colour properties

Jan Ekström jeebjp at gmail.com
Tue Nov 3 13:16:42 EET 2020


On Tue, Nov 3, 2020 at 12:59 AM Mark Thompson <sw at jkqxz.net> wrote:
>
> The properties should always be set; only the presence flags want to be
> conditional.
>
> Fixes #8959.
> ---

Thanks for this on such short notice. Patch set LGTM, and simplifies
the logic nicely for these values (always set, only set flags if
required).

Tested the H.264 and MPEG-2 encoders locally with my X230, with and
without additional color space info with the general command mentioned
in #8959. Unfortunately I lack the hardware to test HEVC for now.

Jan


More information about the ffmpeg-devel mailing list