Re: [FFmpeg-devel] [PATCH] libRIST: allow setting fifo size and fail on overflow.

2022-01-31 Thread Gijs Peskens
Bump, could this get another review? On 11-01-2022 16:07, Gijs Peskens wrote: Introduce fifo_size and overrun_nonfatal params to configure fifo buffer behavior. Use newly introduced RIST_DATA_FLAGS_OVERFLOW flag to check for overrun and error out in that case. --- doc/protocols.texi| 9 +

Re: [FFmpeg-devel] [PATCH 1/4] Use appropriate method for device discovery, fix crash with bogus device index.

2022-01-31 Thread Thilo Borgmann
Hi, Am 30.01.22 um 18:30 schrieb to...@rastageeks.org: From: Romain Beauxis This updates the code for avfoundation to use modern device lookup APIs and also adds a check to avoid querying the video devices array beyound its maximum size. --- libavdevice/avfoundation.m | 71 +++

Re: [FFmpeg-devel] [PATCH] avfilter: Added siti filter

2022-01-31 Thread Thilo Borgmann
Am 24.01.22 um 12:10 schrieb Thilo Borgmann: Am 18.01.22 um 14:58 schrieb Thilo Borgmann: Am 15.01.22 um 09:27 schrieb Paul B Mahol: On Sat, Jan 15, 2022 at 9:25 AM Paul B Mahol wrote: On Fri, Jan 14, 2022 at 5:22 PM Thilo Borgmann wrote: Hi, Am 19.01.21 um 05:49 schrieb Lynne: Jan 19

Re: [FFmpeg-devel] [PATCH v2] lavfi/drawtext: Add localtime_ms for millisecond precision

2022-01-31 Thread Thilo Borgmann
Am 20.01.22 um 16:32 schrieb Thilo Borgmann: Am 20.01.22 um 16:03 schrieb Andreas Rheinhardt: Thilo Borgmann: Am 20.01.22 um 13:04 schrieb Thilo Borgmann: Am 19.01.22 um 04:16 schrieb "zhilizhao(赵志立)": On Jan 18, 2022, at 8:52 PM, Thilo Borgmann wrote: Am 16.01.22 um 12:06 schrieb Nicola

Re: [FFmpeg-devel] [PATCH] avfilter: Added siti filter

2022-01-31 Thread Anton Khirnov
Quoting Thilo Borgmann (2022-01-18 14:58:07) > >> Violations of code style. > > Enhanced. Not enough. There are still many remaining, e.g. * opening brace of a function definition should be on its own line * the context should generally be the first argument * unsigned char* should be uint8_t* *

Re: [FFmpeg-devel] [PATCH] avfilter: Added siti filter

2022-01-31 Thread James Almer
On 1/31/2022 8:53 AM, Anton Khirnov wrote: Quoting Thilo Borgmann (2022-01-18 14:58:07) Violations of code style. Enhanced. Not enough. There are still many remaining, e.g. * opening brace of a function definition should be on its own line * the context should generally be the first argum

Re: [FFmpeg-devel] [PATCH] libavutil: include assembly with full path from source root

2022-01-31 Thread Anton Khirnov
Quoting Alexander Kanavin (2022-01-18 23:38:59) > On Tue, 18 Jan 2022 at 23:32, Lynne wrote: > > > > > Not what I'm seeing here in the binaries: "libavutil/x86/tx_float.asm", > > rather than a full path. > > > > Are you using a separate build directory? This is a sample reproducibility > failure

Re: [FFmpeg-devel] [PATCH] libavutil: include assembly with full path from source root

2022-01-31 Thread Alexander Kanavin
On Mon, 31 Jan 2022 at 13:52, Anton Khirnov wrote: > > Are you using a separate build directory? This is a sample > reproducibility > > failure we've observed: > > > https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20220118-y69m0ean/packages/diff-html/ > > With a separate build directo

Re: [FFmpeg-devel] [PATCH v2] lavfi/drawtext: Add localtime_ms for millisecond precision

2022-01-31 Thread Nicolas George
Thilo Borgman (12022-01-31): > > v10 attached. > > Also going to apply soon if there are no more comments. I think you neglected to attach the file. Regards, -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 001/293 v8] avutil/channel_layout: Add a new channel layout API

2022-01-31 Thread Anton Khirnov
Quoting Nicolas George (2022-01-27 16:08:10) > > + * be checked) > > + * > > + * No new fields may be added to it without a major version bump, except > > for > > + * new elements of the union fitting in sizeof(uint64_t). > > + */ > > +typedef struct AVChannelLayout { > > +/** > > + * Chan

Re: [FFmpeg-devel] [PATCH] libavutil: include assembly with full path from source root

2022-01-31 Thread Alexander Kanavin
On Mon, 31 Jan 2022 at 13:52, Anton Khirnov wrote: > With a separate build directory, I'm getting > $ strings libavutil/x86/tx_float.o |grep asm > src/libavutil/x86/tx_float.asm > The key piece is ../configure --disable-stripping With stripping disabled and without the patch, you should see: a

[FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support

2022-01-31 Thread Mark Gaiser
Hi, This patch series adds support for IPFS. I've been adviced to keep the patches as they are (split). If needed, I can squash them to a single patch. The following is a short summary. In the IPFS ecosystem you access it's content by a "Content IDentifier" (CID). This CID is, in simplified ter

[FFmpeg-devel] [PATCH 1/5] Early version of IPFS protocol support.

2022-01-31 Thread Mark Gaiser
Signed-off-by: Mark Gaiser --- configure | 1 + doc/protocols.texi | 30 ++ libavformat/Makefile| 1 + libavformat/ipfs.c | 202 libavformat/protocols.c | 2 + 5 files changed, 236 insertions(+) create mode 100644 li

[FFmpeg-devel] [PATCH 2/5] Fix up IPNS support.

2022-01-31 Thread Mark Gaiser
Signed-off-by: Mark Gaiser --- configure| 1 + libavformat/Makefile | 1 + libavformat/ipfs.c | 12 ++-- 3 files changed, 8 insertions(+), 6 deletions(-) diff --git a/configure b/configure index e466f924a3..6ff09e7974 100755 --- a/configure +++ b/configure @@ -3586,6 +358

[FFmpeg-devel] [PATCH 3/5] Merge IPNS and IPFS handling.

2022-01-31 Thread Mark Gaiser
Only the open function needs to detect which one is used. Signed-off-by: Mark Gaiser --- libavformat/ipfs.c | 99 +++--- 1 file changed, 22 insertions(+), 77 deletions(-) diff --git a/libavformat/ipfs.c b/libavformat/ipfs.c index 8daf032dd2..22487c6cc8 10

[FFmpeg-devel] [PATCH 4/5] Implement logic to determine the IPFS gateway.

2022-01-31 Thread Mark Gaiser
Inform the user about how to set a gateway if none could be found. Signed-off-by: Mark Gaiser --- libavformat/ipfs.c | 159 + 1 file changed, 147 insertions(+), 12 deletions(-) diff --git a/libavformat/ipfs.c b/libavformat/ipfs.c index 22487c6cc8..8f6

[FFmpeg-devel] [PATCH 5/5] Fix review feedback

2022-01-31 Thread Mark Gaiser
Little bit of consistency in punctuations. Signed-off-by: Mark Gaiser --- libavformat/ipfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/ipfs.c b/libavformat/ipfs.c index 8f6f91d85d..1644c74eac 100644 --- a/libavformat/ipfs.c +++ b/libavformat/ipf

Re: [FFmpeg-devel] [PATCH 1/4] Use appropriate method for device discovery, fix crash with bogus device index.

2022-01-31 Thread Romain Beauxis
Le lun. 31 janv. 2022 à 04:11, Thilo Borgmann a écrit : > > Hi, > > Am 30.01.22 um 18:30 schrieb to...@rastageeks.org: > > From: Romain Beauxis > > > > This updates the code for avfoundation to use modern device lookup APIs and also adds a check to avoid querying the video devices array beyound i

Re: [FFmpeg-devel] http: honor response headers in redirect caching

2022-01-31 Thread Ronald S. Bultje
Hi, On Sun, Jan 30, 2022 at 1:15 AM Eran Kornblau wrote: > > > > -Original Message- > > From: Eran Kornblau > > Sent: Wednesday, 19 January 2022 9:14 > > To: FFmpeg development discussions and patches > > Subject: RE: [FFmpeg-devel] http: honor response headers in redirect > caching > >

Re: [FFmpeg-devel] [PATCH 16/41] avcodec/mpegvideo: Use typedefs for MPV(Main)?(Dec|Enc)?Context

2022-01-31 Thread Michael Niedermayer
On Mon, Jan 31, 2022 at 12:05:11AM +0100, Andreas Rheinhardt wrote: > Michael Niedermayer: > > On Sun, Jan 30, 2022 at 07:27:24AM +0100, Andreas Rheinhardt wrote: > >> This is in preparation for actually adding such contexts > >> and e.g. moving fields only used by the main thread to > >> the corre

Re: [FFmpeg-devel] [PATCH 5/5] Fix review feedback

2022-01-31 Thread Michael Niedermayer
On Mon, Jan 31, 2022 at 02:51:16PM +0100, Mark Gaiser wrote: > Little bit of consistency in punctuations. > > Signed-off-by: Mark Gaiser > --- > libavformat/ipfs.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) Review fixes should be integrated into the patches when they a

Re: [FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support

2022-01-31 Thread Tomas Härdin
mån 2022-01-31 klockan 14:51 +0100 skrev Mark Gaiser: > > There are multiple ways to access files on the IPFS network. This > patch series > uses the gateway driven way. An IPFS node - by default - exposes a > local > gateway (say http://localhost:8080) which is then used to get content > from IP

Re: [FFmpeg-devel] [PATCH 1/5] Early version of IPFS protocol support.

2022-01-31 Thread Michael Niedermayer
On Mon, Jan 31, 2022 at 02:51:12PM +0100, Mark Gaiser wrote: > Signed-off-by: Mark Gaiser > --- > configure | 1 + > doc/protocols.texi | 30 ++ > libavformat/Makefile| 1 + > libavformat/ipfs.c | 202 > libavformat/pro

Re: [FFmpeg-devel] [PATCH 2/5] Fix up IPNS support.

2022-01-31 Thread Michael Niedermayer
On Mon, Jan 31, 2022 at 02:51:13PM +0100, Mark Gaiser wrote: > Signed-off-by: Mark Gaiser > --- > configure| 1 + > libavformat/Makefile | 1 + > libavformat/ipfs.c | 12 ++-- > 3 files changed, 8 insertions(+), 6 deletions(-) as with other fixes they should be stashed in

Re: [FFmpeg-devel] [PATCH 1/5] Early version of IPFS protocol support.

2022-01-31 Thread James Almer
On 1/31/2022 10:51 AM, Mark Gaiser wrote: Signed-off-by: Mark Gaiser --- configure | 1 + doc/protocols.texi | 30 ++ libavformat/Makefile| 1 + libavformat/ipfs.c | 202 libavformat/protocols.c | 2 + 5 fi

Re: [FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support

2022-01-31 Thread Mark Gaiser
On Mon, Jan 31, 2022 at 4:52 PM Tomas Härdin wrote: > mån 2022-01-31 klockan 14:51 +0100 skrev Mark Gaiser: > > > > There are multiple ways to access files on the IPFS network. This > > patch series > > uses the gateway driven way. An IPFS node - by default - exposes a > > local > > gateway (say

Re: [FFmpeg-devel] [PATCH 5/5] Fix review feedback

2022-01-31 Thread Mark Gaiser
On Mon, Jan 31, 2022 at 4:46 PM Michael Niedermayer wrote: > On Mon, Jan 31, 2022 at 02:51:16PM +0100, Mark Gaiser wrote: > > Little bit of consistency in punctuations. > > > > Signed-off-by: Mark Gaiser > > --- > > libavformat/ipfs.c | 12 ++-- > > 1 file changed, 6 insertions(+), 6 de

Re: [FFmpeg-devel] [PATCH 1/5] Early version of IPFS protocol support.

2022-01-31 Thread Mark Gaiser
On Mon, Jan 31, 2022 at 5:06 PM James Almer wrote: > > > On 1/31/2022 10:51 AM, Mark Gaiser wrote: > > Signed-off-by: Mark Gaiser > > --- > > configure | 1 + > > doc/protocols.texi | 30 ++ > > libavformat/Makefile| 1 + > > libavformat/ipfs.c | 202 +++

Re: [FFmpeg-devel] http: honor response headers in redirect caching

2022-01-31 Thread Ronald S. Bultje
Hi, On Mon, Jan 31, 2022 at 9:37 AM Ronald S. Bultje wrote: > Hi, > > On Sun, Jan 30, 2022 at 1:15 AM Eran Kornblau > wrote: > >> > >> > -Original Message- >> > From: Eran Kornblau >> > Sent: Wednesday, 19 January 2022 9:14 >> > To: FFmpeg development discussions and patches > > >> > Su

Re: [FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support

2022-01-31 Thread Tomas Härdin
mån 2022-01-31 klockan 17:31 +0100 skrev Mark Gaiser: > On Mon, Jan 31, 2022 at 4:52 PM Tomas Härdin > wrote: > > > mån 2022-01-31 klockan 14:51 +0100 skrev Mark Gaiser: > > > > > > There are multiple ways to access files on the IPFS network. This > > > patch series > > > uses the gateway driven

Re: [FFmpeg-devel] [PATCH 1/5] Early version of IPFS protocol support.

2022-01-31 Thread Lynne
Jan 31, 2022, 14:51 by mark...@gmail.com: > Signed-off-by: Mark Gaiser > --- > configure | 1 + > doc/protocols.texi | 30 ++ > libavformat/Makefile| 1 + > libavformat/ipfs.c | 202 > libavformat/protocols.c | 2 + >

Re: [FFmpeg-devel] [PATCH v2] lavfi/drawtext: Add localtime_ms for millisecond precision

2022-01-31 Thread Thilo Borgmann
Am 31.01.22 um 14:08 schrieb Nicolas George: Thilo Borgman (12022-01-31): v10 attached. Also going to apply soon if there are no more comments. I think you neglected to attach the file. omg stupid me. Here it is... -ThiloFrom 56e3af9f9039aa09b9daea3a47dea9fb24b19cec Mon Sep 17 00:00:00 20

Re: [FFmpeg-devel] [PATCH 0/5] Add IPFS and IPNS protocol support

2022-01-31 Thread Mark Gaiser
On Mon, Jan 31, 2022 at 9:23 PM Tomas Härdin wrote: > mån 2022-01-31 klockan 17:31 +0100 skrev Mark Gaiser: > > On Mon, Jan 31, 2022 at 4:52 PM Tomas Härdin > > wrote: > > > > > mån 2022-01-31 klockan 14:51 +0100 skrev Mark Gaiser: > > > > > > > > There are multiple ways to access files on the I

Re: [FFmpeg-devel] [PATCH 1/5] Early version of IPFS protocol support.

2022-01-31 Thread Mark Gaiser
On Mon, Jan 31, 2022 at 9:26 PM Lynne wrote: > Jan 31, 2022, 14:51 by mark...@gmail.com: > > > Signed-off-by: Mark Gaiser > > --- > > configure | 1 + > > doc/protocols.texi | 30 ++ > > libavformat/Makefile| 1 + > > libavformat/ipfs.c | 202 +++

[FFmpeg-devel] [PATCH] libavformat/mov: Expose Quicktime poster_time value as metadata TAG.

2022-01-31 Thread Bryce Newman
I need the ability to derive the poster time found in the mvhd, so I can use that value to create a thumbnail from ffmpeg. More details can be found here https://www.mail-archive.com/ffmpeg-user@ffmpeg.org/msg30003.html Signed-off-by: Bryce Chester Newman --- libavformat/mov.c | 12 +++- 1

[FFmpeg-devel] [PATCH] libavformat/mov: Expose Quicktime poster_time value as metadata TAG.

2022-01-31 Thread Bryce Newman
I need the ability to derive the poster time found in the mvhd, so I can use that value to create a thumbnail from ffmpeg. More details can be found here https://www.mail-archive.com/ffmpeg-user@ffmpeg.org/msg30003.html Signed-off-by: Bryce Chester Newman --- libavformat/mov.c | 12 +++- 1

Re: [FFmpeg-devel] [PATCH] libRIST: allow setting fifo size and fail on overflow.

2022-01-31 Thread Marton Balint
On Tue, 11 Jan 2022, Gijs Peskens wrote: Introduce fifo_size and overrun_nonfatal params to configure fifo buffer behavior. Use newly introduced RIST_DATA_FLAGS_OVERFLOW flag to check for overrun and error out in that case. --- doc/protocols.texi| 9 + libavformat/librist.c | 37

Re: [FFmpeg-devel] [PATCH v2] lavfi/drawtext: Add localtime_ms for millisecond precision

2022-01-31 Thread Andreas Rheinhardt
Thilo Borgmann: > Am 31.01.22 um 14:08 schrieb Nicolas George: >> Thilo Borgman (12022-01-31): v10 attached. >>> >>> Also going to apply soon if there are no more comments. >> >> I think you neglected to attach the file. > > omg stupid me. Here it is... > > -Thilo > > > + > +i

[FFmpeg-devel] [PATCH] avformat/imf_cpl: do not use filesize when reading XML file

2022-01-31 Thread Marton Balint
Similar to the earlier patch applied to imfdec. Signed-off-by: Marton Balint --- libavformat/imf_cpl.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/libavformat/imf_cpl.c b/libavformat/imf_cpl.c index f2ad9c05d6..102a6b4549 100644 --- a/libavformat/imf_cpl.c +

[FFmpeg-devel] [PATCH] libavformat/mov: Expose Quicktime poster_time value as metadata TAG.

2022-01-31 Thread Bryce Newman
I need the ability to derive the poster time found in the mvhd, so I can use that value to create a thumbnail from ffmpeg. More details can be found here https://www.mail-archive.com/ffmpeg-user@ffmpeg.org/msg30003.html Signed-off-by: Bryce Chester Newman --- libavformat/mov.c | 12 +++- 1

[FFmpeg-devel] [PATCH] libavformat/mov: Expose Quicktime poster_time value as metadata TAG.

2022-01-31 Thread Bryce Newman
I need the ability to derive the poster time found in the mvhd, so I can use that value to create a thumbnail from ffmpeg. More details can be found here https://www.mail-archive.com/ffmpeg-user@ffmpeg.org/msg30003.html Signed-off-by: Bryce Chester Newman --- libavformat/mov.c | 12 +++- 1

Re: [FFmpeg-devel] [PATCH v2] lavfi/drawtext: Add localtime_ms for millisecond precision

2022-01-31 Thread Andreas Rheinhardt
Thilo Borgmann: > Am 31.01.22 um 14:08 schrieb Nicolas George: >> Thilo Borgman (12022-01-31): v10 attached. >>> >>> Also going to apply soon if there are no more comments. >> >> I think you neglected to attach the file. > > omg stupid me. Here it is... > > -Thilo > Seems like I misunderst

[FFmpeg-devel] [PATCH 1/4] avformat/demux: don't propagate unsupported skip samples packet side data values

2022-01-31 Thread James Almer
Should fix ticket #9622 Signed-off-by: James Almer --- I'm not sure if this is ok or not. The AV_PKT_DATA_SKIP_SAMPLES doxy states the skip samples value is a little endian uint32 value, so even if the mov demuxer wrote a truncated int64_t value in sti->skip_samples (which, being an int, can be n

[FFmpeg-devel] [PATCH 2/4] avcodec/decode: ignore unsupported skip samples packet side data values

2022-01-31 Thread James Almer
Same as in the AV_FRAME_FLAG_DISCARD codepath, ensure avci->skip_samples is not negative. Signed-off-by: James Almer --- libavcodec/decode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/decode.c b/libavcodec/decode.c index 0912f86a14..4f9b949926 100644 --- a/libavcodec/decode.c

[FFmpeg-devel] [PATCH 3/4] avformat/demux: print skip_samples and discard_padding as unsigned values in debug log

2022-01-31 Thread James Almer
It's the type they should be interpreted as in the AV_PKT_DATA_SKIP_SAMPLES side data. Signed-off-by: James Almer --- libavformat/demux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/demux.c b/libavformat/demux.c index 09d539af68..4509015847 100644 --- a/liba

[FFmpeg-devel] [PATCH 4/4] avformat/demux: don't truncate the return value of ts_to_samples()

2022-01-31 Thread James Almer
Signed-off-by: James Almer --- libavformat/demux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/demux.c b/libavformat/demux.c index 4509015847..ec34b65288 100644 --- a/libavformat/demux.c +++ b/libavformat/demux.c @@ -1346,7 +1346,7 @@ static int read_frame_inte

[FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs.

2022-01-31 Thread Ze Yuan
>From babcceafbd30eff677b2366a0c470d31c503bed1 Mon Sep 17 00:00:00 2001 From: TianBo Zheng Date: Mon, 10 Jan 2022 11:18:56 + Subject: [PATCH] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs. MKV codec mappings: V_AVS2 and V_AVS3 (https://github.com/ietf-wg-cellar/matroska-specification

Re: [FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs.

2022-01-31 Thread Andreas Rheinhardt
Ze Yuan: > From babcceafbd30eff677b2366a0c470d31c503bed1 Mon Sep 17 00:00:00 2001 > From: TianBo Zheng > Date: Mon, 10 Jan 2022 11:18:56 + > Subject: [PATCH] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs. > > MKV codec mappings: > V_AVS2 and V_AVS3 > (https://github.com/ietf-wg-cella

Re: [FFmpeg-devel] [PATCH v1 3/4] avformat/imf: fix packet pts, dts and muxing

2022-01-31 Thread Andreas Rheinhardt
Pierre-Anthony Lemieux: > On Sun, Jan 30, 2022 at 2:16 PM Andreas Rheinhardt > wrote: >> >> p...@sandflow.com: >>> From: Pierre-Anthony Lemieux >>> >>> The IMF demuxer does not set the DTS and PTS of packets accurately in all >>> scenarios. Moreover, audio packets are not trimmed when they exceed

Re: [FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs.

2022-01-31 Thread Ze Yuan
From: Andreas Rheinhardt Sent: Tuesday, February 1, 2022 12:43 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs. >Ze Yuan: >> From babcceafbd30eff677b2366a0c470d31c503bed1 Mon Sep 17 00:00:00 2001 >> From: TianBo Zheng >> Date: Mon, 10