Re: [FFmpeg-devel] [PATCH] configure: Fix Microsoft tools detection

2022-02-02 Thread Kacper Michajlow
On Wed, 26 Jan 2022 at 15:00, Martin Storsjö wrote: > > Hi, > > On Sat, 22 Jan 2022, Kacper Michajłow wrote: > > > LLVM tools print installation path upon execution. If one uses LLVM > > tools bundled with Microsoft Visual Studio installation, they would be > > incorrectly detected as Microsoft's

[FFmpeg-devel] [PATCH v3 4/4] avformat/imf: document IMFVirtualTrackResourcePlaybackCtx

2022-02-02 Thread pal
From: Pierre-Anthony Lemieux --- libavformat/imfdec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/imfdec.c b/libavformat/imfdec.c index 5be4411cb1..02e7bcc33f 100644 --- a/libavformat/imfdec.c +++ b/libavformat/imfdec.c @@ -96,12 +96,12 @@ typedef

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

2022-02-02 Thread pal
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 the duration of the underlying resource. imf-cpl-with-repeat FATE ref file is regenerated. Addresses https://trac.ffmpeg.org

[FFmpeg-devel] [PATCH v3 2/4] avformat/imf: fix missing error reporting when opening resources

2022-02-02 Thread pal
From: Pierre-Anthony Lemieux --- libavformat/imfdec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/imfdec.c b/libavformat/imfdec.c index e6a1020ecc..658ddc40f2 100644 --- a/libavformat/imfdec.c +++ b/libavformat/imfdec.c @@ -550,7 +550,9 @@ static int set_co

[FFmpeg-devel] [PATCH v3 1/4] avformat/imf: open resources only when first needed

2022-02-02 Thread pal
From: Pierre-Anthony Lemieux IMF CPLs can reference thousands of files, which can result in system limits for the number of open files to be exceeded. The following patch opens and closes files as needed. Addresses https://trac.ffmpeg.org/ticket/9623 --- libavformat/imfdec.c | 15 -

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix handling of backslashes

2022-02-02 Thread Oneric
On Wed, Feb 02, 2022 at 22:44:18 +, Soft Works wrote: > > Sent: Wednesday, February 2, 2022 11:19 PM > > To: FFmpeg development discussions and patches > de...@ffmpeg.org> > > > > [claims about git and smtp: part 1] > > > > [claims about git and smtp: part 2] > None of this relates to th

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

2022-02-02 Thread Pierre-Anthony Lemieux
Works for me. There are several other IMF-related patches waiting, some of them rather trivial. Should they be combined and/or merged together? On Mon, Jan 31, 2022 at 3:13 PM Marton Balint wrote: > > Similar to the earlier patch applied to imfdec. > > Signed-off-by: Marton Balint > --- > liba

Re: [FFmpeg-devel] [PATCH 1/2] http: Improve handling of Content-Range with Transfer-Encoding:chunked

2022-02-02 Thread Aman Karmani
On Wed, Feb 2, 2022 at 3:50 PM Vittorio Giovara wrote: > From: Justin Ruggles > > When Transfer-Encoding:chunked is used, the client must ignore a > Content-Length header, if present. However, it should not ignore a > Content-Range header, which also includes the full size of the > entity. > ---

[FFmpeg-devel] [PATCH 78/80] avcodec/mpegvideo: Move packed-b-frames stuff to H263DecContext

2022-02-02 Thread Andreas Rheinhardt
It is only used by the MPEG-4 decoder, yet it is used in the common H.263-code (i.e. in libavcodec/h263dec.c) and can therefore not be put in Mpeg4DecContext. Yet moving it to H263DecContext is possible. Given that this is nevertheless an MPEG-4 only field, it is kept in sync in the MPEG-4-decoder

[FFmpeg-devel] [PATCH 80/80] avcodec/mpegvideo_enc: Remove redundant unref+ref

2022-02-02 Thread Andreas Rheinhardt
Setting current_picture will already be done in frame_start(). Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo_enc.c | 5 - 1 file changed, 5 deletions(-) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index ddfd123c46..9c68983205 100644 --- a/libavcodec/mpe

[FFmpeg-devel] [PATCH 79/80] avcodec/mpegvideo: Move custom_pcf to H263DecContext

2022-02-02 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.h| 1 + libavcodec/ituh263dec.c | 9 - libavcodec/mpegvideo.h | 1 - 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/libavcodec/h263dec.h b/libavcodec/h263dec.h index 04c238d90a..539f2df01a 100644 --- a/libavcode

[FFmpeg-devel] [PATCH 77/80] avcodec/mpegvideo: Move pb_frame to H263DecContext

2022-02-02 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.h | 1 + libavcodec/intelh263dec.c | 6 +++--- libavcodec/ituh263dec.c | 21 - libavcodec/mpegvideo.h| 1 - 4 files changed, 16 insertions(+), 13 deletions(-) diff --git a/libavcodec/h263dec.h b/libavcodec

[FFmpeg-devel] [PATCH 76/80] avcodec/mpegvideo: Move ehc_mode to H263DecContext

2022-02-02 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/flvdec.c | 2 +- libavcodec/h263dec.c| 2 +- libavcodec/h263dec.h| 1 + libavcodec/ituh263dec.c | 2 +- libavcodec/mpegvideo.h | 1 - 5 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/flvdec.c b/libavcodec/flvde

[FFmpeg-devel] [PATCH 75/80] avcodec/mpegvideo: Move h263_long_vectors to H263DecContext

2022-02-02 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/flvdec.c | 2 +- libavcodec/h263dec.h | 1 + libavcodec/intelh263dec.c | 2 +- libavcodec/ituh263dec.c | 9 ++--- libavcodec/mpegvideo.h| 1 - libavcodec/rv10.c | 2 +- 6 files changed, 10 insertions(+), 7 deletions(-)

[FFmpeg-devel] [PATCH 73/80] avcodec/h263dec.h: Add H263DecContext

2022-02-02 Thread Andreas Rheinhardt
This is in preparation for moving fields only used by h263dec-based decoders from MPVContext. Signed-off-by: Andreas Rheinhardt --- libavcodec/dxva2_vc1.c | 16 +-- libavcodec/flvdec.c| 6 ++-- libavcodec/flvdec.h| 4 +-- libavcodec/h263dec.c |

[FFmpeg-devel] [PATCH 74/80] avcodec/mpegvideo: Move decode_mb to H263DecContext

2022-02-02 Thread Andreas Rheinhardt
Only used by h263dec-based decoders; and only set by them in their main threads. Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.c | 7 +-- libavcodec/h263dec.h | 2 ++ libavcodec/mpeg4videodec.c | 8 libavcodec/mpegvideo.h | 1 - libavcodec/msmpeg4dec.c

[FFmpeg-devel] [PATCH] avformat/rtpdec_rfc4175: fix interlace format

2022-02-02 Thread Patrick Keroulas
In previous state, a new frame was allocated on each timestamp step, i.e. each frame/field transition. However, for interlace, a new frame should be allocated on 1st field, completed with the 2nd and finally freed. This commit fixes the frame allocation and the detection of missing RTP markers. S

[FFmpeg-devel] [PATCH 2/2] http: Send a Range header even when the offset is 0

2022-02-02 Thread Vittorio Giovara
From: Justin Ruggles Using Range allows for getting the full file size from the Content-Range header in the response, even if the server sends back the response using chunked Transfer-Encoding, which does not allow using Content-Length. --- libavformat/http.c | 4 ++-- 1 file changed, 2 insertio

[FFmpeg-devel] [PATCH 1/2] http: Improve handling of Content-Range with Transfer-Encoding:chunked

2022-02-02 Thread Vittorio Giovara
From: Justin Ruggles When Transfer-Encoding:chunked is used, the client must ignore a Content-Length header, if present. However, it should not ignore a Content-Range header, which also includes the full size of the entity. --- libavformat/http.c | 8 +++- 1 file changed, 7 insertions(+), 1

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

2022-02-02 Thread Bryce Chester Newman
From: Bryce Chester 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 --- liba

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix handling of backslashes

2022-02-02 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Soft > Works > Sent: Wednesday, February 2, 2022 11:19 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix > handling of backslashes > > >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix handling of backslashes

2022-02-02 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Wednesday, February 2, 2022 6:03 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix > handling of backslashes > > It appea

Re: [FFmpeg-devel] fatal: Resolve operation not in progress, we are not resuming.

2022-02-02 Thread Andriy Gelman
Hi, On Wed, 02. Feb 16:02, Bryce Newman wrote: > Hi, > I am trying to apply this patch > https://patchwork.ffmpeg.org/project/ffmpeg/patch/d412c54e-2e51-4522-a8fb-32c5c6ca6...@gettyimages.com/. > I am able to `git apply` the patch locally and `make fate passes`, so I am > not sure what I need to

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix handling of backslashes

2022-02-02 Thread Oneric
It appears my previous mail has been interpreted as some sort of attack against you; this is not the case. I am sorry for creating such a misunderstanding. The mail was merely meant to clear up the misconception that it would be impossible to send mixed line-endings via mail due to some SMTP prope

Re: [FFmpeg-devel] [PATCH v4] avformat/hls: Implement support for using AVSEEK_FLAG_BACKWARD when seeking

2022-02-02 Thread Gustav Grusell
Any comments on this? cheers, Gustav On Fri, Jan 28, 2022 at 9:24 PM Gustav Grusell wrote: > Before, seeking in hls streams would always seek to the next keyframe > after the given timestamp. With this fix, if seeking in videostream and > AVSEEK_FLAG_BACKWARD is set, seeking will be to the firs

[FFmpeg-devel] fatal: Resolve operation not in progress, we are not resuming.

2022-02-02 Thread Bryce Newman
Hi, I am trying to apply this patch https://patchwork.ffmpeg.org/project/ffmpeg/patch/d412c54e-2e51-4522-a8fb-32c5c6ca6...@gettyimages.com/. I am able to `git apply` the patch locally and `make fate passes`, so I am not sure what I need to do to get this patch to not throw warnings. I have tried

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

2022-02-02 Thread Ze Yuan
Let’s merge it. From: Ze Yuan Sent: Tuesday, February 1, 2022 12:36 PM To: ffmpeg-devel Subject: [FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs. From babcceafbd30eff677b2366a0c470d31c503bed1 Mon Sep 17 00:00:00 2001 From: TianBo Zheng Date: Mon, 10 Jan 2022 11:18:56 + Sub

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Mark Gaiser
On Wed, Feb 2, 2022 at 3:24 PM Timo Rothenpieler wrote: > On 02.02.2022 14:56, Mark Gaiser wrote: > > On Wed, Feb 2, 2022 at 2:21 PM Tomas Härdin wrote: > > > >> tis 2022-02-01 klockan 22:58 +0100 skrev Mark Gaiser: > >> > >>> > >>> +typedef struct Context { > >>> +AVClass *class; > >>> +

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Mark Gaiser
On Wed, Feb 2, 2022 at 2:29 PM Michael Niedermayer wrote: > On Tue, Feb 01, 2022 at 10:58:30PM +0100, Mark Gaiser wrote: > > This patch adds support for: > > - ffplay ipfs:// > > - ffplay ipns:// > > > > IPFS data can be played from so called "ipfs gateways". > > A gateway is essentially a webser

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Timo Rothenpieler
On 02.02.2022 14:56, Mark Gaiser wrote: On Wed, Feb 2, 2022 at 2:21 PM Tomas Härdin wrote: tis 2022-02-01 klockan 22:58 +0100 skrev Mark Gaiser: +typedef struct Context { +AVClass *class; +URLContext *inner; +char *gateway; Is there not a maximum length that an HTTP URL can be

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Mark Gaiser
On Wed, Feb 2, 2022 at 2:21 PM Tomas Härdin wrote: > tis 2022-02-01 klockan 22:58 +0100 skrev Mark Gaiser: > > > > > +typedef struct Context { > > +AVClass *class; > > +URLContext *inner; > > +char *gateway; > > Is there not a maximum length that an HTTP URL can be? At least without >

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

2022-02-02 Thread Michael Niedermayer
On Tue, Feb 01, 2022 at 05:43:24PM +0100, Tomas Härdin wrote: > tis 2022-02-01 klockan 11:06 +0100 skrev Michael Niedermayer: > > On Mon, Jan 31, 2022 at 09:22:52PM +0100, Tomas Härdin wrote: > > [...] > > > It strikes me that this borders on incorporating business logic > > > within > > > lavf. A

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

2022-02-02 Thread Mark Gaiser
On Wed, Feb 2, 2022 at 1:52 PM Tomas Härdin wrote: > tis 2022-02-01 klockan 22:18 +0100 skrev Mark Gaiser: > > > > To give you an idea of how it looks. Kodi has so called strm (stream) > > files. They can contain an url that can be played. > > > > Without this patch an ipfs file would look like t

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Michael Niedermayer
On Tue, Feb 01, 2022 at 10:58:30PM +0100, Mark Gaiser wrote: > This patch adds support for: > - ffplay ipfs:// > - ffplay ipns:// > > IPFS data can be played from so called "ipfs gateways". > A gateway is essentially a webserver that gives access to the > distributed IPFS network. > > This protoc

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Tomas Härdin
tis 2022-02-01 klockan 22:58 +0100 skrev Mark Gaiser: > > +typedef struct Context { > +    AVClass *class; > +    URLContext *inner; > +    char *gateway; Is there not a maximum length that an HTTP URL can be? At least without query parameters. That way you avoid dynamic allocations. You'd have

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

2022-02-02 Thread Tomas Härdin
tis 2022-02-01 klockan 22:18 +0100 skrev Mark Gaiser: > > To give you an idea of how it looks. Kodi has so called strm (stream) > files. They can contain an url that can be played. > > Without this patch an ipfs file would look like this: > > http://localhost:8080/ipfs/QmbGtJg23skhvFmu9mJiePVBy

Re: [FFmpeg-devel] [PATCH 1/4] libswscale: Re-factor ff_shuffle_filter_coefficients.

2022-02-02 Thread Alan Kelly
Hi, Is anybody interested in this patch set? Thanks! On Mon, Jan 10, 2022, 15:58 Alan Kelly wrote: > Make the code more readable, follow the style guide and propagate memory > allocation errors. > --- > libswscale/swscale_internal.h | 2 +- > libswscale/utils.c| 68 ++

Re: [FFmpeg-devel] [PATCH v2 1/1] avformat: Add IPFS protocol support.

2022-02-02 Thread Lynne
2 Feb 2022, 03:51 by mark...@gmail.com: > On Wed, Feb 2, 2022 at 3:29 AM Lynne wrote: > >> 1 Feb 2022, 22:58 by mark...@gmail.com: >> >> > This patch adds support for: >> > - ffplay ipfs:// >> > - ffplay ipns:// >> > >> > IPFS data can be played from so called "ipfs gateways". >> > A gateway is e