Re: [FFmpeg-devel] [PATCH v7 0/7] webp: add support for animated WebP decoding

[Michael Niedermayer <michael@niedermayer.cc>]

[-- Attachment #1.1: Type: text/plain, Size: 1483 bytes --]

On Wed, Dec 06, 2023 at 02:34:58AM +0100, Thilo Borgmann via ffmpeg-devel wrote:
> Still images fixed, includes FATE tests, VP8 decoder decoupled so there are no more data races, fixed more asserts, fixed ffprobe regression.
> 
> Patch 5/7 is still there for making changes in lavc/webp reviewable but shall be stashed when pushing.
> 
> -Thilo
> 
> Josef Zlomek (2):
>   libavcodec/webp: add support for animated WebP decoding
>   libavformat/webp: add WebP demuxer
> 
> Thilo Borgmann (5):
>   avcodec/webp: move definitions into header
>   avcodec/webp: remove unused definitions
>   avcodec/webp_parser: parse each frame into one packet
>   avcodec/webp: make init_canvas_frame static
>   fate: add test for animated WebP

something in this patch leads to new warnings:

./ffmpeg -i fate-suite/exif/image_small.webp -vcodec copy -bitexact  file-4087B.webp
./ffmpeg -i file-4087B.webp -f null -

after this patchset

./ffmpeg -i fate-suite/exif/image_small.webp -vcodec copy -bitexact  file-4087B.webp
./ffmpeg -i file-4087B.webp -f null -
[webp @ 0x55a5284bcb80] EXIF chunk present, but Exif bit not set in the VP8X header

something similar happens with "ALPHA chunk present, but alpha bit not set in the VP8X header"
with ticket 4087/5_webp_a.webp

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is a danger to trust the dream we wish for rather than
the science we have, -- Dr. Kenneth Brown

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
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".