Add AVIF support #36
Add AVIF support #36
Conversation
libheif needs those, and disabling them just for libheif leads to linking issues. I could probably disable them in linker too only if AVIF is enabled, but not sure it's worth the complexity.
This has issues with -frtti which is now enabled for libheif.
|
Converted to draft because, aside from tests, this clearly needs more work to be supported as a dynamic module. |
Depends on commit: strukturag/libheif@024d32c
Resolves a `BindingError` during dynamic lib load. ``` BindingError: Cannot register public name 'heif_error_code' twice ``` See commit: kleisauke/libheif@34dda27
The testsuite passes without increasing the stack size.
|
Great! I was able to support this as a dynamic module and also added some test coverage for this (by porting libvips' AVIF tests). |
Feel free to push directly to my PR, might be easier than tracking across different branches. |
|
Or, alternatively, I can close mine and you open yours. Whichever is easier. |
I just pushed it to the branch associated with this PR. |
Note this currently doesn't port upstream tests; not sure if I'll get around to that, but figured I'll submit PR anyway.