[FFmpeg-devel] [PATCH v6 0/5] JPEG XL Parser (and bug fixes)

[Leo Izen <leo.izen@gmail.com>]

Changes from v5:
- Attached an extra commit to fix existing bug with libjxldec
- Made various changes based on comments by Andreas Reinhardt
-- removed jpegxl_parse.c from avformat, and jpegxl_parse.o from avformat/Makefile/OBJS
-- checked for failure in init_vlc_lengths
-- used a macro to initialize VLCElem structs
-- used heap allocations for a buffer of size 250k, instead of stack
-- renamed "code" to "len"
- Fix demuxer to avoid using avio_size, breaking piped input
- Fix some parser bugs discovered during more extensive testing

Changes from v4:
- Added an entropy decoder and full parser, which finds the
    boundaries between files correctly
- Removed unnecessary logging in libjxldec

Changes from v3:

- Don't remove AV_CODEC_CAP_DR1 from libjxldec
- jpegxl_parse.o added to STLIBOBJS in avcodec/Makefile
- add pipe demuxer to avformat/Makefile's SHLIBOBJS

Changes from v2:

- Fix libjxldec to work with packets that are smaller than one frame
- Change how code is shared between libavcodec and libavformat to be more sensible.
- Fix the parser to work with large headers that proceed the codestream in a container format
    (for example, if several-KB Exif boxes preceed the codestream.)
- Modify the parser to set width/height instead of avctx
  Note: avctx->pix_fmt and s->format are both set, because otherwise the CLI tools won't print
    the pixel format without libjxl enabled.
- Update the fate test based on the new parser's packetization

This test relies on FATE samples that haven't been uploaded yet. To test, unzip
the following zipfile[1] in the FATE_SAMPLES directory, placing the test images in jxl/.

[1]: https://buzo.us/y.zip

sha256sum: 43a2eeb0dfdf471b47a9fdfb1653974fa156ceceb776891cc137569a8ebf0e75
signature: https://buzo.us/R.asc


Leo Izen (5):
  avcodec/libjxldec: fix errors when decoding grayscale after rgb
  avcodec/libjxldec: use internal AVFrame as buffered space
  avcodec/jpegxl_parser: add JPEG XL parser
  avformat/jpegxl: remove jpegxl_probe, instead call
    avcodec/jpegxl_parse
  fate/jpegxl_anim: add demuxer fate test for jpegxl_anim

 libavcodec/Makefile                    |    3 +
 libavcodec/jpegxl.h                    |   94 ++
 libavcodec/jpegxl_parse.c              |  520 +++++++++
 libavcodec/jpegxl_parse.h              |   72 ++
 libavcodec/jpegxl_parser.c             | 1477 ++++++++++++++++++++++++
 libavcodec/libjxldec.c                 |   41 +-
 libavcodec/parsers.c                   |    1 +
 libavcodec/version.h                   |    2 +-
 libavformat/Makefile                   |    6 +-
 libavformat/img2dec.c                  |    4 +-
 libavformat/jpegxl_anim_dec.c          |  132 +--
 libavformat/jpegxl_probe.c             |  412 -------
 libavformat/jpegxl_probe.h             |   37 -
 libavformat/version.h                  |    2 +-
 tests/Makefile                         |    1 +
 tests/fate/jxl.mak                     |   16 +
 tests/ref/fate/jxl-anim-demux-belgium  |    6 +
 tests/ref/fate/jxl-anim-demux-icos4d   |    6 +
 tests/ref/fate/jxl-anim-demux-lenna256 |    6 +
 tests/ref/fate/jxl-anim-demux-newton   |    6 +
 20 files changed, 2270 insertions(+), 574 deletions(-)
 create mode 100644 libavcodec/jpegxl.h
 create mode 100644 libavcodec/jpegxl_parse.c
 create mode 100644 libavcodec/jpegxl_parse.h
 create mode 100644 libavcodec/jpegxl_parser.c
 delete mode 100644 libavformat/jpegxl_probe.c
 delete mode 100644 libavformat/jpegxl_probe.h
 create mode 100644 tests/fate/jxl.mak
 create mode 100644 tests/ref/fate/jxl-anim-demux-belgium
 create mode 100644 tests/ref/fate/jxl-anim-demux-icos4d
 create mode 100644 tests/ref/fate/jxl-anim-demux-lenna256
 create mode 100644 tests/ref/fate/jxl-anim-demux-newton

-- 
2.41.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".