[FFmpeg-devel] [PATCH v2 3/6] avcodec: add API for automatic handling of icc profiles

[Anton Khirnov]

Quoting Niklas Haas (2022-06-29 12:12:48)
> From: Niklas Haas <git at haasn.dev>
> 
> This functionally already exists, but as pointed out in #9672 and #9673,
> requiring users to manually include filters is clumsy, error-prone and
> hard to use together with tools like ffplay.
> 
> To streamline ICC profile support, add a new AVCodecContext option to
> globally enable reading and writing ICC profiles, automatically, for all
> appropriate media types.
> 
> Note that this commit only includes the new API. The implementation is
> split off to separate commits for readability.
> 
> Signed-off-by: Niklas Haas <git at haasn.dev>
> ---
>  doc/APIchanges       |  4 ++++
>  libavcodec/avcodec.h | 10 ++++++++++
>  libavcodec/version.h |  2 +-
>  3 files changed, 15 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/APIchanges b/doc/APIchanges
> index 20b944933a..b8acf86352 100644
> --- a/doc/APIchanges
> +++ b/doc/APIchanges
> @@ -14,6 +14,10 @@ libavutil:     2021-04-27
>  
>  API changes, most recent first:
>  
> +2022-06-28 - xxxxxxxxx - lavc 59.35.100 - avcodec.h
> +  Add the icc_profiles option to AVCodecContext, to enable automatic reading
> +  and writing of embedded ICC profiles in image files.
> +
>  2022-06-12 - xxxxxxxxxx - lavf 59.25.100 - avio.h
>    Add avio_vprintf(), similar to avio_printf() but allow to use it
>    from within a function taking a variable argument list as input.
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 4dae23d06e..8ee9be22d0 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -979,6 +979,16 @@ typedef struct AVCodecContext {
>       */
>      enum AVChromaLocation chroma_sample_location;
>  
> +    /**
> +     * Whether to decode/encode ICC profiles. If set, libavcodec will
> +     * generate/parse ICC profiles as appropriate for the type of file. No
> +     * effect on codecs which cannot contain embedded ICC profiles, or when
> +     * compiled without support for lcms2.
> +     * - encoding: Set by user
> +     * - decoding: Set by user
> +     */
> +    int icc_profiles;

This is an ABI break. You could also just add a new AV_CODEC_FLAG
instead.

Also, why should this not be enabled by default?

-- 
Anton Khirnov