Re: [FFmpeg-devel] avcodec/proresenc_aw improvements

2018-07-21 Thread Michael Niedermayer
On Sat, Jul 21, 2018 at 02:33:24PM +0200, Martin Vignali wrote: [...] > 004 : calculate dct part before the quantif search. Avoid to recalculate it > each time > On a basic test Prores decoding -> prores encoding, the global speed > improvment is around 2% nice speedup [...] -- Michael Gn

Re: [FFmpeg-devel] [PATCH] avcodec/parser: move parsers list and related API to its own file

2018-07-21 Thread Michael Niedermayer
On Sat, Jul 21, 2018 at 05:36:02PM -0300, James Almer wrote: > On 7/21/2018 4:54 PM, Michael Niedermayer wrote: > > On Sat, Jul 21, 2018 at 02:28:21PM -0300, James Almer wrote: > >> And add it to the list of configurable components in Makefile, so changes > >> to > >> the file will be tracked and

Re: [FFmpeg-devel] [PATCH]lavf/mov: Make auxiliary_offsets consistently size_t.

2018-07-21 Thread Michael Niedermayer
On Sat, Jul 21, 2018 at 03:52:19PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes a warning on ppc32: > libavformat/mov.c: In function ‘mov_read_saio’: > libavformat/mov.c:6207:45: warning: assignment from incompatible > pointer type [-Wincompatible-pointer-types] > encrypt

Re: [FFmpeg-devel] [PATCH] avcodec/parser: move parsers list and related API to its own file

2018-07-21 Thread James Almer
On 7/21/2018 4:54 PM, Michael Niedermayer wrote: > On Sat, Jul 21, 2018 at 02:28:21PM -0300, James Almer wrote: >> And add it to the list of configurable components in Makefile, so changes to >> the file will be tracked and the usual warning to suggest re-running >> configure >> will be shown. >>

Re: [FFmpeg-devel] [PATCH] avcodec/parser: move parsers list and related API to its own file

2018-07-21 Thread Michael Niedermayer
On Sat, Jul 21, 2018 at 02:28:21PM -0300, James Almer wrote: > And add it to the list of configurable components in Makefile, so changes to > the file will be tracked and the usual warning to suggest re-running configure > will be shown. > > Signed-off-by: James Almer > --- > Makefile

Re: [FFmpeg-devel] [PATCH] avcodec/parser: move parsers list and related API to its own file

2018-07-21 Thread James Almer
On 7/21/2018 2:58 PM, Rostislav Pehlivanov wrote: > On 21 July 2018 at 18:28, James Almer wrote: > >> And add it to the list of configurable components in Makefile, so changes >> to >> the file will be tracked and the usual warning to suggest re-running >> configure >> will be shown. >> >> Signed

[FFmpeg-devel] [PATCH] swresample: Use channel count in rematrix initialization

2018-07-21 Thread Marcin Gorzel
Rematrixing supports up to 64 channels. However, there is only a limited number of channel layouts defined. Since the in/out channel count is currently obtained from the channel layout, for undefined layouts (e.g. for 9, 10, 11 channels etc.) the rematrixing fails. This patch changes rematrix i

Re: [FFmpeg-devel] avcodec/proresenc_aw : add support for prores 444

2018-07-21 Thread Paul B Mahol
On 7/21/18, Martin Vignali wrote: > Hello > > Patch in attach add support for prores 444 profile in AW prores encoder > > Need to be apply after patch in discussion "avcodec/proresenc_aw > improvments" > > Speed (from prores 422 HQ file -> Prores 444) > > ./ffmpeg -i prores422hqfile.mov -pix_fmt y

Re: [FFmpeg-devel] [PATCH] libswresample: Use channel count if channel layout is undefined

2018-07-21 Thread Marcin Gorzel
Of course, I will update the patch and send it out shortly. Using in.ch_count should be ok. The relevant check is in swresample.c:292, so we should never reach that code if the in.ch_count is 0. On Fri, Jul 20, 2018 at 7:36 PM Michael Niedermayer wrote: > On Fri, Jul 20, 2018 at 09:51:49AM +0200

Re: [FFmpeg-devel] [PATCH] avcodec/parser: move parsers list and related API to its own file

2018-07-21 Thread Rostislav Pehlivanov
On 21 July 2018 at 18:28, James Almer wrote: > And add it to the list of configurable components in Makefile, so changes > to > the file will be tracked and the usual warning to suggest re-running > configure > will be shown. > > Signed-off-by: James Almer > --- > Makefile | 1 + >

Re: [FFmpeg-devel] [PATCH] avcodec/decode: copy the output parameters from the last bsf in the chain back to the AVCodecContext

2018-07-21 Thread James Almer
On 7/21/2018 2:34 PM, Michael Niedermayer wrote: > On Fri, Jul 20, 2018 at 08:30:19PM -0300, James Almer wrote: >> On 7/20/2018 8:23 PM, Michael Niedermayer wrote: >>> On Thu, Jul 19, 2018 at 10:54:23PM -0300, James Almer wrote: Certain AVCodecParameters, like the contents of the extradata, ma

Re: [FFmpeg-devel] avcodec/proresenc_aw improvements

2018-07-21 Thread Michael Niedermayer
On Sat, Jul 21, 2018 at 02:33:24PM +0200, Martin Vignali wrote: > Hello, > > Patch in attach improve some part of the prores_aw encoder. > > 001 : use scantable in prores_data instead of a duplicate one. This could negativly affect the performance of the changed inner loop have you checked that

Re: [FFmpeg-devel] [PATCH] avcodec/decode: copy the output parameters from the last bsf in the chain back to the AVCodecContext

2018-07-21 Thread Michael Niedermayer
On Fri, Jul 20, 2018 at 08:30:19PM -0300, James Almer wrote: > On 7/20/2018 8:23 PM, Michael Niedermayer wrote: > > On Thu, Jul 19, 2018 at 10:54:23PM -0300, James Almer wrote: > >> Certain AVCodecParameters, like the contents of the extradata, may be > >> changed > >> by the init() function of an

[FFmpeg-devel] [PATCH] avcodec/parser: move parsers list and related API to its own file

2018-07-21 Thread James Almer
And add it to the list of configurable components in Makefile, so changes to the file will be tracked and the usual warning to suggest re-running configure will be shown. Signed-off-by: James Almer --- Makefile | 1 + configure| 2 +- libavcodec/Makefile | 1 + lib

[FFmpeg-devel] [PATCH] avformat/mov: Check default_encrypted_sample before use in mov_read_sample_encryption_info()

2018-07-21 Thread Michael Niedermayer
Fixes: 2018-0721-sample Fixes: null pointer dereference Found-by: Nikita Knyzhov (knnik...@yandex.ru) Signed-off-by: Michael Niedermayer --- libavformat/mov.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index eda3fff6d5..edd7778023 100644 --- a/

[FFmpeg-devel] avcodec/proresenc_aw : add support for prores 444

2018-07-21 Thread Martin Vignali
Hello Patch in attach add support for prores 444 profile in AW prores encoder Need to be apply after patch in discussion "avcodec/proresenc_aw improvments" Speed (from prores 422 HQ file -> Prores 444) ./ffmpeg -i prores422hqfile.mov -pix_fmt yuv444p10 -c:v prores -an -profile:v 4 res_aw.mov -y

Re: [FFmpeg-devel] avcodec/proresenc_aw improvements

2018-07-21 Thread Martin Vignali
> > Am I correct that the output files also change? > Patch 002 : change the header of the output file only, not the compress data. Patch 003 : change the output file mainly for near uniform bloc, where the low scale quantif value where used. But IMHO is better by default to be close to official

[FFmpeg-devel] [PATCH]lavf/mov: Make auxiliary_offsets consistently size_t.

2018-07-21 Thread Carl Eugen Hoyos
Hi! Attached patch fixes a warning on ppc32: libavformat/mov.c: In function ‘mov_read_saio’: libavformat/mov.c:6207:45: warning: assignment from incompatible pointer type [-Wincompatible-pointer-types] encryption_index->auxiliary_offsets = auxiliary_offsets;

Re: [FFmpeg-devel] [PATCH] libavcodec/escape130: fixes and improvements

2018-07-21 Thread Carl Eugen Hoyos
2018-07-20 12:31 GMT+02:00, Michael Chaban : > 2018-07-19 21:28 GMT+03:00 Carl Eugen Hoyos : >> The first commit breaks fate, you have to (confirm the fix and) >> change the reference values in the same patch. > > The first commit fixes the wrong decoding which was error in first > place for FFmpeg

Re: [FFmpeg-devel] [PATCH] doc/filters: update info about avgblur filter to make it more understandable

2018-07-21 Thread Gyan Doshi
On 20-07-2018 10:01 PM, Danil Iashchenko wrote: doc/filters: update info about avgblur filter to make it more understandable --- doc/filters.texi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 705d48e..e3005c3 100644 --- a/d

Re: [FFmpeg-devel] [PATCH 1/2] docs/filters: add documentation to all existing OpenCL filters

2018-07-21 Thread Gyan Doshi
I'll review this within a couple of days. Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] avcodec/proresenc_aw improvements

2018-07-21 Thread Carl Eugen Hoyos
2018-07-21 14:33 GMT+02:00, Martin Vignali : > Patch in attach improve some part of the prores_aw encoder. > > 001 : use scantable in prores_data instead of a duplicate one. > 002 : use for the frame header, primaries, transfert, colorspace like in > proresenc_ks avoid color shift on some software

[FFmpeg-devel] avcodec/proresenc_aw improvements

2018-07-21 Thread Martin Vignali
Hello, Patch in attach improve some part of the prores_aw encoder. 001 : use scantable in prores_data instead of a duplicate one. 002 : use for the frame header, primaries, transfert, colorspace like in proresenc_ks avoid color shift on some software (update fate test) 003 : change qp start value