[FFmpeg-devel] [GASPP PATCH 2/2] Use cl for preprocessing when using armasm

2020-09-23 Thread Martin Storsjö
When invoking gas-preprocessor to preprocess only to get the dependency list, we still need to use cpp, as cl doesn't support the -M options. This allows using gas-preprocessor in an environment with only MSVC available, no GCC/Clang providing a cpp.exe, allowing building e.g. dav1d with MSVC targ

[FFmpeg-devel] Tech consultant for video PoC

2020-09-23 Thread Michal Čudrnák
Hi all, We’re looking for a tech consultant for a commercial proof of concept we’re working on with a video flow involving RT(S)P, ONVIF cameras, some OpenCV, video storage & playback, an iOS/macOS player and video + time-series data presentation sync. We’re all architects and developers but co

Re: [FFmpeg-devel] [PATCH] tests/fate-run: Always overwrite output files for md5 tests

2020-09-23 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Otherwise the result of such tests will not accurately reflect the > current state. > > Signed-off-by: Andreas Rheinhardt > --- > tests/fate-run.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/fate-run.sh b/tests/fate-run.sh > index 1b283e

Re: [FFmpeg-devel] [PATCH 1/3] avformat/swfdec: Fix memleaks on error

2020-09-23 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt > --- > libavformat/swfdec.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/libavformat/swfdec.c b/libavformat/swfdec.c > index e427998744..7a74fa3ac7 100644 > --- a/libavformat/swfdec.c > +++ b/libavformat/s

Re: [FFmpeg-devel] [PATCH 2/3] avformat/swfdec: Reorder allocations/initializations

2020-09-23 Thread Andreas Rheinhardt
Andreas Rheinhardt: > The earlier code would first attempt to allocate two buffers, then > attempt to allocate an AVIOContext, using one of the new buffers I/O > buffer, then check the allocations. On success, a z_stream that is used > in the AVIOContext's read_packet callback is initialized afterw

Re: [FFmpeg-devel] [PATCH] tests/fate-run: Always overwrite output files for md5 tests

2020-09-23 Thread Jan Ekström
On Sun, Sep 20, 2020 at 5:54 PM Andreas Rheinhardt wrote: > > Otherwise the result of such tests will not accurately reflect the > current state. > > Signed-off-by: Andreas Rheinhardt > --- > tests/fate-run.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/fate-ru

Re: [FFmpeg-devel] [PATCH] tests/fate-run: Always overwrite output files for md5 tests

2020-09-23 Thread Andreas Rheinhardt
Jan Ekström: > On Sun, Sep 20, 2020 at 5:54 PM Andreas Rheinhardt > wrote: >> >> Otherwise the result of such tests will not accurately reflect the >> current state. >> >> Signed-off-by: Andreas Rheinhardt >> --- >> tests/fate-run.sh | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >>

Re: [FFmpeg-devel] [PATCH v3] avutil/pixfmt: improve definition of AVColorRange

2020-09-23 Thread Vittorio Giovara
On Sun, Sep 20, 2020 at 8:48 PM Jan Ekström wrote: > On Sun, Sep 20, 2020 at 9:34 PM Vittorio Giovara > wrote: > > > > On Sun, Sep 20, 2020 at 7:12 PM Jan Ekström wrote: > > > > > As it was brought up that the current documentation leaves things > > > as specific to YCbCr only, ICtCp and RGB ar

[FFmpeg-devel] [PATCH] avformat/udp: remove redundant setting of h->max_packet_size

2020-09-23 Thread Zhao Zhili
h->max_packet_size is being reset in the following code. --- libavformat/udp.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/libavformat/udp.c b/libavformat/udp.c index 30d8041433..96d8b8ee11 100644 --- a/libavformat/udp.c +++ b/libavformat/udp.c @@ -650,9 +650,6 @@ static int udp_open(URL

[FFmpeg-devel] [PATCH] avformat/mov: move av_freep to if block to reduce no-ops

2020-09-23 Thread Zhao Zhili
Use the check since it's there already. --- libavformat/mov.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 4f64e96bc0..dcd263b02a 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2909,9 +2909,10 @@ static int mov_rea

Re: [FFmpeg-devel] [PATCH v3] avutil/pixfmt: improve definition of AVColorRange

2020-09-23 Thread Jan Ekström
On Wed, Sep 23, 2020 at 6:49 PM Vittorio Giovara wrote: > > On Sun, Sep 20, 2020 at 8:48 PM Jan Ekström wrote: > > > On Sun, Sep 20, 2020 at 9:34 PM Vittorio Giovara > > wrote: > > > > > > On Sun, Sep 20, 2020 at 7:12 PM Jan Ekström wrote: > > > > > > > As it was brought up that the current doc

[FFmpeg-devel] [PATCH] avformat/mov: make better use of av_fast_realloc and fix spurious ENOMEM

2020-09-23 Thread Zhao Zhili
If sc->ctts_allocated_size is larger than the new buffer size, av_fast_realloc() will return NULL. Since sc->ctts_data is freed, ctts_allocated_size should be reset to zero. It's better to avoid free sc->ctts_data at the first place to make better use of av_fast_realloc(). --- libavformat/mov.c |

Re: [FFmpeg-devel] [PATCH v3] libavcodec/vp8dec: fix the multi-thread HWAccel decode error

2020-09-23 Thread Eoff, Ullysses A
I tested: PASS: make THREADS=2 THREAD_TYPE=frame fate-vp8 PASS: make THREADS=1 THREAD_TYPE=frame fate-vp8 PASS: make THREADS=8 THREAD_TYPE=frame fate-vp8 PASS: make THREADS=6 THREAD_TYPE=frame fate-vp8 FAIL: make THREADS=2 THREAD_TYPE=tile fate-vp8 FAIL: make THREADS=1 THREAD_TYPE=tile fate-vp8 F

Re: [FFmpeg-devel] [PATCH v3] libavcodec/vp8dec: fix the multi-thread HWAccel decode error

2020-09-23 Thread Eoff, Ullysses A
> -Original Message- > From: ffmpeg-devel On Behalf Of Eoff, > Ullysses A > Sent: Wednesday, September 23, 2020 10:33 AM > To: Xiang, Haihao ; ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v3] libavcodec/vp8dec: fix the > multi-thread HWAccel decode error > > I tested: >

[FFmpeg-devel] [PATCH v2] examples/decode_video: flush parser to fix missing frame

2020-09-23 Thread Zhao Zhili
To reproduce, run decode_video with a single frame sample. No frame was decoded before the patch. --- doc/examples/decode_video.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/doc/examples/decode_video.c b/doc/examples/decode_video.c index 18ee90a6c0..9ce3531d63

Re: [FFmpeg-devel] [PATCH] examples/decode_video: flush parser to fix missing frame

2020-09-23 Thread Zhao Zhili
> On Sep 9, 2020, at 1:55 PM, Guangxin Xu wrote: > > > > On Tue, Aug 18, 2020 at 8:05 PM > wrote: > From: Zhao Zhili mailto:quinkbl...@foxmail.com>> > > To reproduce, run decode_video with a single frame sample. No frame > was decoded before the patch. > --- >

Re: [FFmpeg-devel] [PATCH] avformat/http: fix memleak

2020-09-23 Thread Zhao Zhili
Ping for review. > On Aug 24, 2020, at 12:58 AM, Zhao Zhili wrote: > > --- > libavformat/http.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavformat/http.c b/libavformat/http.c > index 6c39da1a8b..3d25d652d3 100644 > --- a/libavformat/http.c > +++ b/libavformat/

Re: [FFmpeg-devel] [PATCH v3] libavcodec/vp8dec: fix the multi-thread HWAccel decode error

2020-09-23 Thread Ronald S. Bultje
Hi, On Wed, Sep 23, 2020 at 1:48 PM Eoff, Ullysses A wrote: > > -Original Message- > > From: ffmpeg-devel On Behalf Of Eoff, > Ullysses A > > Sent: Wednesday, September 23, 2020 10:33 AM > > To: Xiang, Haihao ; ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] [PATCH v3] libavcode

Re: [FFmpeg-devel] [PATCH] avformat/flvdec: RtmpSampleAccess no longer breaks stream detection

2020-09-23 Thread Michael Niedermayer
On Mon, Sep 21, 2020 at 12:10:04PM +0200, Peter van der Spek wrote: > Hi Michael, > > Friendly ping to get this one merged. will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Nations do behave wisely once they have exhausted all other alternatives.

Re: [FFmpeg-devel] [PATCH] avfilter/setparams: add FF_FILTER_FLAG_HWFRAME_AWARE

2020-09-23 Thread Paul B Mahol
On Mon, Sep 21, 2020 at 09:47:40PM -0600, Pavel Koshevoy wrote: > Allow setparams to be used with hw backed frames and > avoid an assertion failure in avfilter_config_links. > --- > libavfilter/vf_setparams.c | 3 +++ > 1 file changed, 3 insertions(+) > LGTM > diff --git a/libavfilter/vf_setpar

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: check that lowpass_height is >= 3 when used in vertical filter

2020-09-23 Thread Michael Niedermayer
On Tue, Sep 22, 2020 at 08:50:57PM +0200, Paul B Mahol wrote: > Also check for out of buffer access. > Also return early when encountering fatal error. > > Signed-off-by: Paul B Mahol > --- > libavcodec/cfhd.c | 89 +-- > 1 file changed, 56 insertions(

Re: [FFmpeg-devel] [PATCH] avformat/udp: remove redundant setting of h->max_packet_size

2020-09-23 Thread myp...@gmail.com
On Thu, Sep 24, 2020 at 12:15 AM Zhao Zhili wrote: > > h->max_packet_size is being reset in the following code. > --- > libavformat/udp.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/libavformat/udp.c b/libavformat/udp.c > index 30d8041433..96d8b8ee11 100644 > --- a/libavformat/udp.

Re: [FFmpeg-devel] [PATCH] avformat/http: fix memleak

2020-09-23 Thread myp...@gmail.com
On Thu, Sep 24, 2020 at 2:10 AM Zhao Zhili wrote: > > Ping for review. > > > On Aug 24, 2020, at 12:58 AM, Zhao Zhili wrote: > > > > --- > > libavformat/http.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/libavformat/http.c b/libavformat/http.c > > index 6c39da1

Re: [FFmpeg-devel] [PATCH v5 2/2] libavformat/webp: add WebP demuxer

2020-09-23 Thread Zlomek, Josef
On Sat, Sep 12, 2020 at 3:01 PM Lynne wrote: > On 11/09/2020 08:36, Josef Zlomek wrote: > > Adds the demuxer of animated WebP files. > > It supports non-animated, animated, truncated, and concatenated files. > > Reading from a pipe (and other non-seekable inputs) is also supported. > > > > The We