[FFmpeg-devel] [PATCH v7 1/1] avutil/csp: create public API for colorspace structs

[Ronald S. Bultje]

Hi,

On Sat, May 28, 2022 at 9:30 AM Leo Izen <leo.izen at gmail.com> wrote:

> This commit moves some of the functionality from avfilter/colorspace
> into avutil/csp and exposes it as a public API so it can be used by
> libavcodec and/or libavformat. It also converts those structs from
> double values to AVRational to make regression testing easier and
> more consistent.
> ---
>  libavfilter/colorspace.c    | 143 ++++++++----------------------------
>  libavfilter/colorspace.h    |  31 +-------
>  libavfilter/fflcms2.c       |  33 +++++----
>  libavfilter/fflcms2.h       |   4 +-
>  libavfilter/vf_colorspace.c |  37 +++++-----
>  libavfilter/vf_iccdetect.c  |   5 +-
>  libavfilter/vf_tonemap.c    |  17 +----
>  libavutil/Makefile          |   2 +
>  libavutil/csp.c             | 128 ++++++++++++++++++++++++++++++++
>  libavutil/csp.h             | 106 ++++++++++++++++++++++++++
>  libavutil/version.h         |   2 +-
>  11 files changed, 317 insertions(+), 191 deletions(-)
>  create mode 100644 libavutil/csp.c
>  create mode 100644 libavutil/csp.h
>

As far as I can see, all comments have been addressed and there's support
for merging this from multiple people (including me). I intend to push this
in 24 hrs if there are no further comments. Please comment if you disagree.

Ronald