[FFmpeg-devel] [PATCH] vf_colorspace: Add support for film primaries

[Ronald S. Bultje]

Hi,

On Mon, Oct 31, 2016 at 2:43 PM, Vittorio Giovara <
vittorio.giovara at gmail.com> wrote:

> Signed-off-by: Vittorio Giovara <vittorio.giovara at gmail.com>
> ---
> This is the last easy picking, the remaining trc/prm require more
> convoluted
> changes to the filter that will need more discussion.
> Please CC.
> Vittorio
>
>  libavfilter/vf_colorspace.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
> index 4265aa1..997c594 100644
> --- a/libavfilter/vf_colorspace.c
> +++ b/libavfilter/vf_colorspace.c
> @@ -278,6 +278,7 @@ static const struct ColorPrimaries
> color_primaries[AVCOL_PRI_NB] = {
>      [AVCOL_PRI_BT470BG]   = { WP_D65, 0.640, 0.330, 0.290, 0.600, 0.150,
> 0.060,},
>      [AVCOL_PRI_SMPTE170M] = { WP_D65, 0.630, 0.340, 0.310, 0.595, 0.155,
> 0.070 },
>      [AVCOL_PRI_SMPTE240M] = { WP_D65, 0.630, 0.340, 0.310, 0.595, 0.155,
> 0.070 },
> +    [AVCOL_PRI_FILM]      = { WP_C,   0.681, 0.319, 0.243, 0.692, 0.145,
> 0.049 },
>      [AVCOL_PRI_SMPTE431]  = { WP_DCI, 0.680, 0.320, 0.265, 0.690, 0.150,
> 0.060 },
>      [AVCOL_PRI_SMPTE432]  = { WP_D65, 0.680, 0.320, 0.265, 0.690, 0.150,
> 0.060 },
>      [AVCOL_PRI_BT2020]    = { WP_D65, 0.708, 0.292, 0.170, 0.797, 0.131,
> 0.046 },
> @@ -1084,6 +1085,7 @@ static const AVOption colorspace_options[] = {
>      ENUM("bt470bg",      AVCOL_PRI_BT470BG,    "prm"),
>      ENUM("smpte170m",    AVCOL_PRI_SMPTE170M,  "prm"),
>      ENUM("smpte240m",    AVCOL_PRI_SMPTE240M,  "prm"),
> +    ENUM("film",         AVCOL_PRI_FILM,       "prm"),
>      ENUM("smpte431",     AVCOL_PRI_SMPTE431,   "prm"),
>      ENUM("smpte432",     AVCOL_PRI_SMPTE432,   "prm"),
>      ENUM("bt2020",       AVCOL_PRI_BT2020,     "prm"),
> --
> 2.10.0


I can't believe you found content that uses this :D

lgtm.

Ronald