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

[Ronald S. Bultje]

Hi,

On Tue, May 31, 2022 at 10:26 AM Ronald S. Bultje <rsbultje at gmail.com>
wrote:

> 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.
>

Merged.

Ronald