Re: [FFmpeg-devel] [PATCH 1/2] avfilter/all: propagate errors of functions from avfilter/formats

Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > -ff_channel_layouts_ref(layouts, >out_channel_layouts); > +if (!layouts) > +return AVERROR(ENOMEM); > +if ((ret = ff_channel_layouts_ref(layouts, > >out_channel_layouts)) < 0) { > +

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/all: propagate errors of functions from avfilter/formats

On Tue, Oct 06, 2015 at 04:40:36PM +0200, Nicolas George wrote: > Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > > -ff_channel_layouts_ref(layouts, >out_channel_layouts); > > +if (!layouts) > > +return AVERROR(ENOMEM); > > +if ((ret =

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/all: propagate errors of functions from avfilter/formats

Hi, On Tue, Oct 6, 2015 at 9:25 AM, Nicolas George wrote: > Le quintidi 15 vendémiaire, an CCXXIV, Ronald S. Bultje a écrit : > > > // inlink supports any channel layout > > > layouts = ff_all_channel_counts(); > > > -ff_channel_layouts_ref(layouts,

Re: [FFmpeg-devel] modify ffplay for dumpstream in rtsp?

On Sun, 4 Oct 2015 18:37:31 +0200 Nicolas George wrote: > Le duodi 12 vendémiaire, an CCXXIV, compn a écrit : > > he wants to play and dump at the same time. like vlc can. > > > > should this functionality be added to ffmpeg or ffplay? > > ffmpeg can already display video on

Re: [FFmpeg-devel] modify ffplay for dumpstream in rtsp?

On Tue, 6 Oct 2015 10:50:30 -0400 compn wrote: > On Sun, 4 Oct 2015 18:37:31 +0200 > Nicolas George wrote: > > > Le duodi 12 vendémiaire, an CCXXIV, compn a écrit : > > > he wants to play and dump at the same time. like vlc can. > > > > > > should this

[FFmpeg-devel] [PATCH] vp9: add x86 simd (sse2/ssse3) for iadst4 10bpp functions.

--- libavcodec/x86/vp9dsp_init.h| 6 ++ libavcodec/x86/vp9dsp_init_16bpp_template.c | 21 +- libavcodec/x86/vp9itxfm.asm | 58 libavcodec/x86/vp9itxfm_16bpp.asm | 100 ++--

Re: [FFmpeg-devel] modify ffplay for dumpstream in rtsp?

On Tue, Oct 06, 2015 at 10:50:30 -0400, compn wrote: > this works for me: > > ffmpeg -i http://samples.ffmpeg.org/V-codecs/sn40sample.avi -i > http://samples.ffmpeg.org/V-codecs/sn40sample.avi -f sdl out.mp4 But it ignores the second input file (what was that good for?) > is there a way to

[FFmpeg-devel] [PATCH] vp9: add 10bpp simd (mmxext/ssse3) for idct_idct_4x4.

--- libavcodec/x86/constants.c | 2 + libavcodec/x86/constants.h | 1 + libavcodec/x86/vp9dsp_init_16bpp_template.c | 12 libavcodec/x86/vp9itxfm.asm | 50 +- libavcodec/x86/vp9itxfm_16bpp.asm | 100

Re: [FFmpeg-devel] [PATCH] add CONTRIBUTING.md

On date Monday 2015-10-05 15:26:46 -0800, Lou Logan encoded: > For the Github users to ignore. > > Signed-off-by: Lou Logan > --- > CONTRIBUTING.md | 36 > 1 file changed, 36 insertions(+) > create mode 100644 CONTRIBUTING.md > > diff --git

[FFmpeg-devel] [PATCH] doc/developer: use https instead of http

Change to https for FFmpeg websites. Signed-off-by: Ganesh Ajjanagadde --- doc/developer.texi | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index 4d0fc9f..31952d8 100644 --- a/doc/developer.texi +++

Re: [FFmpeg-devel] [PATCH] vp9: add x86 simd (sse2/ssse3) for iadst4 10bpp functions.

On Tue, Oct 6, 2015 at 5:42 PM, Ronald S. Bultje wrote: > +cglobal vp9_%1_%3_4x4_add_10, 3, 3, 0, dst, stride, block, eob [...] > +movam0, [blockq+0*16+0] > +movam4, [blockq+0*16+8] > +movam1, [blockq+1*16+0] > +

[FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

Since AVFrame.extended_data is apparently not padded, simd functions could in some cases overread, so make the decoder use a temp buffer unconditionally. Signed-off-by: James Almer --- libavcodec/alac.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-)

Re: [FFmpeg-devel] [PATCH] vp9: 16bpp tm/dc/h/v intra pred simd (mostly sse2) functions.

On 6 October 2015 at 07:51, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 5, 2015 at 7:39 AM, Ronald S. Bultje > wrote: > > > Hi, > > > > On Mon, Oct 5, 2015 at 5:54 AM, Henrik Gramner > wrote: > > > >> On Mon, Oct 5, 2015 at 10:55

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

On 10/6/15, James Almer wrote: > Since AVFrame.extended_data is apparently not padded, simd functions > could in some cases overread, so make the decoder use a temp buffer > unconditionally. > > Signed-off-by: James Almer > --- > libavcodec/alac.c | 18

[FFmpeg-devel] [PATCH] vp9: initial attempt at a idct_idct_4x4 12bpp x86 simd (sse2) impl.

The trouble with this function is that intermediates overflow 31+sign bits, so I've added some helpers (that will also be used in 10/12bpp 8x8, 16x16 and 32x32) to make that easier, basically emulating a half- assed pmaddqd using 2xpmaddwd. It's currently sse2-only, if anyone sees potential in

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

On 10/6/2015 4:40 PM, Paul B Mahol wrote: > On 10/6/15, James Almer wrote: >> Since AVFrame.extended_data is apparently not padded, simd functions >> could in some cases overread, so make the decoder use a temp buffer >> unconditionally. >> >> Signed-off-by: James Almer

Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: better support for 4:4:4

On Mon, Oct 05, 2015 at 08:44:44PM +0200, Christophe Gisquet wrote: > Profiles 1256 & 1270 (currently) signal at the frame header and MB > levels the colorspace used, either RGB or YUV. While a MB-level > varying colorspace is not supported, whether it is constant can be > tracked so as to

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

On 10/6/15, James Almer wrote: > On 10/6/2015 4:40 PM, Paul B Mahol wrote: >> On 10/6/15, James Almer wrote: >>> Since AVFrame.extended_data is apparently not padded, simd functions >>> could in some cases overread, so make the decoder use a temp buffer >>>

Re: [FFmpeg-devel] [PATCH] Add space after commas in HTTP/RTSP auth header

On Tue, Oct 06, 2015 at 10:00:29AM -0400, Calvin Walton wrote: > On Thu, 2015-10-01 at 13:56 +0300, Andrey Utkin wrote: > > This fixes access to Grandstream cameras, which return 401 to ffmpeg > > otherwise. > > VLC sends Authorization: header with spaces between parameters, and > > it > > is

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: add av_warn_unused_result attributes

Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > This adds av_warn_unused_result whenever it is relevant. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/buffersrc.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) I do not

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/formats: add av_warn_unused_result to function prototypes

Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > Off topic, but I would greatly appreciate any help/suggestions for how > to get gmail to work correctly, or alternative mailbox providers for > patch sending purposes. I suspect it is related to using gmail as a webmail, and

[FFmpeg-devel] [PATCH 2/3] lavfi/af_pan: check ff_add_channel_layout() return.

Signed-off-by: Nicolas George --- libavfilter/af_pan.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libavfilter/af_pan.c b/libavfilter/af_pan.c index d116818..5062252 100644 --- a/libavfilter/af_pan.c +++ b/libavfilter/af_pan.c @@ -227,6 +227,7 @@

[FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

Fixes trac #3842 --- libavformat/isom.h | 2 + libavformat/mov.c | 245 - 2 files changed, 208 insertions(+), 39 deletions(-) diff --git a/libavformat/isom.h b/libavformat/isom.h index aee9d6e..6e921c0 100644 --- a/libavformat/isom.h +++

Re: [FFmpeg-devel] [PATCH 2/2] x86/alacdsp: add simd optimized functions

On 10/6/15, James Almer wrote: > On 10/5/2015 6:48 PM, Paul B Mahol wrote: >> On 10/4/15, James Almer wrote: >>> Signed-off-by: James Almer >>> --- >>> libavcodec/alacdsp.c | 3 + >>> libavcodec/alacdsp.h | 1 + >>>

[FFmpeg-devel] [PATCH 1/3] lavfi/af_pan: check ff_all_channel_counts() return.

Fix CID 1325680. Signed-off-by: Nicolas George --- libavfilter/af_pan.c | 2 ++ 1 file changed, 2 insertions(+) Will push this series soon unless somebody objects. diff --git a/libavfilter/af_pan.c b/libavfilter/af_pan.c index 9117cc0..d116818 100644 ---

[FFmpeg-devel] [PATCH 3/3] lavfi/af_astreamsync: check ff_all_samplerates() return.

Fix CID 1325679. Signed-off-by: Nicolas George --- libavfilter/af_astreamsync.c | 2 ++ 1 file changed, 2 insertions(+) The way the formats and layouts are set look fishy, but I do not have time to look into it right now. And this filter is not useful for users anyway. diff

[FFmpeg-devel] Issue using AVIOContext

Hi, I'm testing a program using AVIOContext to extract some frames of a video, I need to use an InputStream not a URL. I'm using the source code of avio_reading.c to test my sample videos ( https://www.ffmpeg.org/doxygen/trunk/avio__reading_8c_source.html) I tested this program with video1.wmv

[FFmpeg-devel] [PATCH] vp9: add 12bpp sse2 versions of iadst4.

--- libavcodec/x86/vp9dsp_init_16bpp_template.c | 8 +- libavcodec/x86/vp9itxfm_16bpp.asm | 119 2 files changed, 123 insertions(+), 4 deletions(-) diff --git a/libavcodec/x86/vp9dsp_init_16bpp_template.c b/libavcodec/x86/vp9dsp_init_16bpp_template.c

[FFmpeg-devel] [PATCHv2] avcodec/libx264: silence -Waddress

This patch moves the pointer validity check outside the macro, and silences the -Waddress observed with GCC 5.2. Note that this changes the error message slightly, from: "bad option..." to "Error parsing option...". Signed-off-by: Ganesh Ajjanagadde ---

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Sat, Oct 3, 2015 at 8:38 AM, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 1:32 PM, Ganesh Ajjanagadde > wrote: >> On Sat, Sep 19, 2015 at 1:00 AM, Ganesh Ajjanagadde >> wrote: >>> Commit 6dac8c8327 disabled

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

On Sat, Oct 3, 2015 at 9:14 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Sep 17, 2015 at 7:03 AM, Ganesh Ajjanagadde > wrote: > >> On Thu, Sep 17, 2015 at 6:15 AM, Hendrik Leppkes >> wrote: >> > On Thu, Sep 17, 2015 at 12:19 AM,

[FFmpeg-devel] [PATCH] RFC: Automatic bitstream filtering

This solves the problem discussed in https://ffmpeg.org/pipermail/ffmpeg-devel/2015-September/179238.html by allowing AVCodec::write_header to be delayed until after packets have been run through required bitstream filters in order to generate global extradata. It also provides a mechanism by

Re: [FFmpeg-devel] [PATCH] add CONTRIBUTING.md

On Tue, 6 Oct 2015 18:37:47 +0200, Stefano Sabatini wrote: > If we keep CONTRIBUTING.md, better to make it independent from Github > - that is I'm suggesting that we move all contributing indications to > README.md, or we keep them in a separate file CONTRIBUTING.md, as > proposed by this patch.

Re: [FFmpeg-devel] [PATCH] RFC: Automatic bitstream filtering

Hi, On Tue, Oct 6, 2015 at 9:07 PM, Rodger Combs wrote: > This solves the problem discussed in > https://ffmpeg.org/pipermail/ffmpeg-devel/2015-September/179238.html > by allowing AVCodec::write_header to be delayed until after packets have > been > run through required

[FFmpeg-devel] [PATCH 2/2] ffplay: eliminate stream_component_close forward declaration

No change in fuctionality. Signed-off-by: Marton Balint --- ffplay.c | 116 +++ 1 file changed, 57 insertions(+), 59 deletions(-) diff --git a/ffplay.c b/ffplay.c index 7f73664..4a084b4 100644 --- a/ffplay.c +++

[FFmpeg-devel] [PATCH 1/2] ffplay: close streams and AVFormatContext in the main thread

To avoid race conditions. Signed-off-by: Marton Balint --- ffplay.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/ffplay.c b/ffplay.c index 79f430d..7f73664 100644 --- a/ffplay.c +++ b/ffplay.c @@ -1132,11 +1132,24 @@ static

Re: [FFmpeg-devel] [PATCH] checkasm: add alacdsp tests

On 10/5/2015 5:08 PM, Henrik Gramner wrote: > On Mon, Oct 5, 2015 at 12:50 AM, James Almer wrote: >> Signed-off-by: James Almer >> --- >> tests/checkasm/Makefile | 1 + >> tests/checkasm/alacdsp.c | 119 >>

[FFmpeg-devel] [PATCHv2] avfilter/buffersrc: add av_warn_unused_result attributes

This adds av_warn_unused_result whenever it is relevant. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/buffersrc.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavfilter/buffersrc.h b/libavfilter/buffersrc.h index cd3d95f..847c093 100644 ---

Re: [FFmpeg-devel] [PATCH 2/2] x86/alacdsp: add simd optimized functions

On 10/6/2015 4:44 AM, Paul B Mahol wrote: > On 10/6/15, James Almer wrote: >> On 10/5/2015 6:48 PM, Paul B Mahol wrote: >>> On 10/4/15, James Almer wrote: Signed-off-by: James Almer --- libavcodec/alacdsp.c | 3

[FFmpeg-devel] [PATCH] w32pthreads: add pthread_once emulation

The emulation uses native InitOnce* APIs on Windows Vista+, and a lock-free/allocation-free approach using atomics and spinning for Windows XP. --- This is in preparation to use pthread_once for global static init functions, and eventually removing the global lock in avcodec_open2

Re: [FFmpeg-devel] [PATCH 1/3] lavfi/af_pan: check ff_all_channel_counts() return.

On Tue, Oct 6, 2015 at 4:18 AM, Nicolas George wrote: > Fix CID 1325680. > > Signed-off-by: Nicolas George > --- > libavfilter/af_pan.c | 2 ++ > 1 file changed, 2 insertions(+) > > > Will push this series soon unless somebody objects. The 1000 line diff I

[FFmpeg-devel] [PATCH] vp9: don't keep a stack pointer if we don't need it.

This saves one register in a few cases on 32bit builds with unaligned stack (e.g. MSVC), making the code slightly easier to maintain. (Can someone please test this on 32bit+msvc and confirm make fate-vp9 and tests/checkasm/checkasm still work after this patch?) ---

Re: [FFmpeg-devel] [PATCH] vp9: don't keep a stack pointer if we don't need it.

On Tue, Oct 6, 2015 at 2:38 PM, Ronald S. Bultje wrote: > This saves one register in a few cases on 32bit builds with unaligned > stack (e.g. MSVC), making the code slightly easier to maintain. > > (Can someone please test this on 32bit+msvc and confirm make fate-vp9 > and

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: add av_warn_unused_result attributes

Hi, On Tue, Oct 6, 2015 at 4:01 AM, Nicolas George wrote: > Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > > This adds av_warn_unused_result whenever it is relevant. > > > > Signed-off-by: Ganesh Ajjanagadde > > --- > >

Re: [FFmpeg-devel] [PATCH] avfilter/buffersrc: add av_warn_unused_result attributes

On Tue, Oct 6, 2015 at 4:01 AM, Nicolas George wrote: > Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> This adds av_warn_unused_result whenever it is relevant. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >>

[FFmpeg-devel] [PATCH] vp9: add 10/12bpp mmxext-optimized iwht_iwht_4x4 function.

--- libavcodec/x86/Makefile | 1 + libavcodec/x86/vp9dsp_init.c| 4 +- libavcodec/x86/vp9dsp_init.h| 15 ++-- libavcodec/x86/vp9dsp_init_16bpp_template.c | 14 +++- libavcodec/x86/vp9itxfm.asm | 16 +

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/all: propagate errors of functions from avfilter/formats

Hello, On Mon, Oct 5, 2015 at 3:43 PM, Ganesh Ajjanagadde wrote: > Hi, > > I have attached the patch. > // inlink supports any channel layout > layouts = ff_all_channel_counts(); > -ff_channel_layouts_ref(layouts, >out_channel_layouts); > +if (!layouts) > +

[FFmpeg-devel] Closing down the ABI again

Hey, Libav is talking about closing the change window and stabilizing the ABI, and I think we should follow suit. Its been a good couple of weeks since the ABI bump, and should have given everyone ample time to get their changes in. Does anyone still have any ABI-relevant changes they want to

Re: [FFmpeg-devel] [PATCH] Add space after commas in HTTP/RTSP auth header

On Thu, 2015-10-01 at 13:56 +0300, Andrey Utkin wrote: > This fixes access to Grandstream cameras, which return 401 to ffmpeg > otherwise. > VLC sends Authorization: header with spaces between parameters, and > it > is known to work with Grandstream devices and broad range of other > HTTP > and

Re: [FFmpeg-devel] [PATCH] README: replace http with https

On Mon, Oct 05, 2015 at 11:39:08PM -0400, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > README.md | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/3] lavfi/af_pan: check ff_all_channel_counts() return.

Le quintidi 15 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > The 1000 line diff I posted should take care of this and all other > such unchecked stuff in query_formats across libavfilter :). I don't > mind either way, though if this is applied, I will perhaps need to > rebase my stuff to

Re: [FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

On Tue, Oct 06, 2015 at 03:50:23AM -0500, Rodger Combs wrote: > Fixes trac #3842 > --- > libavformat/isom.h | 2 + > libavformat/mov.c | 245 > - > 2 files changed, 208 insertions(+), 39 deletions(-) i think this should be applied unless

Re: [FFmpeg-devel] [PATCH] add CONTRIBUTING.md

On Tue, Oct 6, 2015, at 07:07 AM, Stefano Sabatini wrote: > > I wonder if adding the contents to README.md is not a better choice > > Note also since we're writing about github, that the README.md will > have a better visibility there, since it is shown on the project main > page: >

Re: [FFmpeg-devel] [PATCH] add CONTRIBUTING.md

On date Tuesday 2015-10-06 08:20:11 -0800, Lou Logan encoded: > On Tue, Oct 6, 2015, at 07:07 AM, Stefano Sabatini wrote: > > > > I wonder if adding the contents to README.md is not a better choice > > > > Note also since we're writing about github, that the README.md will > > have a better

Re: [FFmpeg-devel] Closing down the ABI again

On Tue, Oct 6, 2015 at 6:48 PM, Ganesh Ajjanagadde wrote: > On Tue, Oct 6, 2015 at 9:42 AM, Hendrik Leppkes wrote: >> Hey, >> >> Libav is talking about closing the change window and stabilizing the >> ABI, and I think we should follow suit. Its been a good

Re: [FFmpeg-devel] modify ffplay for dumpstream in rtsp?

On date Tuesday 2015-10-06 11:27:31 -0400, compn encoded: > On Tue, 6 Oct 2015 10:50:30 -0400 > compn wrote: > > > On Sun, 4 Oct 2015 18:37:31 +0200 > > Nicolas George wrote: > > > > > Le duodi 12 vendémiaire, an CCXXIV, compn a écrit : > > > > he wants to

Re: [FFmpeg-devel] [PATCH] vp9: add 10/12bpp mmxext-optimized iwht_iwht_4x4 function.

On Tue, Oct 6, 2015 at 3:43 PM, Ronald S. Bultje wrote: > --- > libavcodec/x86/Makefile | 1 + > libavcodec/x86/vp9dsp_init.c| 4 +- > libavcodec/x86/vp9dsp_init.h| 15 ++-- > libavcodec/x86/vp9dsp_init_16bpp_template.c

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/all: propagate errors of functions from avfilter/formats

On Tue, Oct 6, 2015 at 10:40 AM, Nicolas George wrote: > Le quartidi 14 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> -ff_channel_layouts_ref(layouts, >out_channel_layouts); >> +if (!layouts) >> +return AVERROR(ENOMEM); >> +if ((ret =

Re: [FFmpeg-devel] Closing down the ABI again

On Tue, Oct 6, 2015 at 9:42 AM, Hendrik Leppkes wrote: > Hey, > > Libav is talking about closing the change window and stabilizing the > ABI, and I think we should follow suit. Its been a good couple of > weeks since the ABI bump, and should have given everyone ample time to

Re: [FFmpeg-devel] [PATCH] videotoolbox: allow to enable the async decoding.

On Thu, Sep 24, 2015 at 11:45:42AM +0200, Clément Bœsch wrote: > On Sun, Aug 09, 2015 at 01:11:44PM +0200, Sebastien Zwickert wrote: > > This patch allows to use the Videotoolbox API in asynchonous mode. > > Note that when using async decoding the user is responsible for > > releasing the async