Jon Sneyers

This one doesn't reproduce anymore in the current version (cfd25e57578ccd047dd2177aea2924f5a3fa1e5f). Instead of a heap overflow, it now results in a memleak.

Makes sense. I don't currently have time to make these changes, but if anyone feels like making a pull request... go ahead!

I added a comment in flif_enc.h (https://github.com/FLIF-hub/FLIF/blob/master/src/library/flif_enc.h#L59), I hope that is enough. The default is indeed to not encode RGB values of fully transparent pixels. Whether or not that is...

I suppose if you want to integrate the decoder by copying the code (instead of linking to the library), then having a namespace would help. Not sure why you would...

LGTM.

Probably a good idea to also report this to clang?

Could this be modified to put the `JPEGXL_LIBRARY_SOVERSION` in the .h file instead of the major/minor/patch version? The SOVERSION tells you exactly what API is available, so it is what...

> > Could this be modified to put the `JPEGXL_LIBRARY_SOVERSION` in the .h file instead of the major/minor/patch version? > > The SOVERSION tells you exactly what API is available,...

We could in principle expose the list of global modular transforms, which can include global palette. One caveat is that a 3- or 4-channel palette transform without delta entries and...

You have to use `d` in benchmark_xl, the `q` option is ignored for jxl (it's only there for other codecs like webp).