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

[Tomas Härdin]

tis 2023-07-25 klockan 16:18 +0200 skrev 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?

Nah mostly just curious. It just feels so weird when VP8 intra already
exists. Maybe I'm missing something. Browsers already support VP8 after
all.

> 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?

Nope

/Tomas