From: James Zern <jzern-at-google.com@ffmpeg.org>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v3 0/6] webp: add support for animated WebP decoding
Date: Mon, 24 Jul 2023 13:44:49 -0700
Message-ID: <CABWgkX+eVOVzG1hXGp=szF99hwcdqgoeYq8UnZv-obNznoRUTg@mail.gmail.com> (raw)
In-Reply-To: <20230720230752.68246-1-thilo.borgmann@mail.de>
On Thu, Jul 20, 2023 at 4:08 PM Thilo Borgmann <thilo.borgmann@mail.de> wrote:
>
> All issues of v2 fixed. Makes tsan happy now as well.
>
> Patch 5/6 is still there for making changes in lavc/webp reviewable but
> shall be stashed when pushing.
>
This looks to fail fate:
https://patchwork.ffmpeg.org/check/83180/
It seems to regress on still images:
https://www.gstatic.com/webp/gallery3/1_webp_a.webp
[webp @ 0x555880fd3040] [IMGUTILS @ 0x7f34f04a8ce0] Picture size 0x0 is invalid
> -Thilo
>
> Josef Zlomek (2):
> libavcodec/webp: add support for animated WebP decoding
> libavformat/webp: add WebP demuxer
>
> Thilo Borgmann (4):
> avcodec/webp: move definitions into header
> avcodec/webp: remove unused definitions
> avcodec/webp_parser: parse each frame into one packet
> avcodec/webp: make init_canvas_frame static
>
> Changelog | 2 +
> doc/demuxers.texi | 28 +
> libavcodec/codec_desc.c | 3 +-
> libavcodec/version.h | 2 +-
> libavcodec/webp.c | 714 +++++++++++++++++--
> libavcodec/webp.h | 38 +
> libavcodec/webp_parser.c | 130 ++--
> libavformat/Makefile | 1 +
> libavformat/allformats.c | 1 +
> libavformat/version.h | 2 +-
> libavformat/webpdec.c | 733 ++++++++++++++++++++
> tests/ref/fate/exif-image-webp | 12 +-
> tests/ref/fate/webp-rgb-lena-lossless | 2 +-
> tests/ref/fate/webp-rgb-lena-lossless-rgb24 | 2 +-
> tests/ref/fate/webp-rgb-lossless | 2 +-
> tests/ref/fate/webp-rgb-lossy-q80 | 2 +-
> tests/ref/fate/webp-rgba-lossless | 2 +-
> tests/ref/fate/webp-rgba-lossy-q80 | 2 +-
> 18 files changed, 1563 insertions(+), 115 deletions(-)
> create mode 100644 libavcodec/webp.h
> create mode 100644 libavformat/webpdec.c
>
> --
> 2.37.1 (Apple Git-137.1)
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2023-07-24 20:45 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-20 23:07 Thilo Borgmann
2023-07-20 23:07 ` [FFmpeg-devel] [PATCH v3 1/6] avcodec/webp: move definitions into header Thilo Borgmann
2023-07-20 23:07 ` [FFmpeg-devel] [PATCH v3 2/6] avcodec/webp: remove unused definitions Thilo Borgmann
2023-07-20 23:07 ` [FFmpeg-devel] [PATCH v3 3/6] avcodec/webp_parser: parse each frame into one packet Thilo Borgmann
2023-07-20 23:07 ` [FFmpeg-devel] [PATCH v3 4/6] libavcodec/webp: add support for animated WebP decoding Thilo Borgmann
2023-07-20 23:07 ` [FFmpeg-devel] [PATCH v3 5/6] avcodec/webp: make init_canvas_frame static Thilo Borgmann
2023-07-20 23:07 ` [FFmpeg-devel] [PATCH v3 6/6] libavformat/webp: add WebP demuxer Thilo Borgmann
2023-07-24 20:44 ` James Zern [this message]
2023-07-25 8:59 ` [FFmpeg-devel] [PATCH v3 0/6] webp: add support for animated WebP decoding Thilo Borgmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CABWgkX+eVOVzG1hXGp=szF99hwcdqgoeYq8UnZv-obNznoRUTg@mail.gmail.com' \
--to=jzern-at-google.com@ffmpeg.org \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git