mbox series

[FFmpeg-devel,0/1,WIP] avutil/csp changes

Message ID 20220520155313.171100-1-leo.izen@gmail.com
Headers show
Series avutil/csp changes | expand

Message

Leo Izen May 20, 2022, 3:53 p.m. UTC 
This patch is a work in progress example for swapping these
structs from doubles to AVRationals.

There's two main discussions here to be had
- Is this API to be exposed as avpriv_ or av_?
- Should these structs use AVRational or double values?

I don't believe a consensus has been reached on this yet, but I've attached
an AVRational version of it so we can see the pros/cons.

Leo Izen (1):
  avutil/csp: create public API for colorspace structs

 libavfilter/colorspace.c    | 143 ++++++++----------------------------
 libavfilter/colorspace.h    |  31 +-------
 libavfilter/fflcms2.c       |  25 ++++---
 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             | 121 ++++++++++++++++++++++++++++++
 libavutil/csp.h             |  49 ++++++++++++
 libavutil/version.h         |   4 +-
 11 files changed, 250 insertions(+), 188 deletions(-)
 create mode 100644 libavutil/csp.c
 create mode 100644 libavutil/csp.h