Re: [FFmpeg-devel] [PATCH]lavf/mpjpegdec: Do not av_log() while probing

2016-01-13 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > mpjpegdec.c |3 +++ > > 1 file changed, 3 insertions(+) > > acd4f042b52c6400a6aaf613b89ecad9ca3405ac patchmpjpeglog.diff > > should be ok Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 4/9] lavc/ccaption_dec: default rollup to row 10

2016-01-13 Thread Anshul
On 13 January 2016 7:12:55 AM IST, Aman Gupta wrote: >From: Aman Gupta > >This ensures that captions are written towards the bottom of the screen >when tuning into mid-stream. The row will be reset on the receipt of >the >next PAC command. Row 10 was chosen as

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

2016-01-13 Thread wm4
On Tue, 12 Jan 2016 10:07:08 -0500 Ganesh Ajjanagadde wrote: > On Tue, Jan 12, 2016 at 9:43 AM, Ronald S. Bultje wrote: > > Hi, > > > > On Tue, Jan 12, 2016 at 7:52 AM, Ganesh Ajjanagadde > > wrote: > > > >> On Tue, Jan 12, 2016 at 4:38

Re: [FFmpeg-devel] [PATCH v2 3/9] fate: add test for realtime ccaption decoder

2016-01-13 Thread Anshul
On 13 January 2016 7:12:54 AM IST, Aman Gupta wrote: >From: Aman Gupta > >--- > tests/fate/subtitles.mak | 3 +++ >tests/ref/fate/sub-cc-realtime | 42 >++ > 2 files changed, 45 insertions(+) > create mode 100644

Re: [FFmpeg-devel] MPlayer and QuickTime fourcc 'raw '

2016-01-13 Thread Mats Peterson
On 01/13/2016 09:06 AM, Mats Peterson wrote: slightly misleading, since it says that the 'raw ' fourcc stands for "Uncompressed RGB". However, below are two files with the 'raw ' fourcc I suppose what they mean by "uncompressed RGB" here is just a way to differentiate it from uncompressed

Re: [FFmpeg-devel] [PATCH v2 8/9] lavc/ccaption_dec: implement positioning for closed captions

2016-01-13 Thread wm4
On Wed, 13 Jan 2016 08:57:30 +0100 Clément Bœsch wrote: > On Tue, Jan 12, 2016 at 05:42:59PM -0800, Aman Gupta wrote: > > From: Aman Gupta > > > > Positioning math is based on the guidelines in > >

[FFmpeg-devel] MPlayer and QuickTime fourcc 'raw '

2016-01-13 Thread Mats Peterson
MPlayer seems to naively assume that a QuickTime file (or Matroska file with V_QUICKTIME, for that matter) with the video fourcc 'raw ' contains 24-bit RGB video data. The QuickTime File Format Specification is slightly misleading, since it says that the 'raw ' fourcc stands for "Uncompressed

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

2016-01-13 Thread wm4
On Tue, 12 Jan 2016 20:51:46 -0600 Rodger Combs wrote: > --- > libavcodec/avcodec.h| 2 ++ > libavcodec/mlp_parser.c | 5 + > libavcodec/mlpdec.c | 2 ++ > libavcodec/profiles.c | 6 ++ > libavcodec/profiles.h | 1 + > 5 files changed, 16

Re: [FFmpeg-devel] [PATCH]lavc/mjpeg2jpeg: Accept more mjpeg streams as input

2016-01-13 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > New patch attached that fixes ticket #5151. > > LGTM Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 2/9] lavc/ccaption_dec: implement real_time option

2016-01-13 Thread Anshul
On 13 January 2016 7:12:53 AM IST, Aman Gupta wrote: >From: Aman Gupta > >This new mode is useful for realtime decoding of closed captions so >they >can be display along with mpeg2 frames. > >Closed caption streams contain two major types of captions: > >- POPON

Re: [FFmpeg-devel] [PATCH v2 1/9] lavc/ccaption_dec: flush context on seek

2016-01-13 Thread Anshul
On 13 January 2016 7:12:52 AM IST, Aman Gupta wrote: >From: Aman Gupta > >--- > libavcodec/ccaption_dec.c | 21 + > 1 file changed, 21 insertions(+) > >diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c >index ca497e5..a9dfc94

[FFmpeg-devel] [PATCH] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

2016-01-13 Thread Mats Peterson
Match the use of AV_PIX_FMT_PAL8 for 1-bit QuickTime Animation in lavc/qtrle. To reiterate, 1-bit video is not necessary black & white in QuickTime, merely bi-level. The two colors can be any color. The palette, either included in the sample description, or the default Macintosh palette (black &

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

2016-01-13 Thread Kieran Kunhya
On Wed, 13 Jan 2016 at 02:16 Andrey Turkin wrote: > Why not use libzvbi's slicer? It should be pretty robust with > less-than-ideal signal. > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

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

2016-01-13 Thread Kieran Kunhya
On Wed, 13 Jan 2016 at 02:16 Andrey Turkin wrote: > Why not use libzvbi's slicer? It should be pretty robust with > less-than-ideal signal. > (now with a proper response). IMO FFmpeg shouldn't be doing analogue signal processing. This should be left to libzvbi. Kieran

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

2016-01-13 Thread wm4
On Wed, 13 Jan 2016 10:48:01 -0500 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: > > > >> On Tue, Jan 12, 2016 at 9:43 AM, Ronald S.

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

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 1:14 PM, wm4 wrote: > On Wed, 13 Jan 2016 10:48:01 -0500 > 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

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

2016-01-13 Thread foo86
On Tue, Jan 12, 2016 at 11:58:43PM +0100, Andreas Cadhalpun wrote: > It's not completely fixed yet, because the check is only done if > static_fields_present is 1. You are right, placing the check there doesn't account for case when static_fields_present becomes 0 later. I moved the check closer

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

2016-01-13 Thread Michael Niedermayer
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 --git a/libavutil/common.h b/libavutil/common.h > index f3276a2..5ae2847 100644 > --- a/libavutil/common.h > +++ b/libavutil/common.h >

Re: [FFmpeg-devel] [PATCH] dca: fix misaligned access in avpriv_dca_convert_bitstream

2016-01-13 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 12:56:39AM +0100, Andreas Cadhalpun wrote: > src and dst are only 8-bit-aligned, so accessing them as uint16_t causes > SIGBUS crashes on architectures like sparc. > > This fixes ubsan runtime error: load of misaligned address for type > 'const uint16_t', which requires 2

Re: [FFmpeg-devel] [PATCH v2] libi264: Add Hardware Accelerated H.264 Encoder based on libVA

2016-01-13 Thread Hamza Shahid
Hi Will, I think we addressed all the review comments in this patch. Which are as follows: 1. Remove inline asm for color conversion and replace with swscale, which I did. 2. Replace static variables with #defines, which I did. 3. Remove unnecessary stuff from configure script, which I did. 4.

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

2016-01-13 Thread Michael Niedermayer
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 back with VBI from TV signal and in my

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

2016-01-13 Thread Carl Eugen Hoyos
Andrey Turkin gmail.com> writes: > Why not use libzvbi's slicer? It should be pretty > robust with less-than-ideal signal. It is not entirely available under LGPL afaict. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [libav-devel] [PATCH] dca: fix misaligned access in avpriv_dca_convert_bitstream

2016-01-13 Thread Andreas Cadhalpun
On 13.01.2016 08:01, Luca Barbato wrote: > On 13/01/16 00:56, Andreas Cadhalpun wrote: >> src and dst are only 8-bit-aligned, so accessing them as uint16_t causes >> SIGBUS crashes on architectures like sparc. >> >> This fixes ubsan runtime error: load of misaligned address for type >> 'const

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

2016-01-13 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] swscale/yuv2rgb: Increase YUV2RGB table headroom

2016-01-13 Thread Christophe Gisquet
> -#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 everywhere it is actually used without showing up. Best regards, Christophe

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

2016-01-13 Thread Andreas Cadhalpun
On 13.01.2016 19:18, foo86 wrote: > On Tue, Jan 12, 2016 at 11:58:43PM +0100, Andreas Cadhalpun wrote: >> It's not completely fixed yet, because the check is only done if >> static_fields_present is 1. > > You are right, placing the check there doesn't account for case when >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacenc: Check for +-Inf too

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer wrote: > ffmpeg | branch: master | Michael Niedermayer | Wed > Jan 13 22:33:59 2016 +0100| [92465a2347d959cbd9864b017a39b2a4ab9313ff] | > committer: Michael Niedermayer > > avcodec/aacenc: Check for

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

2016-01-13 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 01:41:51AM +0300, Andrey Turkin wrote: > 2016-01-13 22:32 GMT+03:00 Michael Niedermayer : > > > to calculate the phase > > > > the area that has alternating 0/1 values can be correlated with a > > sin() and a cos(), if you consider the 2 resulting

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

2016-01-13 Thread Andrey Turkin
Slicer is a part of the library and as such is under LGPL; in fact FFmpeg already uses the library to decode DVB teletext. 2016-01-14 1:27 GMT+03:00 Carl Eugen Hoyos : > Andrey Turkin gmail.com> writes: > > > Why not use libzvbi's slicer? It should be pretty > > robust with

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

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 6:59 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

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacenc: Check for +-Inf too

2016-01-13 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote: > On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer > wrote: > > ffmpeg | branch: master | Michael Niedermayer | > > Wed Jan 13 22:33:59 2016 +0100|

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacenc: Check for +-Inf too

2016-01-13 Thread Hendrik Leppkes
On Thu, Jan 14, 2016 at 12:24 AM, Ganesh Ajjanagadde wrote: > On Wed, Jan 13, 2016 at 6:16 PM, Michael Niedermayer > wrote: >> On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote: >>> On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer

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

2016-01-13 Thread Andreas Cadhalpun
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 --git a/libavutil/common.h b/libavutil/common.h >> index f3276a2..5ae2847 100644 >>

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

2016-01-13 Thread Andrey Turkin
2016-01-13 22:32 GMT+03:00 Michael Niedermayer : > to calculate the phase > > the area that has alternating 0/1 values can be correlated with a > sin() and a cos(), if you consider the 2 resulting values as x and y > coordinates the angle they form from the origin is the

[FFmpeg-devel] [PATCH] avcodec/diracdec: Fix qfactor/offset tables

2016-01-13 Thread Michael Niedermayer
From: Michael Niedermayer It seems the previous tables where calculated with 32bit integers ignoring overflows. Also check for the max qindex, the value is choosen so that the qfactor/offset fit in int32. Fixes:

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacenc: Check for +-Inf too

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 6:16 PM, Michael Niedermayer wrote: > On Wed, Jan 13, 2016 at 06:03:32PM -0500, Ganesh Ajjanagadde wrote: >> On Wed, Jan 13, 2016 at 5:53 PM, Michael Niedermayer >> wrote: >> > ffmpeg | branch: master | Michael Niedermayer

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/aacenc: Check for +-Inf too

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 6:28 PM, Hendrik Leppkes wrote: > On Thu, Jan 14, 2016 at 12:24 AM, Ganesh Ajjanagadde wrote: >> On Wed, Jan 13, 2016 at 6:16 PM, Michael Niedermayer >> wrote: >>> On Wed, Jan 13, 2016 at 06:03:32PM -0500,

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

2016-01-13 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- configure| 1 + libavutil/libm.h | 4 2 files changed, 5 insertions(+) diff --git a/configure b/configure index 415d939..a3aaa25 100755 --- a/configure +++ b/configure @@ -1821,6 +1821,7 @@ MATH_FUNCS=" exp2f

Re: [FFmpeg-devel] [PATCH] dca: fix misaligned access in avpriv_dca_convert_bitstream

2016-01-13 Thread Andreas Cadhalpun
On 13.01.2016 20:07, Michael Niedermayer wrote: > On Wed, Jan 13, 2016 at 12:56:39AM +0100, Andreas Cadhalpun wrote: >> src and dst are only 8-bit-aligned, so accessing them as uint16_t causes >> SIGBUS crashes on architectures like sparc. >> >> This fixes ubsan runtime error: load of misaligned

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

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 8:07 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Jan 13, 2016 at 7:48 PM, Ganesh Ajjanagadde > wrote: >> >> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Wed, Jan 13,

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

2016-01-13 Thread Ronald S. Bultje
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, 2016 at 7:48 PM, Ganesh Ajjanagadde < > gajjanaga...@gmail.com> > > wrote: > >> > >> On Wed,

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

2016-01-13 Thread Ronald S. Bultje
Hi, On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > configure| 1 + > libavutil/libm.h | 4 > 2 files changed, 5 insertions(+) > > diff --git a/configure b/configure > index

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

2016-01-13 Thread Ronald S. Bultje
Hi, On Wed, Jan 13, 2016 at 7:48 PM, Ganesh Ajjanagadde wrote: > On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde < > gajjanaga...@gmail.com> > > wrote: > >> > >>

Re: [FFmpeg-devel] Discussion related to the use of a global thread pool rather than the current threading model

2016-01-13 Thread Jean-Yves Avenard
On 9 January 2016 at 01:37, Derek Buitenhuis wrote: > On 1/8/2016 2:19 PM, Jean-Yves Avenard wrote: >> I will come on Monday if that's okay... could setup a time that works >> for you. I'm based in Melbourne Australia > > Most of the channel is usually around during

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

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 8:11 PM, James Almer wrote: > On 1/13/2016 9:48 PM, Ganesh Ajjanagadde wrote: >> On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde

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

2016-01-13 Thread Claudio Freire
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. > > Alternatively, if one wants "full accuracy", one can use powf, or sqrt > instead of sqrtf.

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

2016-01-13 Thread James Almer
On 1/13/2016 9:48 PM, Ganesh Ajjanagadde wrote: > On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde >> wrote: >>> >>> Signed-off-by: Ganesh Ajjanagadde >>>

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

2016-01-13 Thread Michael Niedermayer
From: Michael Niedermayer This makes SWS more robust Fixes: 07650a772d98aa63b0fed6370dc89037/asan_heap-oob_27ddeaf_2657_2c81ff264dee5d9712cb3251fb9c3bbb.264 Fixes: out of array read Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael

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

2016-01-13 Thread Rodger Combs
--- 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 files changed, 17 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h

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

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 7:39 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Jan 13, 2016 at 7:10 PM, Ganesh Ajjanagadde > wrote: >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> configure| 1 + >> libavutil/libm.h | 4

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

2016-01-13 Thread Ganesh Ajjanagadde
On Wed, Jan 13, 2016 at 4:05 AM, wm4 wrote: > On Tue, 12 Jan 2016 10:07:08 -0500 > Ganesh Ajjanagadde wrote: > >> On Tue, Jan 12, 2016 at 9:43 AM, Ronald S. Bultje wrote: >> > Hi, >> > >> > On Tue, Jan 12, 2016 at 7:52 AM, Ganesh

Re: [FFmpeg-devel] [PATCH] diracdec: Add slice threading to HQ profile

2016-01-13 Thread Rostislav Pehlivanov
On Tue, 2016-01-12 at 21:15 +0100, Michael Niedermayer wrote: > On Tue, Jan 12, 2016 at 01:01:24AM +, Kieran Kunhya wrote: > > > > >  diracdec.c |   38 ++ > >  1 file changed, 30 insertions(+), 8 deletions(-) > > 73ce8405d52477db0faa2a2b76daca698a49bff2  

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/diracdec: Handle the 0 vlc case at the top of coeff_unpack_golomb()

2016-01-13 Thread Rostislav Pehlivanov
On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > encoding changes from 17 to 20 fps > > Signed-off-by: Michael Niedermayer > --- >  libavcodec/diracdec.c |   18 -- >  1 file changed, 12

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/diracdec: Inline svq3_get_ue_golomb() and merge the sign bit decoding into it

2016-01-13 Thread Rostislav Pehlivanov
On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > This avoids closing and opening the bit reader > > Signed-off-by: Michael Niedermayer > --- >  libavcodec/diracdec.c |   37

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

2016-01-13 Thread James Darnley
Around 35% faster than the avx version. --- libavcodec/v210enc.c | 5 ++-- libavcodec/v210enc.h | 1 + libavcodec/x86/v210enc.asm| 53 +++ libavcodec/x86/v210enc_init.c | 7 ++ 4 files changed, 49 insertions(+), 17 deletions(-)

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/diracdec: Factor +2 out of the inner loop

2016-01-13 Thread Rostislav Pehlivanov
On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- >  libavcodec/diracdec.c |   10 +- >  1 file changed, 5 insertions(+), 5 deletions(-) > LGTM, Thanks

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/diracdec: Factor +2 out of the inner loop

2016-01-13 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 04:12:54PM +, Rostislav Pehlivanov wrote: > On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > >  libavcodec/diracdec.c |   

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/diracdec: Handle the 0 vlc case at the top of coeff_unpack_golomb()

2016-01-13 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 04:12:17PM +, Rostislav Pehlivanov wrote: > On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > encoding changes from 17 to 20 fps > > > > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/diracdec: Inline svq3_get_ue_golomb() and merge the sign bit decoding into it

2016-01-13 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 04:11:40PM +, Rostislav Pehlivanov wrote: > On Tue, 2016-01-12 at 23:32 +0100, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > This avoids closing and opening the bit reader > > > > Signed-off-by: Michael Niedermayer