[FFmpeg-devel] [PATCH v4 3/7] avcodec/webp_parser: parse each frame into one packet

[Thilo Borgmann]

Am 25.07.23 um 14:24 schrieb Tomas Härdin:
>> +    // Extremely simplified key frame detection:
>> +    // - the first frame (containing headers) is marked as a key
>> frame
>> +    // - other frames are marked as non-key frames
> 
> Is there a more proper way of doing this? 

All frames (except the ANMF chunks) are INTRA, and all of them have a WEBP tag.
Whereas all ANMF chunks are in the same WEBP chunk as their reference frame.
So it should really be as simple as it is to mark all WEBP frames as key frames as the code does.
What more dedicated do you have in mind?

The logic as-is works with all samples I have, animated and not.
Seems to also align well with their example file layouts.
You have a more weird one?


> Looking briefly at the spec
> one wonders why they didn't just use regular VP* inter frames..

I assume the whole canvas idea could be more beneficial than vp8 inter - but otoh I don't know about vp8 compositing capabilities, if any..

-Thilo