[FFmpeg-devel] [PATCH v4 1/2] avcodec/libjxl: add animated JPEG XL encoder

Marth64 marth64 at proxyid.net
Wed Dec 25 00:23:43 EET 2024


Hi Leo,

Code in v4 is clean and LGTM.
Also, I tested today successfully and got a good visual result when
playing back (though I had to use data muxer).
```
ffmpeg -i pizza.mkv -map 0:v -c:v libjxl_anim -f data test.jxl
```

Sorry for the back and forth on this last bit, but in libavcodec/version.c,
can anyone chime in on what is the etiquette here when adding a new Codec ID?
I think that the assert in avcodec_version() needs to be updated,
but should the minor version be updated too?

Or are these changed at release time?
Could this also be accomplished without a new codec ID (eg. treated
like MJPEG/JPEG)?


More information about the ffmpeg-devel mailing list