Re: [FFmpeg-devel] [PATCH] lavu/libm: add isfinite fallback

2016-01-14 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 9:09 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Jan 13, 2016 at 8:09 PM, Ganesh Ajjanagadde > wrote: >> >> On Wed, Jan 13, 2016 at 8:07 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Wed, Jan 13,

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread James Almer
On 1/14/2016 11:12 AM, Ganesh Ajjanagadde wrote: > On Thu, Jan 14, 2016 at 5:02 AM, Henrik Gramner wrote: >> Use the x86inc syntax for FMA instructions (basically FMA4 syntax that >> gets assembled as FMA3) since normal FMA3 opcodes are horrible to >> read, nobody ever

[FFmpeg-devel] [PATCH 1/7] avcodec/dca: remove old DCA decoder

2016-01-14 Thread foo86
--- configure|1 - libavcodec/Makefile |3 - libavcodec/aarch64/Makefile |3 - libavcodec/aarch64/dcadsp_init.c | 15 +- libavcodec/aarch64/dcadsp_neon.S | 109 -- libavcodec/allcodecs.c |2 +- libavcodec/arm/Makefile

[FFmpeg-devel] [PATCH 2/7] avcodec/dca: add REV1AUX sync word

2016-01-14 Thread foo86
--- libavcodec/dca_syncwords.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/dca_syncwords.h b/libavcodec/dca_syncwords.h index 6981cb8..4d2cd5f 100644 --- a/libavcodec/dca_syncwords.h +++ b/libavcodec/dca_syncwords.h @@ -31,5 +31,6 @@ #defineDCA_SYNCWORD_XLL

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 11:42:47AM -0500, Ronald S. Bultje wrote: > Hi, > > On Thu, Jan 14, 2016 at 11:34 AM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > > > On Thu, Jan 14, 2016 at 10:09:13AM -0500, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Wed, Jan 13, 2016 at 9:50 PM,

[FFmpeg-devel] [PATCH] avformat/riff: add YUYV FourCC (Drastic YUYV)

2016-01-14 Thread Piotr Bandurski
0001-avformat-riff-add-YUYV-FourCC-Drastic-YUYV.diff Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Ronald S. Bultje
Hi, On Thu, Jan 14, 2016 at 12:06 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Thu, Jan 14, 2016 at 11:42:47AM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Thu, Jan 14, 2016 at 11:34 AM, Michael Niedermayer < > > mich...@niedermayer.cc> wrote: > > > > > On Thu, Jan 14, 2016

[FFmpeg-devel] [PATCH] avformat/riff: add C210 FourCC (Canopus C210)

2016-01-14 Thread Piotr Bandurski
0001-avformat-riff-add-C210-FourCC-Canopus-C210.diff Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 08:58:20AM +0100, Christophe Gisquet wrote: > > -#define YUVRGB_TABLE_HEADROOM 256 > > +#define YUVRGB_TABLE_HEADROOM 512 > [...] > > -const int yoffs = fullRange ? 384 : 326; > > +const int yoffs = fullRange ? 896 : 838; > > I think it's time to use that macro

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Christophe Gisquet
Hi, 2016-01-14 9:41 GMT+01:00 Michael Niedermayer : > there are 2 seperate tables, the headroom in them also differs > curently, also the exact relation between input values and headroom > needed in the 2nd table depends on the yuv-rgb coefficients > > i can replace the

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Paul B Mahol
On 1/14/16, Christophe Gisquet wrote: > Hi, > > 2016-01-14 9:41 GMT+01:00 Michael Niedermayer : >> there are 2 seperate tables, the headroom in them also differs >> curently, also the exact relation between input values and headroom >> needed

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Henrik Gramner
Use the x86inc syntax for FMA instructions (basically FMA4 syntax that gets assembled as FMA3) since normal FMA3 opcodes are horrible to read, nobody ever remembers the ordering of operands. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH]lavc/x264: Improve level setting

2016-01-14 Thread Michael Niedermayer
On Tue, Jan 12, 2016 at 02:39:35PM +0100, Carl Eugen Hoyos wrote: > On Tuesday 12 January 2016 02:31:48 pm Michael Niedermayer wrote: > > > > > +level_id = atoi(x4->level); > > > > > > > > this should check that teres no "tail" after the integer > > > > > > For which command line will

Re: [FFmpeg-devel] [PATCH] lavc/mlpdec: report presence of Atmos substreams as a profile

2016-01-14 Thread Hendrik Leppkes
On Thu, Jan 14, 2016 at 2:32 AM, Rodger Combs wrote: > --- > libavcodec/avcodec.h| 2 ++ > libavcodec/codec_desc.c | 1 + > libavcodec/mlp_parser.c | 5 + > libavcodec/mlpdec.c | 2 ++ > libavcodec/profiles.c | 6 ++ > libavcodec/profiles.h | 1 + > 6

Re: [FFmpeg-devel] [PATCH] lavu: check for overflow in av_clip_intp2_c

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 01:21:38AM +0100, Andreas Cadhalpun wrote: > On 13.01.2016 20:06, Michael Niedermayer wrote: > > On Wed, Jan 13, 2016 at 12:52:21AM +0100, Andreas Cadhalpun wrote: > >> --- > >> libavutil/common.h | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 10:09:02AM +, Paul B Mahol wrote: > On 1/14/16, Christophe Gisquet wrote: > > Hi, > > > > 2016-01-14 9:41 GMT+01:00 Michael Niedermayer : > >> there are 2 seperate tables, the headroom in them also differs > >>

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 5:02 AM, Henrik Gramner wrote: > Use the x86inc syntax for FMA instructions (basically FMA4 syntax that > gets assembled as FMA3) since normal FMA3 opcodes are horrible to > read, nobody ever remembers the ordering of operands. 1. It is very easy to

Re: [FFmpeg-devel] [RFC] avcodec: Add native DCA decoder based on libdcadec.

2016-01-14 Thread foo86
On Thu, Jan 14, 2016 at 12:28:32AM +0100, Andreas Cadhalpun wrote: > On 13.01.2016 19:18, foo86 wrote: > > Hmm, core sample rate and number of samples are already checked to be > > compatible with XLL at this point. There must be some bug in upsampling > > logic if these asserts are triggering,

[FFmpeg-devel] [PATCH 6/7] avcodec/synth_filter: add more filters

2016-01-14 Thread foo86
--- libavcodec/synth_filter.c | 116 +- libavcodec/synth_filter.h | 13 ++ 2 files changed, 128 insertions(+), 1 deletion(-) diff --git a/libavcodec/synth_filter.c b/libavcodec/synth_filter.c index 9eab9c5..1c5dab5 100644 ---

[FFmpeg-devel] [PATCH 5/7] avcodec/dca: add math helpers and fixed point DCT

2016-01-14 Thread foo86
--- libavcodec/dcadct.c | 362 +++ libavcodec/dcadct.h | 32 + libavcodec/dcamath.h | 64 + 3 files changed, 458 insertions(+) create mode 100644 libavcodec/dcadct.c create mode 100644 libavcodec/dcadct.h create mode 100644

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Henrik Gramner
On Thu, Jan 14, 2016 at 5:26 PM, Ganesh Ajjanagadde wrote: > readability still no. " dst, mult1, mult2, add" is significantly more readable than " src1, src2, src3" where you need to mentally parse which source operand corresponds to which mathematical operator depending on the

[FFmpeg-devel] [PATCH 0/7] Replace native DCA decoder with libdcadec based one

2016-01-14 Thread foo86
Full diff output has been omitted for deleted files. If git complains about applying the first patch, this can be also pulled from dca-replace branch at [1]. [1]: https://github.com/foo86/FFmpeg.git foo86 (7): avcodec/dca: remove old DCA decoder avcodec/dca: add REV1AUX sync word

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 05:34:25PM +0100, Michael Niedermayer wrote: > On Thu, Jan 14, 2016 at 10:09:13AM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Wed, Jan 13, 2016 at 9:50 PM, Michael Niedermayer > > wrote: > > > > > From: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 10:09:13AM -0500, Ronald S. Bultje wrote: > Hi, > > On Wed, Jan 13, 2016 at 9:50 PM, Michael Niedermayer > wrote: > > > From: Michael Niedermayer > > > > This makes SWS more robust > > Fixes: > >

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Ronald S. Bultje
Hi, On Thu, Jan 14, 2016 at 11:34 AM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Thu, Jan 14, 2016 at 10:09:13AM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Wed, Jan 13, 2016 at 9:50 PM, Michael Niedermayer > > wrote: > > > > > From: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] avcodec: add a native BBC Dirac VC-2 HQ encoder

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 07:00:48PM +, Rostislav Pehlivanov wrote: [...] > +int diracenc_init_transforms(DiracTransforms *s, int p_width, int p_height) [..,] > +void diracenc_deinit_transforms(DiracTransforms *s) either needs to be static or needs prefix [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Marton Balint
On Thu, 14 Jan 2016, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 5:39 PM, Vittorio Gambaletta (VittGam) wrote: [..] I see no harm, but a new option should have a justification in the commit message, i.e why do you want it, or what is your use case? I find

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 6:54 PM, Henrik Gramner wrote: > On Thu, Jan 14, 2016 at 11:47 PM, Ganesh Ajjanagadde wrote: >> BTW, this is why I personally don't like the macro: >> so I was moving along, replacing one after the other, till I came to this >> line

Re: [FFmpeg-devel] [PATCH] build: make out-of-tree builds bit-identical to in-tree builds

2016-01-14 Thread Andreas Cadhalpun
On 28.12.2015 22:12, Andreas Cadhalpun wrote: > Previously the full source path was embedded inconsistently in the debug > information between in-tree/out-of-tree builds. > > The 'vpath %.inc' becomes necessary for finding > libavfilter/all_channel_layouts.inc in out-of-tree builds. > > The full

[FFmpeg-devel] [PATCH] avcodec/h264: mmxext 4:2:2 chroma deblock/loop filter

2016-01-14 Thread James Darnley
2.6 times faster --- I have one question now. Should I make the function name match the assembly existing deblock/loop filter functions? I took the current name from the C (as I was originally trying to use a gather instruction but that didn't offer any benefit). ---

Re: [FFmpeg-devel] [PATCH] avcodec/h264: mmxext 4:2:2 chroma deblock/loop filter

2016-01-14 Thread Ronald S. Bultje
Hi, On Thu, Jan 14, 2016 at 9:55 PM, James Almer wrote: > On 1/14/2016 11:05 PM, James Darnley wrote: > > 2.6 times faster > > --- > > I have one question now. Should I make the function name match the > assembly > > existing deblock/loop filter functions? I took the

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 12:30:32PM -0500, Ronald S. Bultje wrote: > Hi, > > On Thu, Jan 14, 2016 at 12:06 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > > > On Thu, Jan 14, 2016 at 11:42:47AM -0500, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Thu, Jan 14, 2016 at 11:34 AM,

[FFmpeg-devel] [PATCH] define AVPixelFormat aliases as enumerators instead of macros

2016-01-14 Thread Richard Smith
libavutil/pixfmt.h defines a collection of endian-specific pixel formats as macros. These macro names can cause conflicts with external projects that use those identifiers for their own purposes. Here's a patch to define these aliases as enumerators instead of macros, please consider merging:

Re: [FFmpeg-devel] [PATCH] lavf/mov: Confine 0x00000000 to raw/twos fourcc mapping to version 0 sample descriptions

2016-01-14 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 12:32:20AM +0100, Mats Peterson wrote: > Confine the 0x to 'raw '/'twos' fourcc mapping to old > version 0 sound sample descriptions, since they are the only valid > sample descriptions for this type of mapping. > > Mats > mov.c |2 +- > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] avcodec/h264: mmxext 4:2:2 chroma deblock/loop filter

2016-01-14 Thread James Almer
On 1/14/2016 11:05 PM, James Darnley wrote: > 2.6 times faster > --- > I have one question now. Should I make the function name match the assembly > existing deblock/loop filter functions? I took the current name from the C > (as > I was originally trying to use a gather instruction but that

Re: [FFmpeg-devel] [PATCH] avcodec: add a native BBC Dirac VC-2 HQ encoder

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 2:00 PM, Rostislav Pehlivanov wrote: [...] > > Signed-off-by: Rostislav Pehlivanov [...] > + > +#include "diracenc_transforms.h" > + > +#define COEF_LUT_TAB 1024 > + > +typedef struct SubBand { > +int stride; > +int width;

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Vittorio Gambaletta (VittGam)
On 15/01/2016 00:09:37 CET, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 6:05 PM, Marton Balint wrote: [..] I think it would make sense to change ffplay so that only the right mouse button would seek. This way we can avoid the issue. Anybody against this? Kind of, it is

Re: [FFmpeg-devel] [PATCH] lavc/aacenc: use isfinite to simplify isnan/isinf logic

2016-01-14 Thread Claudio Freire
On Thu, Jan 14, 2016 at 7:57 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/aacenc.c | 17 + > 1 file changed, 9 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/aacenc.c

Re: [FFmpeg-devel] [PATCH] lavu: check for overflow in av_clip_intp2_c

2016-01-14 Thread Andreas Cadhalpun
On 14.01.2016 10:24, Michael Niedermayer wrote: > On Thu, Jan 14, 2016 at 01:21:38AM +0100, Andreas Cadhalpun wrote: >> On 13.01.2016 20:06, Michael Niedermayer wrote: >>> On Wed, Jan 13, 2016 at 12:52:21AM +0100, Andreas Cadhalpun wrote: --- libavutil/common.h | 2 +- 1 file

Re: [FFmpeg-devel] [PATCHv2] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 6:42 PM, James Almer wrote: > On 1/14/2016 7:46 PM, Ganesh Ajjanagadde wrote: >> This improves accuracy (very slightly) and speed for processors having >> fma3. >> >> Sample benchmark (fate flac-16-lpc-cholesky, Haswell): >> old: >> 5993610 decicycles in

Re: [FFmpeg-devel] [PATCH] lavf: add automatic bitstream filtering

2016-01-14 Thread Rodger Combs
The init functions are required because avpriv_set_pts_info is called in write_header. Since this can affect operations that take place before write_header is called when delayed, it needs to be done earlier. This is probably also true of any other change to the AVFormatContext or AVStreams

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 6:05 PM, Marton Balint wrote: > > On Thu, 14 Jan 2016, Ganesh Ajjanagadde wrote: > >> On Thu, Jan 14, 2016 at 5:39 PM, Vittorio Gambaletta (VittGam) >> wrote: > > > [..] > I see no harm, but a new option should have a

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Henrik Gramner
On Thu, Jan 14, 2016 at 11:47 PM, Ganesh Ajjanagadde wrote: > BTW, this is why I personally don't like the macro: > so I was moving along, replacing one after the other, till I came to this line > vfmadd213pd ymm1, ymm5, COVAR(iq ,1) > I naturally replace by > fmaddpd

[FFmpeg-devel] [PATCHv3] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
This improves accuracy (very slightly) and speed for processors having fma3. Sample benchmark (fate flac-16-lpc-cholesky, Haswell): old: 5993610 decicycles in ff_lpc_calc_coefs, 64 runs, 0 skips 5951528 decicycles in ff_lpc_calc_coefs, 128 runs, 0 skips new: 5252410 decicycles

Re: [FFmpeg-devel] [PATCH 0/7] Replace native DCA decoder with libdcadec based one

2016-01-14 Thread Andreas Cadhalpun
On 14.01.2016 17:25, foo86 wrote: > Full diff output has been omitted for deleted files. If git complains about > applying the first patch, this can be also pulled from dca-replace branch at > [1]. I'd prefer if you would post full patches here, i.e. including deleted files. That aside, the new

[FFmpeg-devel] [PATCH] lavc/aacenc: use isfinite to simplify isnan/isinf logic

2016-01-14 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavcodec/aacenc.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/libavcodec/aacenc.c b/libavcodec/aacenc.c index 9a7d3a8..2e0db7d 100644 --- a/libavcodec/aacenc.c +++ b/libavcodec/aacenc.c @@

Re: [FFmpeg-devel] [PATCHv2] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 6:54 PM, Ganesh Ajjanagadde wrote: > On Thu, Jan 14, 2016 at 6:42 PM, James Almer wrote: >> On 1/14/2016 7:46 PM, Ganesh Ajjanagadde wrote: >>> This improves accuracy (very slightly) and speed for processors having >>> fma3. >>> >>>

[FFmpeg-devel] [PATCH] lavf: add automatic bitstream filtering

2016-01-14 Thread Moritz Barsnick
libavformat/movenc.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) libavformat/flvenc.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) In the spirit of commits 1b5bd4051d1e394f0429cbf6cf6c137f55857478 and

[FFmpeg-devel] [PATCH] lavf/movenc: add automatic bitstream filtering

2016-01-14 Thread Moritz Barsnick
Signed-off-by: Moritz Barsnick --- libavformat/movenc.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 1aff5c5..608bf69 100644 --- a/libavformat/movenc.c +++

[FFmpeg-devel] [PATCH] lavf/flvenc: add automatic bitstream filtering

2016-01-14 Thread Moritz Barsnick
Signed-off-by: Moritz Barsnick --- libavformat/flvenc.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c index 3925768..8d10ef8 100644 --- a/libavformat/flvenc.c +++ b/libavformat/flvenc.c @@

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 5:39 PM, Vittorio Gambaletta (VittGam) wrote: > Hi, > > > On 14/01/2016 23:15:16 CET, Ganesh Ajjanagadde wrote: >> >> On Thu, Jan 14, 2016 at 3:56 PM, Vittorio Gambaletta (VittGam) >> wrote: >>> >>> --- >>> ffplay.c | 10

Re: [FFmpeg-devel] [PATCH] avcodec: add a native BBC Dirac VC-2 HQ encoder

2016-01-14 Thread Kieran Kunhya
> +static av_always_inline void coeff_quantize_encode(PutBitContext *pb, qcoef > coeff, > + int qfactor, int qoffset) > +{ > +uint16_t acoef; > +int sign = coeff < 0; FFSIGN > +coeff -= sign; > +coeff ^= -sign; > +coeff <<=

Re: [FFmpeg-devel] [PATCH 4/5] asfdec_o: break if EOF is reached after asf_read_packet_header

2016-01-14 Thread Andreas Cadhalpun
On 06.01.2016 19:56, Andreas Cadhalpun wrote: > asf_read_payload can unset eof_reached, so check it also before calling > that function. > > This fixes infinite loops. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/asfdec_o.c | 2 ++ > 1 file

Re: [FFmpeg-devel] [PATCHv2] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 7:23 PM, James Almer wrote: > On 1/14/2016 9:06 PM, Ganesh Ajjanagadde wrote: >> On Thu, Jan 14, 2016 at 6:54 PM, Ganesh Ajjanagadde wrote: >>> On Thu, Jan 14, 2016 at 6:42 PM, James Almer wrote: On 1/14/2016

[FFmpeg-devel] [PATCH] avcodec: add a native BBC Dirac VC-2 HQ encoder

2016-01-14 Thread Rostislav Pehlivanov
This commit adds a new encoder capable of creating BBC Dirac VC-2 HQ profile files. Dirac is a wavelet based codec created by the BBC a little more than 10 years ago. Since then, wavelets have mostly gone out of style as they did not provide adequate encoding gains at lower bitrates. Dirac was a

Re: [FFmpeg-devel] [PATCH] avformat/riff: add C210 FourCC (Canopus C210)

2016-01-14 Thread Paul B Mahol
On 1/14/16, Piotr Bandurski wrote: > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > Sample? ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] avformat/riff: add YUYV FourCC (Drastic YUYV)

2016-01-14 Thread Paul B Mahol
On 1/14/16, Piotr Bandurski wrote: > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > Sample? ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] avformat/riff: add C210 FourCC (Canopus C210)

2016-01-14 Thread compn
On Thu, 14 Jan 2016 18:45:27 +0100 Piotr Bandurski wrote: > From 1d6d02aca1d02c237af9f69203cc47844fc66d14 Mon Sep 17 00:00:00 2001 > From: Piotr Bandurski > Date: Thu, 14 Jan 2016 17:01:00 +0100 > Subject: [PATCH] avformat/riff: add C210 FourCC (Canopus C210) >

Re: [FFmpeg-devel] [PATCHv2] mpegtsenc: Do not fail ADTS AAC muxing if the first frame is not ADTS

2016-01-14 Thread Hagen Schmidt
On 15/12/15 08:00, Hagen Schmidt wrote: Fixes ticket 279. --- libavformat/mpegtsenc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 19032f3..a0dca4a 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -1519,8

Re: [FFmpeg-devel] [PATCH 0/7] Replace native DCA decoder with libdcadec based one

2016-01-14 Thread James Almer
On 1/14/2016 9:59 PM, Andreas Cadhalpun wrote: > On 14.01.2016 17:25, foo86 wrote: >> Full diff output has been omitted for deleted files. If git complains about >> applying the first patch, this can be also pulled from dca-replace branch at >> [1]. > > I'd prefer if you would post full patches

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Vittorio Gambaletta (VittGam)
Hi, On 14/01/2016 23:15:16 CET, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 3:56 PM, Vittorio Gambaletta (VittGam) wrote: --- ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffplay.c b/ffplay.c index d2e3dc6..0837d58 100644 ---

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 11:48 AM, James Almer wrote: > On 1/14/2016 1:26 PM, Ganesh Ajjanagadde wrote: >> On Thu, Jan 14, 2016 at 11:16 AM, James Almer wrote: >>> On 1/14/2016 11:12 AM, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 5:02 AM, Henrik

[FFmpeg-devel] [PATCHv2] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
This improves accuracy (very slightly) and speed for processors having fma3. Sample benchmark (fate flac-16-lpc-cholesky, Haswell): old: 5993610 decicycles in ff_lpc_calc_coefs, 64 runs, 0 skips 5951528 decicycles in ff_lpc_calc_coefs, 128 runs, 0 skips new: 5252410 decicycles

[FFmpeg-devel] [PATCH] mpeg4videodec: silence ubsan warning

2016-01-14 Thread Andreas Cadhalpun
s->ac_val[0][0] is of type 'int16_t [16]', but points into a larger buffer. Here it is used as base pointer to find the correct position in the larger buffer by adding 's->block_index[n] * 16' and thus as 'int16_t *'. This fixes clang's ubsan runtime error: index out of bounds for type 'int16_t

Re: [FFmpeg-devel] [PATCH] lavc/aacenc: use isfinite to simplify isnan/isinf logic

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 6:14 PM, Claudio Freire wrote: > On Thu, Jan 14, 2016 at 7:57 PM, Ganesh Ajjanagadde > wrote: >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/aacenc.c | 17 + >> 1 file

Re: [FFmpeg-devel] [PATCHv2] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread James Almer
On 1/14/2016 7:46 PM, Ganesh Ajjanagadde wrote: > This improves accuracy (very slightly) and speed for processors having > fma3. > > Sample benchmark (fate flac-16-lpc-cholesky, Haswell): > old: > 5993610 decicycles in ff_lpc_calc_coefs, 64 runs, 0 skips > 5951528 decicycles in

Re: [FFmpeg-devel] [PATCH 1/5] asfdec_o: check avio_skip in asf_read_simple_index

2016-01-14 Thread Andreas Cadhalpun
On 06.01.2016 19:54, Andreas Cadhalpun wrote: > The loop can be very long, even though the file is very short. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/asfdec_o.c | 8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH 2/2] asfdec_o: reject size > INT64_MAX in asf_read_unknown

2016-01-14 Thread Andreas Cadhalpun
On 05.01.2016 13:26, Andreas Cadhalpun wrote: > Both avio_skip and detect_unknown_subobject use int64_t for the size > parameter. > > This fixes a segmentation fault due to infinite recursion. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/asfdec_o.c

Re: [FFmpeg-devel] [PATCH 1/2] asfdec_o: only set asf_pkt->data_size after sanity checks

2016-01-14 Thread Andreas Cadhalpun
On 05.01.2016 13:25, Andreas Cadhalpun wrote: > Otherwise invalid values are used unchecked in the next run. > This can cause NULL pointer dereferencing. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/asfdec_o.c | 18 ++ > 1 file

Re: [FFmpeg-devel] [PATCH 3/5] asfdec_o: make sure packet_size is non-zero before seeking

2016-01-14 Thread Andreas Cadhalpun
On 06.01.2016 19:55, Andreas Cadhalpun wrote: > This fixes infinite loops due to seeking back. > --- > libavformat/asfdec_o.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/asfdec_o.c b/libavformat/asfdec_o.c > index b81519f..4a3c815 100644 > ---

Re: [FFmpeg-devel] [PATCHv2] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread James Almer
On 1/14/2016 9:06 PM, Ganesh Ajjanagadde wrote: > On Thu, Jan 14, 2016 at 6:54 PM, Ganesh Ajjanagadde wrote: >> On Thu, Jan 14, 2016 at 6:42 PM, James Almer wrote: >>> On 1/14/2016 7:46 PM, Ganesh Ajjanagadde wrote: This improves accuracy (very slightly)

Re: [FFmpeg-devel] [PATCH] define AVPixelFormat aliases as enumerators instead of macros

2016-01-14 Thread wm4
On Thu, 14 Jan 2016 13:58:14 -0800 Richard Smith wrote: > libavutil/pixfmt.h defines a collection of endian-specific pixel formats as > macros. These macro names can cause conflicts with external projects that > use those identifiers for their own purposes. Here's a patch

Re: [FFmpeg-devel] [PATCH] avformat/riff: add C210 FourCC (CanopusC210)

2016-01-14 Thread Piotr Bandurski
> Sample? http://www.datafilehost.com/d/a36bf92f ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/v210: add avx2 version of the line encoder

2016-01-14 Thread James Darnley
On 2016-01-14 21:42, Henrik Gramner wrote: > On Thu, Jan 14, 2016 at 9:27 PM, James Darnley > wrote: >> On 2016-01-14 20:21, Henrik Gramner wrote: >>> xmN can be used unconditionally which gets rid of the %else. E.g. >>> >>> movu xm1, [yq+widthq*2] >>> %if

[FFmpeg-devel] [PATCH 1/3] configure: use -ldl for decklink

2016-01-14 Thread Marton Balint
Signed-off-by: Marton Balint --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index 7cef6f5..450c574 100755 --- a/configure +++ b/configure @@ -5194,6 +5194,8 @@ elif check_func dlopen -ldl; then ldl=-ldl fi

[FFmpeg-devel] [PATCH 2/3] lavd/decklink_dec: add support for teletext

2016-01-14 Thread Marton Balint
It uses the libzvbi slicer, therefore teletext capture requires libzvbi. Signed-off-by: Marton Balint --- doc/indevs.texi | 10 + libavdevice/decklink_common.h | 2 + libavdevice/decklink_common_c.h | 1 + libavdevice/decklink_dec.cpp| 99

[FFmpeg-devel] [PATCH 3/3] configure: remove libzvbi GPL dependency

2016-01-14 Thread Marton Balint
The COPYING.LIB file in the zvbi source tree as well as libzvbi.h references the GNU Library General Public License version 2. Signed-off-by: Marton Balint --- configure| 1 - doc/general.texi | 6 -- 2 files changed, 7 deletions(-) diff --git a/configure

Re: [FFmpeg-devel] [PATCH] avcodec/v210: add avx2 version of the line encoder

2016-01-14 Thread Henrik Gramner
On Thu, Jan 14, 2016 at 9:27 PM, James Darnley wrote: > On 2016-01-14 20:21, Henrik Gramner wrote: >> xmN can be used unconditionally which gets rid of the %else. E.g. >> >> movu xm1, [yq+widthq*2] >> %if cpuflag(avx2) >> vinserti128 m1, m1,

[FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Vittorio Gambaletta (VittGam)
--- ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffplay.c b/ffplay.c index d2e3dc6..0837d58 100644 --- a/ffplay.c +++ b/ffplay.c @@ -336,6 +336,8 @@ static int decoder_reorder_pts = -1; static int autoexit; static int exit_on_keydown; static int exit_on_mousedown;

Re: [FFmpeg-devel] [PATCH 2/3] lavd: add teletext quantizer

2016-01-14 Thread Carl Eugen Hoyos
Andrey Turkin gmail.com> writes: > Slicer is a part of the library and as such is under > LGPL; in fact FFmpeg already uses the library to > decode DVB teletext. But FFmpeg's configure claims libzvbi is GPL, an LGPL replacement for at least some functionality is certainly welcome! Carl

Re: [FFmpeg-devel] [PATCH 00/13] check all fclose usage

2016-01-14 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 10:48 AM, Ganesh Ajjanagadde wrote: > On Wed, Jan 13, 2016 at 4:05 AM, wm4 wrote: >> On Tue, 12 Jan 2016 10:07:08 -0500 >> Ganesh Ajjanagadde wrote: [...] > > > I apply them only when I am convinced universally

Re: [FFmpeg-devel] [PATCH] avcodec: add a native BBC Dirac VC-2 HQ encoder

2016-01-14 Thread Rostislav Pehlivanov
>There is a typo and please sanitize this: Will be corrected in next revision. They happen, the keys were right next to each other. I blame QWERTY. >If you (also) have copyright, say so, if you are only the author, make a note This is exactly the way the Opus decoder is licensed so I don't think

Re: [FFmpeg-devel] [PATCH] avcodec/v210: add avx2 version of the line encoder

2016-01-14 Thread James Darnley
On 2016-01-14 20:21, Henrik Gramner wrote: > On Wed, Jan 13, 2016 at 4:55 PM, James Darnley > wrote: >> diff --git a/libavcodec/x86/v210enc.asm b/libavcodec/x86/v210enc.asm >> index 859e2d9..a8f3d3c 100644 >> --- a/libavcodec/x86/v210enc.asm >> +++

Re: [FFmpeg-devel] [PATCH] avformat/riff: add YUYV FourCC (DrasticYUYV)

2016-01-14 Thread Piotr Bandurski
> Sample? http://www.datafilehost.com/d/0d19a1cb ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/3] lavd: add teletext quantizer

2016-01-14 Thread Marton Balint
On Wed, 13 Jan 2016, Michael Niedermayer wrote: On Wed, Jan 13, 2016 at 05:01:01AM +0300, Andrey Turkin wrote: Why not use libzvbi's slicer? It should be pretty robust with less-than-ideal signal. Regarding your code - is there a need to calculate a frequency? I did something similar a while

[FFmpeg-devel] [PATCH] avformat/riff: add C210 FourCC (Canopus C210)

2016-01-14 Thread Piotr Bandurski
0001-avformat-riff-add-C210-FourCC-Canopus-C210.diff Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: add a native BBC Dirac VC-2 HQ encoder

2016-01-14 Thread Carl Eugen Hoyos
Rostislav Pehlivanov gmail.com> writes: > + * Copyright (C) 2016 Open Broadcast Systemd Ltd. > + * Author(C) 2016 Rostislav Pehlivanov gmail.com> There is a typo and please sanitize this: If you (also) have copyright, say so, if you are only the author, make a note below. Great work!

[FFmpeg-devel] [PATCH v6 3/3] mips: add support for R6

2016-01-14 Thread Vicente Olivert Riera
Understanding the mips32r6 and mips64r6 ISAs in the configure script is not enough. In order to have full support for MIPS R6 in FFmpeg we need to be able to build it, and for that we need to make sure we don't use incompatible assembler code which makes the build fail. Ifdefing the offending code

[FFmpeg-devel] [PATCH v6 2/3] mips: do not disable any feature for generic cores

2016-01-14 Thread Vicente Olivert Riera
We don't know which features are available when the user selects a generic core, so don't disable anything by default and let the user decide. Signed-off-by: Vicente Olivert Riera --- Changes v4 -> v6: - Nothing. Changes v4 -> v5: - Nothing. Changes v3 -> v4: -

[FFmpeg-devel] [PATCH v6 1/3] mips: improve detection of ISAs, FPU and ASEs (DSP, MSA)

2016-01-14 Thread Vicente Olivert Riera
Signed-off-by: Vicente Olivert Riera --- Changes v5 -> v6: - Multiple changes: - Use separate if blocks when detecting the MIPS ISA, this way when a block disables one ISA it will be detected by the next one. Before the elif was preventing this to be done.

Re: [FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-14 Thread Ronald S. Bultje
Hi, On Wed, Jan 13, 2016 at 9:50 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > This makes SWS more robust > Fixes: > 07650a772d98aa63b0fed6370dc89037/asan_heap-oob_27ddeaf_2657_2c81ff264dee5d9712cb3251fb9c3bbb.264 > Fixes: out of array

Re: [FFmpeg-devel] [PATCH] avcodec/aacenc_is: replace pow(x, 0.75) by x/sqrtf(sqrtf(x))

2016-01-14 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 11:27 PM, Claudio Freire wrote: > On Mon, Jan 11, 2016 at 7:23 PM, Ganesh Ajjanagadde > wrote: >> This is quite an accurate approximation; testing shows ~ 2ulp error in >> the floating point result. Tested with FATE. >> >>

[FFmpeg-devel] [PATCH 3/7] avcodec/dca: add more DCA tables

2016-01-14 Thread foo86
--- libavcodec/dcadata.c | 622 ++- libavcodec/dcadata.h | 24 +- 2 files changed, 641 insertions(+), 5 deletions(-) diff --git a/libavcodec/dcadata.c b/libavcodec/dcadata.c index 0d0c218..e991134 100644 --- a/libavcodec/dcadata.c +++

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 11:16 AM, James Almer wrote: > On 1/14/2016 11:12 AM, Ganesh Ajjanagadde wrote: >> On Thu, Jan 14, 2016 at 5:02 AM, Henrik Gramner wrote: >>> Use the x86inc syntax for FMA instructions (basically FMA4 syntax that >>> gets assembled

[FFmpeg-devel] [PATCH 4/7] avcodec/synth_filter: fix whitespace

2016-01-14 Thread foo86
--- libavcodec/synth_filter.c | 45 +++-- 1 file changed, 23 insertions(+), 22 deletions(-) diff --git a/libavcodec/synth_filter.c b/libavcodec/synth_filter.c index 8dfca00..9eab9c5 100644 --- a/libavcodec/synth_filter.c +++ b/libavcodec/synth_filter.c @@

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 11:48 AM, James Almer wrote: > On 1/14/2016 1:26 PM, Ganesh Ajjanagadde wrote: >> On Thu, Jan 14, 2016 at 11:16 AM, James Almer wrote: >>> On 1/14/2016 11:12 AM, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 5:02 AM, Henrik

Re: [FFmpeg-devel] [PATCH] lavu/x86/lls: add fma3 optimizations for update_lls

2016-01-14 Thread James Almer
On 1/14/2016 1:26 PM, Ganesh Ajjanagadde wrote: > On Thu, Jan 14, 2016 at 11:16 AM, James Almer wrote: >> On 1/14/2016 11:12 AM, Ganesh Ajjanagadde wrote: >>> On Thu, Jan 14, 2016 at 5:02 AM, Henrik Gramner wrote: Use the x86inc syntax for FMA

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Ganesh Ajjanagadde
On Thu, Jan 14, 2016 at 3:56 PM, Vittorio Gambaletta (VittGam) wrote: > --- > ffplay.c | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/ffplay.c b/ffplay.c > index d2e3dc6..0837d58 100644 > --- a/ffplay.c > +++ b/ffplay.c > @@ -336,6 +336,8 @@ static