Re: [FFmpeg-devel] [PATCH] Add support for the new key & value API in libaom.

2021-02-08 Thread Christopher Degawa
Quick question as a user that is not entirely related to this patch, would it be possible to add something like a `ffmpeg ... -libaom-params help -f null -` or someother way that would print out the options available in the compiled libaom? (In the case where we do not have aomenc compiled or it's

[FFmpeg-devel] [PATCH] libsvtav1: Add lp option

2021-02-09 Thread Christopher Degawa
From: Christopher Degawa Equivalent to the --lp option for SvtAv1EncApp, and is the only way to control how much cpu power svt-av1 uses for now Not using thread_count as it would be preferable to reserve that until svt-av1 properly implements a threads option 0 == getconf _NPROCESSORS_ONLN

[FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-10 Thread Christopher Degawa
From: Christopher Degawa Equivalent to the --lp option for SvtAv1EncApp, and is the only way to control how much cpu power svt-av1 uses for now Not using thread_count as it would be preferable to reserve that until svt-av1 properly implements a threads option 0 == getconf _NPROCESSORS_ONLN

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add lp option

2021-02-10 Thread Christopher Degawa
> > It is very non-obvious what "lp" means, "logical_processors" would be > better IMO. > Okay, I will change it to `-logical_processors` ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscri

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-10 Thread Christopher Degawa
Based on my limited understanding of the code, it's limiting the usage using pthread_setaffinity_np and CPU_SET on Linux to limit the process to certain CPUs, but it also has a default and max of the return of `sysconf(_SC_NPROCESSORS_ONLN)`. According to Hassene Tmar of SVT-AV1, it is a "target co

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-14 Thread Christopher Degawa
> > That said, if you have a general use-case where this is helpful and the > documentation explains what it is doing (and warns about the bad cases) > then maybe? > The main use case we have internally is to able to run multiple instances of SvtAv1EncApp with `--lp 1` without worrying about poten

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-17 Thread Christopher Degawa
On Sun, Feb 14, 2021 at 12:27 PM Christopher Degawa wrote: > That said, if you have a general use-case where this is helpful and the >> documentation explains what it is doing (and warns about the bad cases) >> then maybe? >> > > The main use case we have internally

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-17 Thread Christopher Degawa
> > It does seem like the number correlates somehow with how much CPU it > uses, but it's not an upper bound. > If it isn't, then that might be an issue, but the idea is that with the logical_processor option you can limit the amount of ram a single encoder instance will use since on a system wit

[FFmpeg-devel] [PATCH v2] libsvtav1: Add logical_processors option

2021-02-19 Thread Christopher Degawa
From: Christopher Degawa Used for limiting the size of memory buffers and threads for a target logical processor count, but does not set thread affinity or limit the amount of threads used, although thread affinities can be controlled with an additional parameters, it is prefered to add them

[FFmpeg-devel] [PATCH v3] libsvtav1: Add logical_processors option

2021-02-26 Thread Christopher Degawa
From: Christopher Degawa Used for limiting the size of memory buffers and threads for a target logical processor count, but does not set thread affinity or limit the amount of threads used, although thread affinities can be controlled with an additional parameters, it is prefered to add them

Re: [FFmpeg-devel] [PATCH v3] libsvtav1: Add logical_processors option

2021-02-26 Thread Christopher Degawa
> Maybe I am reading this wrong or not understanding but this says it is > used for limiting threads and then in the next sentence says it does > not limit threads? > > It is quite confusing. > it's confusing for me too, but the idea is that it doesn't set n threads, but it can be used to make it s

[FFmpeg-devel] [PATCH v4] libsvtav1: Add logical_processors option

2021-03-09 Thread Christopher Degawa
From: Christopher Degawa Used for limiting the size of memory buffers and threads for a target logical processor count, but does not set thread affinity or the total amount of threads used, although thread affinities can be controlled with an additional parameters, it is prefered to add them

Re: [FFmpeg-devel] [PATCH v4] libsvtav1: Add logical_processors option

2021-03-10 Thread Christopher Degawa
> You should restrict the line length here, just for consistency with the > rest of this document. (It doesn't affect the formatting of the output > documents, obviously.) > What would be a good line length for the docs to aim for? 75? > > +unsigned logical_processors; > > The libsvtav1 API d

[FFmpeg-devel] [PATCH] libsvtav1: Add crf and enable_tpl_la options

2021-03-10 Thread Christopher Degawa
libsvtav1 internally changed cqp to be crf while enable-tp-la is on, which is on by default, and uses the same variables as cqp Signed-off-by: Christopher Degawa --- doc/encoders.texi | 14 +- libavcodec/libsvtav1.c | 15 +-- 2 files changed, 26 insertions(+), 3

[FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-03-10 Thread Christopher Degawa
From: Christopher Degawa Used for limiting the size of memory buffers and threads for a target logical processor count, but does not set thread affinity or the total amount of threads used, although thread affinities can be controlled with an additional parameters, it is prefered to add them

[FFmpeg-devel] [PATCH] libavutil/timer: Fix clang reserved-user-defined-literal

2021-03-12 Thread Christopher Degawa
clang errors when compiling with C++11 about needing spaces between literal and identifier Signed-off-by: Christopher Degawa --- libavutil/timer.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/timer.h b/libavutil/timer.h index 0bb353cfce..737bfc160e 100644 --- a

Re: [FFmpeg-devel] [PATCH] libavutil/timer: Fix clang reserved-user-defined-literal

2021-03-13 Thread Christopher Degawa
On Sat, Mar 13, 2021 at 01:09 Chris Degawa wrote: > > > > On Mar 13, 2021, at 00:57, Anton Khirnov wrote: > > > > Quoting Christopher Degawa (2021-03-13 05:20:37) > >> clang errors when compiling with C++11 about needing spaces between > >> literal an

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add crf and enable_tpl_la options

2021-03-16 Thread Christopher Degawa
ping on this patch ___ 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".

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add crf and enable_tpl_la options

2021-03-23 Thread Christopher Degawa
ping again, would rather not have to maintain this and the logical_processor patch out of tree ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-

Re: [FFmpeg-devel] [PATCH v2] libsvtav1: Add logical_processors option

2021-03-30 Thread Christopher Degawa
On Tue, Mar 30, 2021 at 4:53 PM Jan Ekström wrote: > > @@ -218,6 +220,8 @@ static int > config_enc_params(EbSvtAv1EncConfiguration *param, > > param->tile_columns = svt_enc->tile_columns; > > param->tile_rows= svt_enc->tile_rows; > > > > +param->logical_processors = svt_enc->log

[FFmpeg-devel] [PATCH v2] libsvtav1: Add logical_processors option

2021-03-30 Thread Christopher Degawa
From: Christopher Degawa Used as a thread count multiplier with a max of the logical cores available Signed-off-by: Christopher Degawa --- doc/encoders.texi | 3 +++ libavcodec/libsvtav1.c | 7 +++ 2 files changed, 10 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi

Re: [FFmpeg-devel] [GSoC 2021] about proposal

2021-04-08 Thread Christopher Degawa
You might want to check your email client as it has added a bunch of ` ` which mangles your message when viewed ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or em

Re: [FFmpeg-devel] [PATCH v2] libsvtav1: Add logical_processors option

2021-04-10 Thread Christopher Degawa
ping on this patch again ___ 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".

Re: [FFmpeg-devel] [PATCH v2] libsvtav1: Add logical_processors option

2021-04-14 Thread Christopher Degawa
On Sat, Apr 10, 2021 at 3:07 PM Christopher Degawa wrote: > ping on this patch again > pong again ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email

Re: [FFmpeg-devel] Unable to submit patches

2020-02-07 Thread Christopher Degawa
If you are talking about "[FFmpeg-devel] [PATCH] doc/ffmpeg: Fix bug #8204", it seems the list has received it On Sat, Feb 8, 2020 at 12:33 AM Gautam Ramakrishnan wrote: > Hello, > > I wish to contribute to ffmpeg and though of submitting a small patch > to fix a bug listed on the bug tracker. H

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add crf and enable_tpl_la options

2021-04-21 Thread Christopher Degawa
ping. ___ 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".

[FFmpeg-devel] [PATCH] libsvtav1: pass color description info

2021-07-22 Thread Christopher Degawa
these fields are only available past svt-av1 0.8.7 Signed-off-by: Christopher Degawa --- libavcodec/libsvtav1.c | 20 1 file changed, 20 insertions(+) diff --git a/libavcodec/libsvtav1.c b/libavcodec/libsvtav1.c index fabc4e6428..6c12777911 100644 --- a/libavcodec

Re: [FFmpeg-devel] [PATCH] libsvtav1: pass color description info

2021-07-29 Thread Christopher Degawa
On Thu, Jul 22, 2021 at 9:02 PM Christopher Degawa wrote: > these fields are only available past svt-av1 0.8.7 > > Signed-off-by: Christopher Degawa > --- > libavcodec/libsvtav1.c | 20 > 1 file changed, 20 insertions(+) > > diff --git a/libavcodec/

Re: [FFmpeg-devel] [PATCH] libsvtav1: pass color description info

2021-07-30 Thread Christopher Degawa
On Fri, Jul 30, 2021 at 4:48 AM Jan Ekström wrote: > On Fri, Jul 23, 2021 at 5:02 AM Christopher Degawa > wrote: > > +#ifndef SVTAV1_MAKE_VERSION > > +#define SVTAV1_MAKE_VERSION(x,y,z) ((x) << 16 | (y) << 8 | z) > > +#endif > > + >

Re: [FFmpeg-devel] duplicate symbol '_dec_init' in: fftools/ffmpeg_dec.o

2024-03-16 Thread Christopher Degawa
On Sat, Mar 16, 2024 at 09:08 Gnattu OC via ffmpeg-devel < ffmpeg-devel@ffmpeg.org> wrote: > If you are using Xcode >= 15 then you will need to add `-Wl,-ld_classic` > to LDFLAGS. During configure you will also need to set > `--host-ldflags='-Wl,-ld_classic’`. > > > On Mar 16, 2024, at 09:04, Hel

[FFmpeg-devel] [PATCH] libavformat/dashdec: Fix issue with dash on Windows

2020-10-08 Thread Christopher Degawa
ive-mpd-AV-NBS.mpd -f null - Signed-off-by: Christopher Degawa --- libavformat/dashdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c index 99b9c45439..42ea74635b 100644 --- a/libavformat/dashdec.c +++ b/libavformat/dashdec.c

Re: [FFmpeg-devel] [PATCH] libavformat/dashdec: Fix issue with dash on Windows

2020-10-08 Thread Christopher Degawa
> You should also reset c->adaptionset_lang to NULL after freeing it. (I > remember finding this issue myself, but somehow forgot to fix it. Strange.) Ah, I forgot that's what av_freep does in addition to freeing, do you wish to make the change, or do you want me to? > (Actually, the lifetime of

[FFmpeg-devel] [PATCH 1/2] avcodec/libsvtav1: remove compressed_ten_bit_format and simplify alloc_buffer

2022-10-19 Thread Christopher Degawa
From: Christopher Degawa compressed_ten_bit_format has been deprecated upstream and has no effect and can be removed. Plus, technically it was never used in the first place since it would require the app (ffmpeg) to set it and do additional processing of the input frames. Also simplify

[FFmpeg-devel] [PATCH 2/2] avcodec/libsvtav1: replace vbv_bufsize with maximum_buffer_size_ms

2022-10-19 Thread Christopher Degawa
From: Christopher Degawa svt-av1 v1.2.0 has deprecated vbv_bufsize in favor of using - maximum_buffer_size_ms (--buf-sz) - starting_buffer_level_ms (--buf-initial-sz) - optimal_buffer_level_ms (--buf-optimal-sz) and vbv_bufsize has not been in use since svt-av1 v0.8.6 Signed-off-by

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libsvtav1: replace vbv_bufsize with maximum_buffer_size_ms

2022-10-19 Thread Christopher Degawa
On Wed, Oct 19, 2022 at 7:00 PM James Almer wrote: > On 10/19/2022 7:47 PM, Christopher Degawa wrote: > > From: Christopher Degawa > > diff --git a/libavcodec/libsvtav1.c b/libavcodec/libsvtav1.c > > index 28da206cf8..74dad9892b 100644 > > --- a/libavcodec/libsv

[FFmpeg-devel] [PATCH v2 1/2] avcodec/libsvtav1: remove compressed_ten_bit_format and simplify alloc_buffer

2022-10-20 Thread Christopher Degawa
From: Christopher Degawa compressed_ten_bit_format has been deprecated upstream and has no effect and can be removed. Plus, technically it was never used in the first place since it would require the app (ffmpeg) to set it and do additional processing of the input frames. Also simplify

[FFmpeg-devel] [PATCH v2 2/2] avcodec/libsvtav1: replace vbv_bufsize with maximum_buffer_size_ms

2022-10-20 Thread Christopher Degawa
From: Christopher Degawa svt-av1 v1.2.0 has deprecated vbv_bufsize in favor of using - maximum_buffer_size_ms (--buf-sz) - starting_buffer_level_ms (--buf-initial-sz) - optimal_buffer_level_ms (--buf-optimal-sz) and vbv_bufsize has not been in use since svt-av1 v0.8.6 Signed-off-by

[FFmpeg-devel] [PATCH] codec2: use pkg-config file if it is available

2024-07-20 Thread Christopher Degawa
codec2.pc has been available since 2016, and I believe most distros have it by now. codec2-dev's commit: https://github.com/drowe67/codec2-dev/commit/fdfda67436b5b7308cf0758d9192f1dd540d439d Signed-off-by: Christopher Degawa --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 del

Re: [FFmpeg-devel] [PATCH] codec2: use pkg-config file if it is available

2024-07-20 Thread Christopher Degawa
On Sun, Jul 21, 2024 at 12:03 AM Christopher Degawa wrote: > +enabled libcodec2 && { require_pkg_config libcodec2 codec2 > codec2.h codec2_create || > + require libcodec2 codec2/codec2.h > codec2_create -lcodec2; } > I made a mista

[FFmpeg-devel] [PATCH v2] codec2: use pkg-config file if it is available

2024-07-20 Thread Christopher Degawa
codec2.pc has been available since 2016, and I believe most distros have it by now. codec2-dev's commit: https://github.com/drowe67/codec2-dev/commit/fdfda67436b5b7308cf0758d9192f1dd540d439d Signed-off-by: Christopher Degawa --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 del

Re: [FFmpeg-devel] [PATCH v2 1/2] avcodec/libsvtav1: remove compressed_ten_bit_format and simplify alloc_buffer

2022-10-27 Thread Christopher Degawa
On Thu, Oct 20, 2022 at 11:56 PM Christopher Degawa wrote: > From: Christopher Degawa > > compressed_ten_bit_format has been deprecated upstream and has no effect > and can be removed. Plus, technically it was never used in the first place > since it would require the app (ffmpeg

Re: [FFmpeg-devel] [PATCH v2 1/2] avcodec/libsvtav1: remove compressed_ten_bit_format and simplify alloc_buffer

2022-11-17 Thread Christopher Degawa
On Thu, Oct 27, 2022 at 11:23 AM Christopher Degawa wrote: > > > On Thu, Oct 20, 2022 at 11:56 PM Christopher Degawa > wrote: > >> From: Christopher Degawa >> >> compressed_ten_bit_format has been deprecated upstream and has no effect >> and can be remov

[FFmpeg-devel] [PATCH] avcodec/libsvtav1: guard against bit_rate being zero

2022-11-19 Thread Christopher Degawa
division by zero occurs if it's not specified Signed-off-by: Christopher Degawa --- libavcodec/libsvtav1.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/libsvtav1.c b/libavcodec/libsvtav1.c index 48cd58a0b3..06874dfa63 100644 --- a/libavcodec/libsvt

[FFmpeg-devel] [PATCH v2] avcodec/libsvtav1: only set max_buf_sz if both bitrate and rc_buf_sz is set

2022-11-20 Thread Christopher Degawa
maximum_buffer_size_ms should only be set if both are specified or if the user sets it through -svtav1-params buf-sz=val Signed-off-by: Christopher Degawa --- libavcodec/libsvtav1.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/libsvtav1.c b/libavcodec

Re: [FFmpeg-devel] [PATCH] avcodec/libsvtav1: don't force a default value for deprecated options

2022-12-02 Thread Christopher Degawa
On Tue, Nov 29, 2022 at 4:38 PM James Almer wrote: > Stick to the library's default value instead. > > Should address AOMediaCodec/SVT-AV1 issue #2011. > > Signed-off-by: James Almer > --- > libavcodec/libsvtav1.c | 25 +++-- > 1 file changed, 15 insertions(+), 10 deletions(

[FFmpeg-devel] [PATCH] get_cabac_inline_x86: Don't inline if 32-bit Windows

2023-01-02 Thread Christopher Degawa
ted: https://trac.ffmpeg.org/ticket/8903 Signed-off-by: Christopher Degawa --- libavcodec/x86/cabac.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/cabac.h b/libavcodec/x86/cabac.h index b046a56a6b..70f990db8d 100644 --- a/libavcodec/x86/cabac.h +++ b/liba

Re: [FFmpeg-devel] [PATCH] get_cabac_inline_x86: Don't inline if 32-bit Windows

2023-01-03 Thread Christopher Degawa
On Mon, Jan 2, 2023 at 5:36 PM Hendrik Leppkes wrote: > On Tue, Jan 3, 2023 at 12:01 AM Christopher Degawa > wrote: > > > > I regularly build with 12.2 on win32 and its fine. > > In fact, there is a fate station for that: > > https://fate.ffmpeg.org/report.cgi?slo

Re: [FFmpeg-devel] [PATCH] get_cabac_inline_x86: Don't inline if 32-bit Windows

2023-01-03 Thread Christopher Degawa
On Tue, Jan 3, 2023 at 12:32 PM Christopher Degawa wrote: > > > On Mon, Jan 2, 2023 at 5:36 PM Hendrik Leppkes > wrote: > >> On Tue, Jan 3, 2023 at 12:01 AM Christopher Degawa >> wrote: >> > >> >> I regularly build with 12.2 on win32 and its

Re: [FFmpeg-devel] [PATCH] get_cabac_inline_x86: Don't inline if 32-bit Windows

2023-01-03 Thread Christopher Degawa
On Tue, Jan 3, 2023 at 4:16 AM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Christopher Degawa: > > previouslly, it only was an issue with 32-bit clang from msys2's > > mingw32 repo, however, at some point with an update to gcc 12.2.0, > > the sam

Re: [FFmpeg-devel] [PATCH] get_cabac_inline_x86: Don't inline if 32-bit Windows

2023-01-03 Thread Christopher Degawa
On Tue, Jan 3, 2023 at 12:51 PM Christopher Degawa wrote: > > > On Tue, Jan 3, 2023 at 4:16 AM Andreas Rheinhardt < > andreas.rheinha...@outlook.com> wrote: > >> Christopher Degawa: >> > previouslly, it only was an issue with 32-bit clang from msys2's

Re: [FFmpeg-devel] Problem with "-f concat" and mixed path types on Windows.

2021-01-03 Thread Christopher Degawa
> > I used the devel list because I considered, and still consider this as > a bug, and not a user "how-to" question. > My general rule of thumb I use whenever someone asks if they should post in #ffmpeg or #ffmpeg-devel (irc) etc is to only ask on devel if you are specifically hacking on ffmpeg a

Re: [FFmpeg-devel] Setup a rtmps server with nginx + ffmpeg.

2021-01-05 Thread Christopher Degawa
I think this topic is more suited for the ffmpeg-user mailing list instead of the devel mailing list ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-

[FFmpeg-devel] [PATCH] libavdevice/decklink_dec: mark get_bmd_timecode static

2021-01-06 Thread Christopher Degawa
mecode *tc, AVRational frame_rate, BMDTimecodeFormat tc_format, IDeckLinkVideoInputFrame *videoFrame) Signed-off-by: Christopher Degawa --- libavdevice/decklink_dec.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index

Re: [FFmpeg-devel] [PATCH] libavdevice/decklink_dec: mark get_bmd_timecode static

2021-01-06 Thread Christopher Degawa
I forgot to mark get_frame_timecode as static as well ___ 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 "unsubscr

Re: [FFmpeg-devel] [PATCH] lavf/srt: fix build fail when used the libsrt 1.4.1

2020-07-12 Thread Christopher Degawa
> Considering that 1.4.0 was released 10 months ago, I wonder whether > anyone compiled against a recent libsrt since then? I can confirm that this fixes compilation errors with the latest master branch of srt https://github.com/Haivision/srt/commit/0e2201aff6b379979cec43fee5e8f162717f6346 and FFm

Re: [FFmpeg-devel] [PATCH] lavf/srt: fix build fail when used the libsrt 1.4.1

2020-07-25 Thread Christopher Degawa
On Sat, Jul 18, 2020 at 9:24 AM myp...@gmail.com wrote: > > On Thu, Jul 16, 2020 at 9:35 AM myp...@gmail.com wrote: > > > > On Tue, Jul 14, 2020 at 9:47 PM Moritz Barsnick wrote: > > > > > > On Sun, Jul 12, 2020 at 22:44:46 +0800, myp...@gmail.com wrote: > > > > Maybe I give an inaccurate descri

Re: [FFmpeg-devel] Pull Request - Support Windows Subsystem for Linux

2020-08-01 Thread Christopher Degawa
Doesn't break msys2 builds, so it's okay to me. ___ 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".

Re: [FFmpeg-devel] [PATCH] configure: Add additional glslang libraries to make linking work

2020-08-20 Thread Christopher Degawa
This patch is needed to fix compilation on windows using the git master branches of both glslang and ffmpeg, although I don't know what version or commit of glslang started to require it, the earliest record I have of this issue is july 5th. ___ ffmpeg-de

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 3:31 PM Helmut K. C. Tessarek wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > Hello, > > The following 2 commits break the build with a deployment target of 10.9 on > macOS. > > I agree that 10.9 is quite old, but the rest of the code worked perfectly > fine

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 6:39 PM Aman Karmani wrote: > > Could you share the output of `xcrun metal -v` > > xcrun metal -v Apple LLVM version 31001.43 (metalfe-31001.43) Target: air64-apple-darwin19.6.0 Thread model: posix InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDe

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 6:55 PM Aman Karmani wrote: > On Mon, Dec 20, 2021 at 4:52 PM Helmut K. C. Tessarek < > tessa...@evermeet.cx> > wrote: > > > -BEGIN PGP SIGNED MESSAGE- > > Hash: SHA512 > > > > > > > > On 2021-12-20 19:38, Aman Karmani wrote: > > > Could you share the output of `xc

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 7:25 PM Aman Karmani wrote: > On Mon, Dec 20, 2021 at 5:19 PM Christopher Degawa > wrote: > > > On Mon, Dec 20, 2021 at 6:55 PM Aman Karmani wrote: > > > > > On Mon, Dec 20, 2021 at 4:52 PM Helmut K. C. Tessarek < > &

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 7:29 PM Ridley Combs wrote: > The main thing that confuses me here is that this file does build for me > when I use the iOS version of the metal compiler (tbh I didn't even know > there was a difference between them). > > What Xcode and macOS version are you on? > system

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 7:21 PM Aman Karmani wrote: > > > On Mon, Dec 20, 2021 at 5:03 PM Helmut K. C. Tessarek < > tessa...@evermeet.cx> wrote: > >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA512 >> >> On 2021-12-20 19:11, Christopher Degawa w

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-20 Thread Christopher Degawa
On Mon, Dec 20, 2021 at 7:53 PM Aman Karmani wrote: > > > On Mon, Dec 20, 2021 at 5:48 PM Christopher Degawa > wrote: > >> >> >> On Mon, Dec 20, 2021 at 7:21 PM Aman Karmani wrote: >> >>> >>> >>> On Mon, Dec 20, 2021

Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS

2021-12-21 Thread Christopher Degawa
On Tue, Dec 21, 2021 at 1:40 AM Ridley Combs wrote: > Huh, TIL that the metal compiler doesn't exist without Xcode, but the > metal framework does! Wacky. Anyway, this patch oughtta fix that (does more > or less as you suggested); if this is all fine on your end, I'll go ahead > and split this up

[FFmpeg-devel] [PATCH] get_cabac_inline_x86: Don't inline if 32-bit clang on windows

2021-08-17 Thread Christopher Degawa
Fixes https://trac.ffmpeg.org/ticket/8903 relevant https://github.com/msys2/MINGW-packages/discussions/9258 Signed-off-by: Christopher Degawa --- libavcodec/x86/cabac.h | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavcodec/x86/cabac.h b/libavcodec/x86/cabac.h

Re: [FFmpeg-devel] [PATCH v2 6/6] avcodec/libsvtav1: support constant quality mode

2021-09-19 Thread Christopher Degawa
On Fri, Sep 17, 2021 at 9:28 PM wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > As a note, I personally favor this patch over mine since I don't think `enable_tpl_la` is worth exposing to the user if its main role is to switch between crf and cqp __

Re: [FFmpeg-devel] [PATCH v2 6/6] avcodec/libsvtav1: support constant quality mode

2021-09-23 Thread Christopher Degawa
On Sun, Sep 19, 2021 at 2:02 PM Christopher Degawa wrote: > > > On Fri, Sep 17, 2021 at 9:28 PM wrote: > >> From: Limin Wang >> >> Signed-off-by: Limin Wang >> > As a note, I personally favor this patch over mine since I don't think > `enable_tp

[FFmpeg-devel] [PATCH] avcodec/libsvtav1: pass pict_type to library

2022-04-25 Thread Christopher Degawa
match the behavior of SvtAv1EncApp to ensure pic_type is always set before passing it to the library. The other options for pic_type aren't currently used inside the library, so they aren't introduced in this patch. Signed-off-by: Christopher Degawa --- libavcodec/libsvtav1.c | 5 +++

[FFmpeg-devel] [PATCH v2] avcodec/libsvtav1: pass pict_type to library

2022-04-25 Thread Christopher Degawa
match the behavior of SvtAv1EncApp to ensure pic_type is always set before passing it to the library. The other options for pic_type aren't currently used inside the library, so they aren't introduced in this patch. Signed-off-by: Christopher Degawa --- libavcodec/libsvt

[FFmpeg-devel] [PATCH] configure: add check for sdl2 >= 2.23.0

2022-05-10 Thread Christopher Degawa
sdl2 recently changed their versioning, moving the patch level to minor level https://github.com/libsdl-org/SDL/commit/cd7c2f1de7d9e418bb554047d714dd7cacc020ff trac: https://trac.ffmpeg.org/ticket/9768 Signed-off-by: Christopher Degawa --- configure | 7 +++ 1 file changed, 7 insertions

[FFmpeg-devel] [PATCH v2] configure: add check for sdl2 >= 2.23.0

2022-05-10 Thread Christopher Degawa
sdl2 recently changed their versioning, moving the patch level to minor level https://github.com/libsdl-org/SDL/commit/cd7c2f1de7d9e418bb554047d714dd7cacc020ff trac: https://trac.ffmpeg.org/ticket/9768 Signed-off-by: Christopher Degawa --- configure | 7 +++ 1 file changed, 7 insertions

[FFmpeg-devel] [PATCH] configure: extend check to less than 3.0.0

2022-05-11 Thread Christopher Degawa
sdl2 recently changed their versioning, moving the patch level to minor level https://github.com/libsdl-org/SDL/commit/cd7c2f1de7d9e418bb554047d714dd7cacc020ff and have said that they will instead ship sdl3.pc for 3.0.0 trac: https://trac.ffmpeg.org/ticket/9768 Signed-off-by: Christopher Degawa

Re: [FFmpeg-devel] [PATCH] avformat/os_support: use windows stat structs with 64bit time_t

2022-06-10 Thread Christopher Degawa
On Thu, Jun 9, 2022 at 6:22 PM softworkz wrote: > From: softworkz > > Signed-off-by: softworkz > --- > avformat/os_support: use windows stat structs with 64bit time_t > > Signed-off-by: softworkz softwo...@hotmail.com > > > Ping on this patch, this fixes an error with decklink due to cl

[FFmpeg-devel] [PATCH] libavcodec/libsvtav1: Allow -1 for preset option

2023-05-10 Thread Christopher Degawa
Currently, the -1 (MR) preset is disallowed as it's taken as the preset option not set, and the only way to access it was through svtav1-params. Signed-off-by: Christopher Degawa --- libavcodec/libsvtav1.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libav