Nico Weber
Nico Weber
https://github.com/ninja-build/ninja/issues/843 has some background on why ninja does what it does. We tried canonicalizing everything to `/` first, but cmake's cmcldeps tries to find the `/fo` switch and just basically...
I have no memory of requesting this.
I'm not married to adding an `animation` tool – maybe `image` should be able to do this, and there a thousand other alternative viable approaches. For now, this is simple...
Red "Lagom / CI (macOS, macos-14) (pull_request)" is unrelated.
> Maybe worth having a test that tests that a black-and-white bitmap roundtrips ok through gif writer followed by gif loader? I added a roundtrip test for webp to #24212....
(Maybe you forgot to `git push`?)
#24226 turned this into a dbgln() probably.
(It'd generally be nice if animated webp would work like animated gif.)
Also, it looks like the EXIF chunk is added at the front. Per https://developers.google.com/speed/webp/docs/riff_container#extended_file_format that's invalid: EXIF must follow image data (which for animated files is a sequence of ANMF...
Let me know if I can help answer any questions. I know a bit about webp format internals.