Hi, On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote:
> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> > --- > I couldn't find any reference to the name of the whitepoint used for 431, > so I came up with DCI, since it looks like it is only used there. > Please CC. > Vittorio > > libavfilter/vf_colorspace.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c > index 7e0bafa..4265aa1 100644 > --- a/libavfilter/vf_colorspace.c > +++ b/libavfilter/vf_colorspace.c > @@ -56,6 +56,7 @@ enum Colorspace { > enum Whitepoint { > WP_D65, > WP_C, > + WP_DCI, > WP_NB, > }; > > @@ -268,6 +269,7 @@ static const struct TransferCharacteristics * > static const struct WhitepointCoefficients whitepoint_coefficients[WP_NB] > = { > [WP_D65] = { 0.3127, 0.3290 }, > [WP_C] = { 0.3100, 0.3160 }, > + [WP_DCI] = { 0.3140, 0.3510 }, > }; Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 refers to the two whitepoints here as DCI-P3 D65 and DCI-P3 Theater. Calling one D65 and the other DCI seems confusing in that light (assuming the wikipedia page is correct). I'd call it THEATER or DCI_P3_THEATER, to distinguish it from DCI-P3 D65. Is that OK? Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel