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

[James Zern]

On Wed, Jul 26, 2023 at 2:36 PM Tomas Härdin <git at haerdin.se> wrote:
>
> 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.
>

We wanted something lighter weight (memory, cpu) for an image format
rather than going full blown video. Lossless also factored into this.

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