Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Thilo Borgmann <thilo.borgmann@mail.de>
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH v2 0/5] webp: add support for animated WebP decoding
Date: Thu,  6 Jul 2023 13:27:26 +0200
Message-ID: <20230706112731.35895-1-thilo.borgmann@mail.de> (raw)

Now includes the demuxer again. Absence of the demuxer caused wrong
framerate detection. We are promised a review of the demuxer code as
well so it shall not rotten again with the remaining patches. Also adds
loop control suggested in v1.

This makes it feasable again to move some definitions into a shared
header which was nak'd in v1.

Patch 4/5 is still there for making changes in lavc/webp reviewable but
shall be stashed when pushing.

The remaining rendering issue is still worked on though we are not yet
sure how to fix it because of missing reference. The good news is that this
patchset already does a better rendering than libwebp v1.3.1 on the
problematic file. Haven't lost hope that this can yet be improved,
though.

Josef Zlomek (2):
  libavcodec/webp: add support for animated WebP decoding
  libavformat/webp: add WebP demuxer

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                                   |   2 +
 doc/demuxers.texi                           |  28 +
 libavcodec/codec_desc.c                     |   3 +-
 libavcodec/version.h                        |   2 +-
 libavcodec/webp.c                           | 724 +++++++++++++++++--
 libavcodec/webp.h                           |  55 ++
 libavcodec/webp_parser.c                    | 132 ++--
 libavformat/Makefile                        |   1 +
 libavformat/allformats.c                    |   1 +
 libavformat/version.h                       |   2 +-
 libavformat/webpdec.c                       | 733 ++++++++++++++++++++
 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 +-
 17 files changed, 1571 insertions(+), 124 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".

             reply	other threads:[~2023-07-06 11:27 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-06 11:27 Thilo Borgmann [this message]
2023-07-06 11:27 ` [FFmpeg-devel] [PATCH v2 1/5] avcodec/webp: move definitions into header Thilo Borgmann
2023-07-11 22:19   ` James Zern
2023-07-06 11:27 ` [FFmpeg-devel] [PATCH v2 2/5] avcodec/webp_parser: parse each frame into one packet Thilo Borgmann
2023-07-11 22:36   ` James Zern
2023-07-06 11:27 ` [FFmpeg-devel] [PATCH v2 3/5] libavcodec/webp: add support for animated WebP decoding Thilo Borgmann
2023-07-12  0:20   ` James Zern
2023-07-20 23:07     ` Thilo Borgmann
2023-07-06 11:27 ` [FFmpeg-devel] [PATCH v2 4/5] avcodec/webp: make init_canvas_frame static Thilo Borgmann
2023-07-12  0:20   ` James Zern
2023-07-06 11:27 ` [FFmpeg-devel] [PATCH v2 5/5] libavformat/webp: add WebP demuxer Thilo Borgmann
2023-07-06 20:24   ` James Zern
2023-07-06 21:08     ` Thilo Borgmann
2023-07-12  1:22   ` James Zern
2023-07-20 23:07     ` 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=20230706112731.35895-1-thilo.borgmann@mail.de \
    --to=thilo.borgmann@mail.de \
    --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