image-rs / image

Encoding and decoding images in Rust
Apache License 2.0
4.93k stars 610 forks source link

WebP's `into_frames` iterator returns `None` upon encountering `DecodingError::NoMoreFrames` #2278

Closed porkbrain closed 3 months ago

porkbrain commented 3 months ago

There's one problem: the original regression test swallowed errors because it just counted results without checking for Ok. I fixed that, but the WebP decoding fails on

Error decoding "tests/images/webp/extended_images/advertises_rgba_but_frames_are_rgb.webp" frame 3: IoError(Error { kind: UnexpectedEof, message: "failed to fill whole buffer" })


Closes #2263


I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to choose either at their option.

fintelia commented 3 months ago

This looks good to me. It just needs an un-released bugfix from image-webp to resolve the one failing test