[FFmpeg-devel] [PATCH] -- Convert decklink input module to use codecpar

2016-05-07 Thread Felt, Patrick
This patch converts decklink input to use codecpar. There are still a couple of deprecated calls that I’m not sure what to do with. They are both related to some logic around AVCodecContext.coded_frame. I couldn’t find anywhere that really documented what that was, or where it was intended t

Re: [FFmpeg-devel] [PATCH] fate: Remove duplicate wmv8_x8intra.wmv test

2016-05-07 Thread Michael Niedermayer
On Sun, May 08, 2016 at 12:54:10AM +0100, Derek Buitenhuis wrote: > On 5/8/2016 12:42 AM, Michael Niedermayer wrote: > > Also temporary enable the test so we get updated fate failure statistics > > > > Note, this does not work on all platforms, it fails on MIPS > > and ml archives indicate it fail

Re: [FFmpeg-devel] [PATCH] vc2enc_dwt: use 32 bit coefficients by default

2016-05-07 Thread Michael Niedermayer
On Sat, May 07, 2016 at 06:12:43PM +0100, Rostislav Pehlivanov wrote: > The problem is that with particularly complex images and especially at > high bit depths and 5-level transforms the coefficients would overflow, > causing huge artifacts to appear. This was discovered thanks to the fate > tests

Re: [FFmpeg-devel] [PATCH] fate: Remove duplicate wmv8_x8intra.wmv test

2016-05-07 Thread Derek Buitenhuis
On 5/8/2016 12:42 AM, Michael Niedermayer wrote: > Also temporary enable the test so we get updated fate failure statistics > > Note, this does not work on all platforms, it fails on MIPS > and ml archives indicate it failed on x86 openbsd with some compilers as well > > Signed-off-by: Michael Ni

[FFmpeg-devel] [PATCH] fate: Remove duplicate wmv8_x8intra.wmv test

2016-05-07 Thread Michael Niedermayer
Also temporary enable the test so we get updated fate failure statistics Note, this does not work on all platforms, it fails on MIPS and ml archives indicate it failed on x86 openbsd with some compilers as well Signed-off-by: Michael Niedermayer --- tests/fate/microsoft.mak|8 +- tests/

Re: [FFmpeg-devel] [PATCH][libavfilter] codecview: improved options

2016-05-07 Thread Davinder Singh
changed default value of `frames` to zero, if `frames` option is not set, it ignore and check only direction, this way all picture types are considered if we are drawing MVs for ME, since there are more frame types available. On Sun, May 8, 2016 at 3:02 AM Moritz Barsnick wrote: > On Sat, May 0

Re: [FFmpeg-devel] [PATCH][libavfilter] codecview: improved options

2016-05-07 Thread Moritz Barsnick
On Sat, May 07, 2016 at 19:48:31 +, Davinder Singh wrote: > -{ "frames", "set frame types to display MVs of", OFFSET(frames), > AV_OPT_TYPE_FLAGS, {.i64=3}, 0, INT_MAX, FLAGS, "frames" }, > +{ "frames", "set frame types to display MVs of", OFFSET(frames), > AV_OPT_TYPE_FLAGS, {.i64=7

Re: [FFmpeg-devel] [PATCH 3/4] x86: lossless audio: SSE4 madd 32bits

2016-05-07 Thread Michael Niedermayer
On Sat, May 07, 2016 at 05:58:21PM +0200, Paul B Mahol wrote: > On 5/1/16, Christophe Gisquet wrote: > > The unique user so far is wmalossless 24bits. The few samples tested show an > > order of 8, so more unrolling or an avx2 version do not make sense. > > > > Timings: 68 -> 49 cycles > > --- > >

Re: [FFmpeg-devel] [PATCH] vc2enc_dwt: use 32 bit coefficients by default

2016-05-07 Thread Rostislav Pehlivanov
On 7 May 2016 at 18:55, Christophe Gisquet wrote: > 2016-05-07 19:12 GMT+02:00 Rostislav Pehlivanov : > > There is a slight performance drop associated with this change, making > > the encoder slower by 1.15 times, however this is necessary in order to > > avoid undefined behavior and overflows

[FFmpeg-devel] [PATCH][libavfilter] codecview: improved options

2016-05-07 Thread Davinder Singh
separated motion vector types (forward or backward) from frame picture types as MVs are associated with picture types only in video coding. option `mv` can have two values: forward predicted or backward predicted. option `frames` can have three values: p-frames, i-frames and b-frames. ex: only f

Re: [FFmpeg-devel] [PATCH] vc2enc_dwt: use 32 bit coefficients by default

2016-05-07 Thread Christophe Gisquet
2016-05-07 19:12 GMT+02:00 Rostislav Pehlivanov : > The problem is that with particularly complex images and especially at > high bit depths and 5-level transforms the coefficients would overflow I guess it also depends on the transform type, so that counts also for the last comment. > causing hu

[FFmpeg-devel] [PATCH] vc2enc_dwt: use 32 bit coefficients by default

2016-05-07 Thread Rostislav Pehlivanov
The problem is that with particularly complex images and especially at high bit depths and 5-level transforms the coefficients would overflow, causing huge artifacts to appear. This was discovered thanks to the fate tests, which will have to be redone as this fixes a multitude of problems and incre

Re: [FFmpeg-devel] [PATCH 3/4] x86: lossless audio: SSE4 madd 32bits

2016-05-07 Thread Paul B Mahol
On 5/1/16, Christophe Gisquet wrote: > The unique user so far is wmalossless 24bits. The few samples tested show an > order of 8, so more unrolling or an avx2 version do not make sense. > > Timings: 68 -> 49 cycles > --- > libavcodec/x86/lossless_audiodsp.asm| 33 > +++

Re: [FFmpeg-devel] [PATCH] ffplay: force setting alsa buffer size

2016-05-07 Thread Marton Balint
On Sun, 1 May 2016, Reimar Döffinger wrote: On Sat, Apr 30, 2016 at 10:17:33PM +0200, Marton Balint wrote: Signed-off-by: Marton Balint --- ffplay.c | 7 +++ 1 file changed, 7 insertions(+) I have a nagging feeling someone with better knowledge of ALSA and how we handle it might find a

Re: [FFmpeg-devel] [PATCH 2/2] avformat/tee: Use ref instead copy in write_packet

2016-05-07 Thread Marton Balint
On Thu, 5 May 2016, Nicolas George wrote: Le quartidi 14 floréal, an CCXXIV, sebechlebsky...@gmail.com a écrit : From: Jan Sebechlebsky Replace av_copy_packet and deprecated av_dup_packet by creating reference using av_packet_ref. Signed-off-by: Jan Sebechlebsky --- This should be effecti

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tee: Fix TeeSlave.bsfs pointer array size

2016-05-07 Thread Marton Balint
On Thu, 5 May 2016, Nicolas George wrote: Le quintidi 15 floréal, an CCXXIV, sebechlebsky...@gmail.com a écrit : From: Jan Sebechlebsky TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext, so element size should be really size of a pointer, not size of TeeSlave structure. Signed-

Re: [FFmpeg-devel] [PATCH] avformat/riff: assign g721 and g723codec tags to g726 decoder

2016-05-07 Thread compn
On Sun, 01 May 2016 00:09:46 +0200 Piotr Bandurski wrote: > > > On Wed, 27 Apr 2016 20:39:34 +0200 > > Piotr Bandurski wrote: > > > > > Subject: [PATCH] avformat/riff: assign g721 and g723 codec tags to > > > g726 decoder > > > > > + { AV_CODEC_ID_ADPCM_G726, 0x0014 }, > > > + { AV_CODEC_ID_AD