[FFmpeg-devel] [PATCH v2 29/33] avutil/{color_utils, csp}: merge color_utils into csp and expose API

Ronald S. Bultje rsbultje at gmail.com
Sat Feb 4 18:27:46 EET 2023


On Sat, Feb 4, 2023 at 11:50 AM Anton Khirnov <anton at khirnov.net> wrote:

> From: Leo Izen <leo.izen at gmail.com>
> libavutil/color_utils contains some avpriv_ symbols that map
> enum AVTransferCharacteristic values to gamma-curve approximations and
> to the actual transfer functions to invert them (i.e. -> linear).
> There's two issues with this:
> (1) avpriv is evil and should be avoided whenever possible
> (2) libavutil/csp.h exposes a public API for handling color that
>     already handles primaries and matricies
> I don't see any reason this API has to be private, so this commit takes
> the functionality from avutil/color_utils and merges it into avutil/csp
> with an exposed av_ API rather than the previous avpriv_ API.
> Every reference to the previous API has been updated to point to the
> new one. color_utils.h has been deleted as well. This should not break
> any applications as it only contained avpriv_ symbols in the first
> place, so nothing in that header could be referenced by other
> applications.

Ok with me.


More information about the ffmpeg-devel mailing list