eo coding", offsetof(AACEncContext,
options.mid_side), AV_OPT_TYPE_BOOL, {.i64 = -1}, -1, 1, AACENC_FLAGS},
{"aac_is", "Intensity stereo coding", offsetof(AACEncContext,
options.intensity_stereo), AV_OPT_TYPE_BOOL, {.i64 = 1}, -1, 1, AACENC_FLAGS},
On 7/19/24 12:04 PM, Leo Izen wrote:
The PNG specification[1] says that sBIT entries must be at most the bit
depth specified in IHDR, unless the PNG is indexed-color, in which case
sBIT must be between 1 and 8. We should not reject valid sBITs on PNGs
with indexed color.
[1]: https://www.w3.org
://www.w3.org/TR/png-3/#11sBIT
Regression since: c125860892e931d9b10f88ace73c91484815c3a8.
Signed-off-by: Leo Izen
Reported-by: Ramiro Polla:
---
libavcodec/pngenc.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c
index 7ae16fa2c5
84b454935fae2633a8a5dd075e22393f3e8f932f.
Signed-off-by: Leo Izen
Reported-by: Ramiro Polla
---
libavcodec/pngdec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index cb861e5f60..c5b32c166d 100644
--- a/libavcodec/pngdec.c
+++ b
On 7/18/24 4:47 AM, Marton Balint wrote:
On Fri, 12 Jul 2024, Leo Izen wrote:
Indexed color images use three colors for sBIT, but the function
ff_png_get_nb_channels returns 1 in this case. We should avoid erroring
out on valid files in this scenario.
Please mention
On 7/12/24 3:03 PM, Leo Izen wrote:
If the sBIT chunk size is invalid, we should print a more informative
error message rather than return an error and print nothing.
Signed-off-by: Leo Izen
---
libavcodec/pngdec.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
Will merge soon
This test confirms that we can write mDCv and cLLi chunks and read them
back via the png decoder. It uses an HEVC conformance sample with this
metadata as the base source for the side data in the frames.
Signed-off-by: Leo Izen
Reported-by: Jan Ekström
Reviewed-by: Jan Ekström
Reviewed
av_frame_clone(buf);
if (!buf_out)
return AVERROR(ENOMEM);
This LGTM, I submitted a similar patch.
- Leo Izen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscrib
Declare AVFrame *buf_out with the rest of the declarations to fix a
compiler warning about Declarations after Statements in ISO C90.
Signed-off-by: Leo Izen
---
libavfilter/af_channelsplit.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavfilter/af_channelsplit.c b
On 7/9/24 12:45 PM, Leo Izen wrote:
This test confirms that we can write mDCv chunks and read them back via
the png decoder. It uses an HEVC conformance sample with this metadata
as the base source for the side data in the frames.
Signed-off-by: Leo Izen
Reported-by: Jan Ekström
Reviewed
If the sBIT chunk size is invalid, we should print a more informative
error message rather than return an error and print nothing.
Signed-off-by: Leo Izen
---
libavcodec/pngdec.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
Indexed color images use three colors for sBIT, but the function
ff_png_get_nb_channels returns 1 in this case. We should avoid erroring
out on valid files in this scenario.
Signed-off-by: Leo Izen
Reported-by: Ramiro Polla
---
libavcodec/pngdec.c | 2 +-
1 file changed, 1 insertion(+), 1
= num;
+sc->v_spacing = den;
}
If den == 0, nothing is assigned at all, but if den != 0 and if num ==
0, we assign 0 to h_spacing, which will end up doing nothing because we
check h_spacing && v_spacing later on in the other line you add.
dst, src, asrc, i, abase) \
Y= src[ 2 * i]; \
Are these able to be negative? If not, it may be wise to cast to
uint32_t instead as left shifting a negative integer is UB.
- Leo Izen (Traneptora)
___
ffmpeg-d
This test confirms that we can write mDCv chunks and read them back via
the png decoder. It uses an HEVC conformance sample with this metadata
as the base source for the side data in the frames.
Signed-off-by: Leo Izen
Reported-by: Jan Ekström
Reviewed-by: Andreas Rheinhardt
---
tests/fate
Changes since v1:
- changed transcode command
Leo Izen (1):
fate/png: add mDCv read and write test
tests/fate/image.mak| 6 ++
tests/ref/fate/png-mdcv | 22 ++
2 files changed, 28 insertions(+)
create mode 100644 tests/ref/fate/png-mdcv
--
2.45.2
This test confirms that we can write mDCv chunks and read them back via
the png decoder. It uses an HEVC conformance sample with this metadata
as the base source for the side data in the frames.
Signed-off-by: Leo Izen
Reported-by: Jan Ekström
---
tests/fate/image.mak| 5 +
tests/ref
On 7/2/24 9:44 AM, Leo Izen wrote:
When mDCv support was added, there was a typo in both variable names
and also the MKTAG itself, incorrectly listing it as mDVc. The tag name
stands for Mastering Display Color Volume so mDCv is correct.
Typo originally introduced
When mDCv support was added, there was a typo in both variable names
and also the MKTAG itself, incorrectly listing it as mDVc. The tag name
stands for Mastering Display Color Volume so mDCv is correct.
Typo originally introduced in 78949041417caaef0c82b2b23d7defdd88aa2378.
Signed-off-by: Leo
in c7a57b0f70f8d1574aa0f0dbe98db85d8ac91c76.
Signed-off-by: Leo Izen
Reported-by: Ramiro Polla
---
libavcodec/pngdec.c | 53 +++--
1 file changed, 27 insertions(+), 26 deletions(-)
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index 1c910e6a5b..180806e5e1 100644
On 6/27/24 1:27 PM, Lynne via ffmpeg-devel wrote:
On 21/06/2024 23:20, Leo Izen wrote:
int idx, and int first_tcx_flag are set but not used, so this commit
removes their declarations and assignments.
Sorry, missed this patch, LGTM.
Applied as 539d2e989d7ce3556a549d59ee9c5125ae7d7b6b
- Leo
On 6/21/24 5:20 PM, Leo Izen wrote:
int idx, and int first_tcx_flag are set but not used, so this commit
removes their declarations and assignments.
---
libavcodec/aac/aacdec_lpd.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/libavcodec/aac/aacdec_lpd.c b
ink that one is unnecessary, but this one is fine.
I have no comment on the generic patches.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
int idx, and int first_tcx_flag are set but not used, so this commit
removes their declarations and assignments.
---
libavcodec/aac/aacdec_lpd.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/libavcodec/aac/aacdec_lpd.c b/libavcodec/aac/aacdec_lpd.c
index
int idx, and int first_tcx_flag are set but not used, so this commit
removes their declarations and assignments.
Signed-off-by: Leo Izen
---
libavcodec/aac/aacdec_lpd.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/libavcodec/aac/aacdec_lpd.c b/libavcodec/aac
Use SHLIBOBJS and STLIBOBJS in the Makefiles for avcodec and avformat,
and add a stub ffjni.c to libavformat, which allows the symbols to be
duplicated for shared builds but not static builds.
Signed-off-by: Leo Izen
---
libavcodec/Makefile | 1 +
libavformat/Makefile | 1 +
libavformat
Changes since v1:
- Rebased onto master
I was unable to credit the original reporter with a Reported-by because
the name used Chinese characters (and it translated to 'admin').
Leo Izen (1):
avcodec,avformat/ffjni: fix duplicate JNI symbols
libavcodec/Makefile | 1 +
libavformat/Makefile
in commit message, you want libtorch, not libtoch.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org
to
alphabetize it.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
The continue statement will break out of the do/while loop, not the
outer loop as intended. This is one (compound) statement anyway, so we
can remove the do/while entirely.
Signed-off-by: Leo Izen
---
libavutil/film_grain_params.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff
These variables are never read from, so they trigger -Wunused-variables
Signed-off-by: Leo Izen
---
libavutil/film_grain_params.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/libavutil/film_grain_params.c b/libavutil/film_grain_params.c
index fff7252f2f..b3fa37f527 100644
A few minor tweaks to libavutil/film_grain_params.c.
Leo Izen (2):
avutil/film_grain_params: remove unused variables
avutil/film_grain_params: remove do loop in CHECK macro
libavutil/film_grain_params.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
--
2.44.0
avpkt->pos needs to be set for generic indexing or features such as the
stream_loop option will not work.
Co-authored-by: Andreas Rheinhardt
Signed-off-by: Leo Izen
---
libavformat/jpegxl_anim_dec.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavformat/jpegxl_anim_dec.
On 3/11/24 17:48, Marth64 wrote:
This breaks fate
Looking into it - thanks.
It is possible that the change is correct, and that the fate test needs
to be changed to reflect the correct behavior. Not necessarily, but
possibly.
- Leo Izen (Traneptora)
OpenPGP_signature.asc
Description
summaries/counts on the left, could you please elaborate on
how you generated this data? I'm not super familiar with the trac user
interface, but iirc you can embed the search in a query string.
- Leo Izen (Traneptora)
OpenPGP_signature.asc
Description: OpenPGP digital
On 3/9/24 13:27, Marth64 wrote:
Signed-off-by: Marth64
---
doc/demuxers.texi | 43 +-
libavformat/dvdvideodec.c | 314 --
2 files changed, 339 insertions(+), 18 deletions(-)
diff --git a/doc/demuxers.texi b/doc/demuxers.texi
index
become widely used if it was in
someones personal repository and main FFmpeg did not support it
thx
This is true, but also the code hasn't been touched in more than ten
years, which makes it a bit different than, say, ffv1 which was actively
developed for a while and still is.
- Leo Izen
On 3/9/24 15:49, Poorva wrote:
I have attached the git patch containing the changes for your review.
This patch is submitted as part of my qualification task for Google Summer
of Code (GSoC)
Your editor appears to have stripped the newline at the end of the file.
- Leo Izen (Traneptora
On 2/14/24 13:53, Andreas Rheinhardt wrote:
Leo Izen:
Write EXIF metadata exposed AV_FRAME_DATA_EXIF as an eXIf chunk
to PNG files, if present.
Signed-off-by: Leo Izen
---
libavcodec/pngenc.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c
Write EXIF metadata exposed AV_FRAME_DATA_EXIF as an eXIf chunk
to PNG files, if present.
Signed-off-by: Leo Izen
---
libavcodec/pngenc.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c
index 50689cb50c..a302c879da 100644
--- a/libavcodec
Add support to parse eXIf chunks using the new EXIF framework.
Signed-off-by: Leo Izen
---
libavcodec/pngdec.c | 35 +++
1 file changed, 35 insertions(+)
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index 026da30c25..e7951d1802 100644
--- a/libavcodec
be
applied during a major version bump.
Signed-off-by: Leo Izen
---
fftools/ffprobe.c | 27 ++-
libavcodec/Makefile| 1 +
libavcodec/exif.c | 267 +++--
libavcodec/exif.h | 21 ++-
libavcodec/exif_internal.h
as that would be.
[1] https://exiv2.org/makernote.html
[2] http://www.exif.org/samples.html
Leo Izen (3):
various: change EXIF metadata into AVFrameSideData
avcodec/pngdec: parse eXIf chunk
avcodec/pngenc: write eXIf chunks
fftools/ffprobe.c | 27 ++-
libavcodec/Makefile
Add an AVOption to the libjxl encoder wrapper, which exposes the flag
uses_original_profile in libjxl. For highly unusual ICC profiles where
the target needs to stay in the original space, this can be useful.
Signed-off-by: Leo Izen
---
libavcodec/libjxlenc.c | 5 -
libavcodec/version.h
Changes since v1:
- bump micro version, per policy
- rebase onto master
Leo Izen (1):
avcodec/libjxlenc: add option to disable xyb encoding
libavcodec/libjxlenc.c | 5 -
libavcodec/version.h | 2 +-
2 files changed, 5 insertions(+), 2 deletions(-)
--
2.43.0
On 1/31/24 14:37, Leo Izen wrote:
The function signature for bytestream2_seek is (gb, offset, whence);
Before this patch, the code passed (gb, SEEK_SET, offset), which is
incorrect.
Siged-off-by: Leo Izen
---
libavcodec/tiff.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On 2/4/24 04:35, Leo Izen wrote:
This adds support for cLLi and mDVc chunks in the PNG specification[1].
[1]: https://www.w3.org/TR/png-3/
Changes from v1:
- fix regression in cHRM writing, causing fate failure
Leo Izen (2):
avcodec/pngdec: read cLLi and mDVc chunks
avcodec/pngenc
On 2/7/24 13:56, Andreas Rheinhardt wrote:
Leo Izen:
On 1/23/24 14:22, Michael Niedermayer wrote:
Hi all
As it was a little difficult for me to not loose track of what is
blocking a release. I suggest that for all release blocking issues
open a ticket and set Blocking to 7.0
that way
* data loss
* privacy issues
* anything the commuity agrees should be in the release
thx
My EXIF overhaul is going to be an ABI break so I'd like to get it in,
if and only if we are doing an ABI break with the release.
- Leo Izen (Traneptora)
OpenPGP_signature.asc
Description: OpenPGP digital
These chunks contain the Content Light Level Information and the
Mastering Display Color Volume information that FFmpeg already supports
as AVFrameSideData. This patch adds support for the png encoder to save
this metadata as the corresponding chunks in the PNG stream.
Signed-off-by: Leo Izen
.
Signed-off-by: Leo Izen
---
libavcodec/pngdec.c | 63 +
1 file changed, 63 insertions(+)
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index d1aae4c70e..026da30c25 100644
--- a/libavcodec/pngdec.c
+++ b/libavcodec/pngdec.c
@@ -29,6 +29,7
This adds support for cLLi and mDVc chunks in the PNG specification[1].
[1]: https://www.w3.org/TR/png-3/
Changes from v1:
- fix regression in cHRM writing, causing fate failure
Leo Izen (2):
avcodec/pngdec: read cLLi and mDVc chunks
avcodec/pngenc: write cLLi and mDVc chunks
libavcodec
On 2/3/24 14:20, Andreas Rheinhardt wrote:
Forgotten after 82beb46e65e5f820b187355bf757725c22a59c45.
Also use loop-scope for iterators while at it.
Signed-off-by: Andreas Rheinhardt
---
libavformat/nutenc.c | 30 +++---
1 file changed, 15 insertions(+), 15
On 2/1/24 03:29, Anton Khirnov wrote:
Replace it with recreating the codec context.
This is the last remaining blocker for deprecating avcodec_close().
---
libavformat/demux.c | 53 -
1 file changed, 48 insertions(+), 5 deletions(-)
diff --git
These chunks contain the Content Light Level Information and the
Mastering Display Color Volume information that FFmpeg already supports
as AVFrameSideData. This patch adds support for the png encoder to save
this metadata as the corresponding chunks in the PNG stream.
Signed-off-by: Leo Izen
.
Signed-off-by: Leo Izen
---
libavcodec/pngdec.c | 63 +
1 file changed, 63 insertions(+)
diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index d1aae4c70e..026da30c25 100644
--- a/libavcodec/pngdec.c
+++ b/libavcodec/pngdec.c
@@ -29,6 +29,7
This adds support for cLLi and mDVc chunks in the PNG specification[1].
[1]: https://www.w3.org/TR/png-3/
Leo Izen (2):
avcodec/pngdec: read cLLi and mDVc chunks
avcodec/pngenc: write cLLi and mDVc chunks
libavcodec/pngdec.c | 63 +
libavcodec
The function signature for bytestream2_seek is (gb, offset, whence);
Before this patch, the code passed (gb, SEEK_SET, offset), which is
incorrect.
Siged-off-by: Leo Izen
---
libavcodec/tiff.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/tiff.c b/libavcodec
On 1/30/24 10:24, Leo Izen wrote:
+jxl_fmt.align = -frame->linesize[0];
+data = frame->data[0] + frame->linesize[0] * (info.ysize - 1);
+}
Something like this ^ is what you're requesting, right?
- Leo Izen (Traneptora)
OpenPGP_signature.asc
Description
-flip orientation.
Signed-off-by: Leo Izen
---
libavcodec/libjxlenc.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/libavcodec/libjxlenc.c b/libavcodec/libjxlenc.c
index 67be8a01ca..5f5f9751dc 100644
--- a/libavcodec/libjxlenc.c
+++ b/libavcodec/libjxlenc.c
On 1/30/24 09:57, Andreas Rheinhardt wrote:
Please don't rely on the type of jxl_fmt.align here (which is out of our
control). E.g. in the future it may be that libjxl supports only 32bit
align values (i.e. uses uint32_t or so for it), but that we support
64bit (ptrdiff_t) linesizes and
On 1/30/24 09:46, Leo Izen wrote:
libjxl doesn't support negative strides, but JPEG XL has an orientation
flag inside the codestream. We can use this to work around the library
limitation, by taking the absolute value of the negative row stride,
sending the image up-side-down, and telling
-flip orientation.
Signed-off-by: Leo Izen
---
libavcodec/libjxlenc.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/libavcodec/libjxlenc.c b/libavcodec/libjxlenc.c
index 67be8a01ca..4ddd5f9f2c 100644
--- a/libavcodec/libjxlenc.c
+++ b/libavcodec/libjxlenc.c
-flip orientation.
Signed-off-by: Leo Izen
---
libavcodec/libjxlenc.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/libavcodec/libjxlenc.c b/libavcodec/libjxlenc.c
index 67be8a01ca..49020b1036 100644
--- a/libavcodec/libjxlenc.c
+++ b/libavcodec/libjxlenc.c
Add an AVOption to the libjxl encoder wrapper, which exposes the flag
uses_original_profile in libjxl. For highly unusual ICC profiles where
the target needs to stay in the original space, this can be useful.
Signed-off-by: Leo Izen
---
libavcodec/libjxlenc.c | 5 -
1 file changed, 4
that you obtained the file from. You can also host the
sample elsewhere, such as your own website, or on a site like 0x0.st, if
you are concerned about Videolan's jurisdiction.
- Leo Izen (Traneptora)
OpenPGP_signature.asc
Description: OpenPGP digital signature
are also
present in that file, so we no longer need to redefine them.
Signed-off-by: Leo Izen
---
libavcodec/libjxl.h | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/libavcodec/libjxl.h b/libavcodec/libjxl.h
index e305b6e758..0b983a122f 100644
--- a/libavcodec
Produces a -Wunused-variables warning with -Wall.
Signed-off-by: Leo Izen
---
libavformat/mxfenc.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
index 685c11b3a5..febc1ccf8c 100644
--- a/libavformat/mxfenc.c
+++ b/libavformat/mxfenc.c
Patch attached, this email signed with my previous key.
- Leo Izen (Traneptora)
From 09ee68f816d1d2c39097515cedca69031251aa4f Mon Sep 17 00:00:00 2001
From: Leo Izen
Date: Mon, 8 Jan 2024 14:01:59 -0500
Subject: [PATCH] MAINTAINERS: add my new PGP key
Adding my new gpg key that I will be using
On 12/25/23 15:09, Michael Niedermayer wrote:
On Mon, Dec 25, 2023 at 12:04:17PM -0500, Leo Izen wrote:
The specification doesn't mention that clusters cannot have alphabet
sizes greater than 1 << bundle->log_alphabet_size, but the reference
implementation rejects these entrop
module
has to be maintained, and removing that module removes the maintenance
burden. An international obfuscated C contest entry isn't really on-topic.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org
be large enough otherwise.
Fixes #10738.
Found-by: Zeng Yunxiang and Li Zeyuan
Signed-off-by: Leo Izen
---
libavcodec/jpegxl_parser.c | 28 +++-
1 file changed, 19 insertions(+), 9 deletions(-)
diff --git a/libavcodec/jpegxl_parser.c b/libavcodec/jpegxl_parser.c
index 0
/libjxlenc.c
+++ b/libavcodec/libjxlenc.c
@@ -85,8 +85,8 @@ static float quality_to_distance(float quality)
}
Both of these LGTM.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
be large enough otherwise.
Fixes #10738.
Reported-by: Michael Niedermayer
Signed-off-by: Leo Izen
---
libavcodec/jpegxl_parser.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavcodec/jpegxl_parser.c b/libavcodec/jpegxl_parser.c
index 006eb6b295..c9832e43
On 12/15/23 16:31, Zsolt Vadász via ffmpeg-devel wrote:
On Friday, December 15th, 2023 at 10:12 PM, Leo Izen wrote:
On 12/15/23 11:40, Zsolt Vadász via ffmpeg-devel wrote:
On Friday, December 15th, 2023 at 12:20 AM, Leo Izen leo.i...@gmail.com wrote:
+ AVFrame *last;
I don't see
On 12/15/23 11:40, Zsolt Vadász via ffmpeg-devel wrote:
On Friday, December 15th, 2023 at 12:20 AM, Leo Izen wrote:
+ AVFrame *last;
I don't see the purpose of keeping this here.
The name is misleading, I should have named it `previous`, since it always
contains the previous frame.
I
the PTS information those containers provide to work as
expected.
Signed-off-by: Leo Izen
---
libavcodec/libjxldec.c | 57 +-
1 file changed, 29 insertions(+), 28 deletions(-)
diff --git a/libavcodec/libjxldec.c b/libavcodec/libjxldec.c
index 002740d9c1
On 12/14/23 03:28, Anton Khirnov wrote:
Quoting Leo Izen (2023-12-08 18:31:06)
If a sequence of JXL images is encapsulated in a container that has PTS
information, we should use the PTS information from the container. At
this time there is no container that does this, but if JPEG XL support
On 12/13/23 15:53, Zsolt Vadász via ffmpeg-devel wrote:
---
configure | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/libjxlenc.c | 214 +
3 files changed, 177 insertions(+), 39 deletions(-)
diff --git a/configure b/configure
index
--- a/libavcodec/libjxlenc.c
+++ b/libavcodec/libjxlenc.c
If this is necessary for adding an animated encoder it needs to be in
the commit, because this commit by itself doesn't do anything.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel
On 12/11/23 12:05, Zsolt Vadász via ffmpeg-devel wrote:
---
libavcodec/libjxlenc.c | 45 +-
1 file changed, 23 insertions(+), 22 deletions(-)
diff --git a/libavcodec/libjxlenc.c b/libavcodec/libjxlenc.c
index d707f3a61b..92a458d51a 100644
Why?
as it should.
Also, PNG doesn't support nonzero matrices but we only warn and ignore
in that case, so we have no reason to error out for illegal cICP ranges
either (i.e. greater than 1).
Signed-off-by: Leo Izen
Reported-by: Kacper Michajłow
---
libavcodec/pngdec.c | 23 ---
1 file
These pixel formats have always been supported by libjxl, but at the
time this plugin was written, they were not in FFmpeg yet. Now that
they are in FFmpeg, we should support them.
Signed-off-by: Leo Izen
---
libavcodec/libjxlenc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavcodec
These pixel formats have always been supported by libjxl, but at the
time this plugin was written, they were not in FFmpeg yet. Now that
they are in FFmpeg, we should support them.
Signed-off-by: Leo Izen
---
libavcodec/libjxldec.c | 17 +
1 file changed, 9 insertions(+), 8
On 12/10/23 23:20, Kacper Michajlow wrote:
On Tue, 28 Feb 2023 at 20:46, Leo Izen wrote:
On 2/27/23 11:34, Leo Izen wrote:
On 2/21/23 17:35, Leo Izen wrote:
These chunks are lightweight and it's useful information to have when
running ffmpeg -i or ffprobe, for example.
---
libavcodec
On 12/10/23 09:07, Andreas Rheinhardt wrote:
Leo Izen:
This patch requires a sample that hasn't been uploaded yet. It can be found
at [1] and it should be placed at jxl/orange.jxl once uploaded. It's 262k,
which is not very large, but large enough that I did not want to attach it
to this email
it can get a little bit difficult for reviewers to track the
versions. I have no comments on the patch itself, in either direction.
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg
On 12/9/23 05:06, Marth64 wrote:
Hello, I am happy to share a DVD demuxer for ffmpeg powered by libdvdread and
libdvdnav.
I have been working on this on/off throughout the year and think it is in a
good spot
to share at the ML now. This was a major learning experience for me in many
ways and
This test verifies the parser's handling of multiframe JXL files that
have an entropy-encoded permuted table of contents for each frame. The
testcase is actually six JXL codestreams concatenated together, and the
parser needs to be able to find the boundaries.
Signed-off-by: Leo Izen
---
tests
:
9288337dc0d37effd1f539d4e20083a8ffed757e486f4098121520c74e8de6f6
sample signature: https://buzo.us/A.asc
Leo Izen (1):
fate/jpegxl: add multiframe permuted TOC image parser test
tests/fate/jxl.mak | 3 +++
tests/ref/fate/jxl-multiframe-permuted-toc | 11 +++
2 files
the PTS information those containers provide to work as
expected.
Signed-off-by: Leo Izen
---
libavcodec/libjxldec.c | 77 +++---
1 file changed, 57 insertions(+), 20 deletions(-)
diff --git a/libavcodec/libjxldec.c b/libavcodec/libjxldec.c
index 002740d9c1
On 11/23/23 10:31, Leo Izen wrote:
Avoids a -Wstringop-truncation warning by using av_strlcopy instead of
strncpy. Additionally, prints a warning to the log context if this
truncation occurred.
Signed-off-by: Leo Izen
---
Bump, thanks.
- Leo Izen (Traneptora
On 12/7/23 11:32, Nicolas George wrote:
Leo Izen (12023-12-07):
I believe the question was directed at Paul
That does not make my answer any less relevant.
I'm not going to have an argument with you about why a question to Paul
about a decision Paul made should be answered by Paul
not answered yet
(as of writing this).
- Leo Izen (Traneptora)
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject
On 11/27/23 09:10, Leo Izen wrote:
Two bug fixes related to the JPEG XL parser. They're not exactly related and
don't need to be applied in sequence.
v2 change: patches themselves are identical, but now there's a FATE test.
Leo Izen (3):
avcodec/jpegxl_parse{,r}: use correct ISOBMFF
Add a fate test for the above commits fixing extremely small files or
files with extended box sizes.
Signed-off-by: Leo Izen
---
tests/fate/jxl.mak | 8
tests/ref/fate/jxl-small-ext-box | 9 +
2 files changed, 17 insertions(+)
create mode 100644 tests/ref/fate
This patch allows the JXL parser to parse sequences of extremely small
files concatenated together. (e.g. smaller than the parser buffer)
Signed-off-by: Leo Izen
---
libavcodec/jpegxl_parser.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/libavcodec
According to ISO/IEC 14996-12, size == 1 means a 64-bit extended-size
field occurs *after* the 32-bit box type, not before. This fix should
allow correct parsing of JXL files with extended-size boxes.
Signed-off-by: Leo Izen
---
libavcodec/jpegxl_parse.c | 6 +++---
libavcodec/jpegxl_parser.c
Two bug fixes related to the JPEG XL parser. They're not exactly related and
don't need to be applied in sequence.
v2 change: patches themselves are identical, but now there's a FATE test.
Leo Izen (3):
avcodec/jpegxl_parse{,r}: use correct ISOBMFF extended size location
avcodec
On 11/26/23 13:07, Thilo Borgmann wrote:
Am 26.11.2023 um 14:47 schrieb Leo Izen :
On 11/26/23 05:56, Anton Khirnov wrote:
Would be nice to have tests for these.
I have a sample at: https://buzo.us/l.jxl
It would test both of these patches. I can send a fate test for these to the ML
1 - 100 of 554 matches
Mail list logo
