On Tue, Aug 8, 2017 at 4:14 PM, Vittorio Giovara
<vittorio.giov...@gmail.com> wrote:
> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com>
> ---
> Missing version bump.
> Vittorio
>
> libavutil/pixdesc.c | 4 ++++
> libavutil/pixfmt.h | 3 +++
> 2 files changed, 7 insertions(+)
>
> diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c
> index 7fa6dd7c0b..a72ef3faf0 100644
> --- a/libavutil/pixdesc.c
> +++ b/libavutil/pixdesc.c
> @@ -1806,6 +1806,10 @@ static const char * const color_space_names[] = {
> [AVCOL_SPC_BT2020_NCL] = "bt2020nc",
> [AVCOL_SPC_BT2020_CL] = "bt2020c",
> [AVCOL_SPC_SMPTE2085] = "smpte2085",
> + [AVCOL_SPC_CHROMA_DERIVED_NCL] = "chroma-derived-nc",
> + [AVCOL_SPC_CHROMA_DERIVED_CL] = "chroma-derived-c",
> + [AVCOL_SPC_ICPCT] = "icpct",
> +
> };
>
> static const char * const chroma_location_names[] = {
> diff --git a/libavutil/pixfmt.h b/libavutil/pixfmt.h
> index 2ba7ad1c88..93fd95da58 100644
> --- a/libavutil/pixfmt.h
> +++ b/libavutil/pixfmt.h
> @@ -384,6 +384,9 @@ enum AVColorSpace {
> AVCOL_SPC_BT2020_NCL = 9, ///< ITU-R BT2020 non-constant luminance
> system
> AVCOL_SPC_BT2020_CL = 10, ///< ITU-R BT2020 constant luminance system
> AVCOL_SPC_SMPTE2085 = 11, ///< SMPTE 2085, Y'D'zD'x
> + AVCOL_SPC_CHROMA_DERIVED_NCL = 12, ///< Chromaticity-derived
> non-constant luminance system
> + AVCOL_SPC_CHROMA_DERIVED_CL = 13, ///< Chromaticity-derived constant
> luminance system
> + AVCOL_SPC_ICPCT = 14, ///< ITU-R BT.2100-0 ICTCP
> AVCOL_SPC_NB, ///< Not part of ABI
> };
>
The spelling of ICtCp is not consistent (differs between comment and
symbol). From what I can see, ICtCp should be the correct order
according to the standards, and not ICpCt, so please use that
consistently. :)
- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
