mbox series

[FFmpeg-devel,0/2] PNG cICP improvements

Message ID 20230201171227.57864-1-leo.izen@gmail.com
Headers show
Series PNG cICP improvements | expand


Leo Izen Feb. 1, 2023, 5:12 p.m. UTC
The first patch fixes a FATE issue.

The second patch was discussed on IRC. I got a LGTM from Kasper, but sending to ML here and applying in a few days if there's no objections.

A few things discussed:
1) There's no need to expose the gamma table as another function as it's only used here.
2) This patch includes the patch on the ML sent as "avcodec/pngdec: warn when reading tv-range cICP chunks" which got a LGTM from Lynne on IRC. If this is applied that one has no need to be applied.

Leo Izen (2):
  avcodec/pngenc.c: avoid writing cICP when inappropriate
  avcodec/pngdec: read colorspace info when decoding with AVDISCARD_ALL

 libavcodec/pngdec.c | 126 ++++++++++++++++++++++++++------------------
 libavcodec/pngenc.c |  21 +++++---
 2 files changed, 89 insertions(+), 58 deletions(-)