mbox series

[FFmpeg-devel,v1,0/4] webp: add support for animated WebP decoding

Message ID 20230608142029.16564-1-thilo.borgmann@mail.de
Headers show
Series webp: add support for animated WebP decoding | expand

Message

Thilo Borgmann June 8, 2023, 2:20 p.m. UTC 
Follow-up to [1], the decoder only.

In [1] we found a hanging sample. In frame-threading, this led to an
infinite wait, damaged frames otherwise. So warn acordingly and do
best-effort in single-thread mode.

Also split into more patches to make reviewing easier.

[1] http://ffmpeg.org/pipermail/ffmpeg-devel/2023-April/308965.html

Josef Zlomek (1):
  libavcodec/webp: add support for animated WebP decoding

Thilo Borgmann (3):
  avcodec/webp: move definitions into header
  avcodec/webp_parser: parse each frame into one packet
  avcodec/webp: make init_canvas_frame static

 Changelog                |   1 +
 libavcodec/codec_desc.c  |   3 +-
 libavcodec/version.h     |   2 +-
 libavcodec/webp.c        | 724 +++++++++++++++++++++++++++++++++++----
 libavcodec/webp.h        |  55 +++
 libavcodec/webp_parser.c | 132 ++++---
 6 files changed, 800 insertions(+), 117 deletions(-)
 create mode 100644 libavcodec/webp.h

Comments

James Zern June 14, 2023, 2:02 a.m. UTC | #1 
On Thu, Jun 8, 2023 at 7:20 AM Thilo Borgmann <thilo.borgmann@mail.de> wrote:
>
> Follow-up to [1], the decoder only.
>
> In [1] we found a hanging sample. In frame-threading, this led to an
> infinite wait, damaged frames otherwise. So warn acordingly and do
> best-effort in single-thread mode.
>
> Also split into more patches to make reviewing easier.
>
> [1] http://ffmpeg.org/pipermail/ffmpeg-devel/2023-April/308965.html
>
> Josef Zlomek (1):
>   libavcodec/webp: add support for animated WebP decoding
>
> Thilo Borgmann (3):
>   avcodec/webp: move definitions into header
>   avcodec/webp_parser: parse each frame into one packet
>   avcodec/webp: make init_canvas_frame static
>
>  Changelog                |   1 +
>  libavcodec/codec_desc.c  |   3 +-
>  libavcodec/version.h     |   2 +-
>  libavcodec/webp.c        | 724 +++++++++++++++++++++++++++++++++++----
>  libavcodec/webp.h        |  55 +++
>  libavcodec/webp_parser.c | 132 ++++---
>  6 files changed, 800 insertions(+), 117 deletions(-)
>  create mode 100644 libavcodec/webp.h
>

This has a few rendering / framerate issues. I'll send you some samples.