Re: [FFmpeg-devel] [PATCH 3/3] avformat/avio: remove 4k limit from avio_printf

2019-08-06 Thread Marton Balint
On Tue, 6 Aug 2019, Nicolas George wrote: Marton Balint (12019-08-05): We do this by switching to AVBPrint. Signed-off-by: Marton Balint --- doc/APIchanges| 3 +++ libavformat/avio.h| 5 - libavformat/aviobuf.c | 15 ++- libavformat/version.h | 2 +- 4 files

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avio: add avio_print_n_strings and avio_print

2019-08-06 Thread Marton Balint
On Tue, 6 Aug 2019, Nicolas George wrote: Marton Balint (12019-08-05): These functions can be used to print a variable number of strings consecutively to the IO context. Unlike av_bprintf, no temporery buffer is necessary. I do not like this very much: the VA design disables type checking,

[FFmpeg-devel] [PATCH] avcodec/videotoolbox_hevc: avoid leaking cached_hw_frames_ctx

2019-08-06 Thread Pavel Koshevoy
vtctx->cached_hw_frames_ctx is unref'd in videotoolbox_uninit, but videotoolbox_hevc used ff_videotoolbox_uninit which doesn't unref cache_hw_frames_ctx. --- libavcodec/videotoolbox.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/videotoolbox.c b/libavcodec/videoto

Re: [FFmpeg-devel] [PATCH v2] libavformat: Add ZeroMQ as a protocol option

2019-08-06 Thread Jack Waller
On Wed, Aug 7, 2019 at 10:22 AM Andriy Gelman wrote: > On Sun, 04. Aug 14:36, Andriy Gelman wrote: > > Changes in v2: > > 1. Replaced zmq_poll with zmq_msg_recv. > > 2. Remove user timeout option as zmq_msg_recv(.., .., ZMQ_DONTWAIT) is > a > > non-blocking call. > > 3. Updated docs. > >

Re: [FFmpeg-devel] [PATCH v2] avutil/mips: refactor msa SLDI_Bn_0 and SLDI_Bn macros.

2019-08-06 Thread Shiyou Yin
LGTM. >-Original Message- >From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] >On Behalf Of gxw >Sent: Tuesday, August 6, 2019 7:11 PM >To: ffmpeg-devel@ffmpeg.org >Subject: [FFmpeg-devel] [PATCH v2] avutil/mips: refactor msa SLDI_Bn_0 and >SLDI_Bn macros. > >

[FFmpeg-devel] [PATCH V2] FATE/dnn: let fate/dnn tests depend on ffmpeg static libraries

2019-08-06 Thread Guo, Yejun
background: DNN (deep neural network) is a sub module of libavfilter, and FATE/dnn is unit test for the DNN module, one unit test for one dnn layer. The unit tests are not based on the APIs exported by libavfilter, they just directly call into the functions within DNN submodule. There is an issue

Re: [FFmpeg-devel] [PATCH v2] libavformat: Add ZeroMQ as a protocol option

2019-08-06 Thread Andriy Gelman
On Sun, 04. Aug 14:36, Andriy Gelman wrote: > Changes in v2: > 1. Replaced zmq_poll with zmq_msg_recv. > 2. Remove user timeout option as zmq_msg_recv(.., .., ZMQ_DONTWAIT) is a > non-blocking call. > 3. Updated docs. > > Andriy > > From 53e6e00d30c9fbf5127eea9d377686d37e981c0c Mon Sep

[FFmpeg-devel] [PATCH] avcodec/libdav1d: allow setting frame size limit in pixels

2019-08-06 Thread James Almer
Signed-off-by: James Almer --- configure | 2 +- libavcodec/libdav1d.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 34c2adb4a4..eceb21b315 100755 --- a/configure +++ b/configure @@ -6196,7 +6196,7 @@ enabled libcelt && requ

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-08-06 Thread Mark Thompson
On 05/08/2019 20:20, Juan De León wrote: > AVQuantizationParams data structure for extracting qp and storing as > AV_FRAME_DATA_QUANTIZATION_PARAMS AVFrameSideDataType > design doc: > https://docs.google.com/document/d/1WClt3EqhjwdGXhEw386O0wfn3IBQ1Ib-_5emVM1gbnA/edit?usp=sharing > > Signed-off-

[FFmpeg-devel] [PATCH v4] avcodec/vaapi_encode_h264: add support for a/53 closed caption sei

2019-08-06 Thread Aman Gupta
From: Aman Gupta Signed-off-by: Aman Gupta --- libavcodec/vaapi_encode_h264.c | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c index f4965d8b09..f3da3ee299 100644 --- a/libavcodec/v

Re: [FFmpeg-devel] [PATCH] doc/fate: Document how to request samples upload access

2019-08-06 Thread Thilo Borgmann
Am 05.08.19 um 19:06 schrieb Michael Niedermayer: > Signed-off-by: Michael Niedermayer > --- > doc/fate.texi | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/fate.texi b/doc/fate.texi > index 2be61d639c..e04fbfbe43 100644 > --- a/doc/fate.texi > +++ b/doc/fate.texi > @@ -157,6 +15

Re: [FFmpeg-devel] [PATCH 4/6] avcodec/hnm4video: Optimize postprocess_current_frame()

2019-08-06 Thread Michael Niedermayer
On Mon, Aug 05, 2019 at 11:58:01AM +0200, Tomas Härdin wrote: > lör 2019-08-03 klockan 18:57 +0200 skrev Michael Niedermayer: > > On Sat, Aug 03, 2019 at 04:07:22PM +0200, Tomas Härdin wrote: > > > lör 2019-08-03 klockan 01:49 +0200 skrev Michael Niedermayer: > > > > -uint32_t x, y, src_x, src_

Re: [FFmpeg-devel] libavcodec: add timer bitstream filter [v2]

2019-08-06 Thread Alexander Strasser
Hi Andreas! On 2019-08-05 17:25 +, Andreas Håkon wrote: > On Monday, 5 de August de 2019 17:31, Moritz Barsnick > wrote: > [...] > > > +static const AVClass timer_class = { > > > + .class_name = "timer", > > > > In about half of the existing BSFs, this would be called "timer_bsf". I > > d

Re: [FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Alexander Strasser
On 2019-08-06 22:51 +0200, Nicolas George wrote: > Tomas Härdin (12019-08-06): > > I think they should. See > > Documenting the coding standard and changing it are different matters. I guess a change was intended, because it should be fairly obvious that in the current code base the described styl

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avio: add avio_print_n_strings and avio_print

2019-08-06 Thread Alexander Strasser
Hi Marton! Not really sure if we need the API, but it definitely looks handy. Just not sure how often it will used and really result in clearer or shorter code. Not opposed to it though; no strong opinion on this one. Some comments follow inline. No in depth review, just what came to my mind whe

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/pafvideo: Only clear frame when it was written to

2019-08-06 Thread Lynne
Aug 6, 2019, 10:30 PM by mich...@niedermayer.cc: > This avoids unneeded operations and makes the code faster. > > Fixes: Timeout (12sec -> 9sec) > Fixes: > 15724/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PAF_VIDEO_fuzzer-5750842205929472 > > Found-by: continuous fuzzing process > https:/

Re: [FFmpeg-devel] [PATCH 3/3] avformat/avio: remove 4k limit from avio_printf

2019-08-06 Thread Nicolas George
Marton Balint (12019-08-05): > We do this by switching to AVBPrint. > > Signed-off-by: Marton Balint > --- > doc/APIchanges| 3 +++ > libavformat/avio.h| 5 - > libavformat/aviobuf.c | 15 ++- > libavformat/version.h | 2 +- > 4 files changed, 18 insertions(+), 7 d

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avio: add avio_print_n_strings and avio_print

2019-08-06 Thread Nicolas George
Marton Balint (12019-08-05): > These functions can be used to print a variable number of strings > consecutively > to the IO context. Unlike av_bprintf, no temporery buffer is necessary. I do not like this very much: the VA design disables type checking, which makes it rather fragile, and the ben

[FFmpeg-devel] [PATCH 1/5] avcodec/vp56: Consider the alpha start as end of the prior header

2019-08-06 Thread Michael Niedermayer
Fixes: Timeout (23sec -> 71ms) Fixes: 15661/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VP6A_fuzzer-6257865947348992 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/vp56.c | 4 ++-- 1 fi

[FFmpeg-devel] [PATCH 5/5] avcodec/pafvideo: Only clear frame when it was written to

2019-08-06 Thread Michael Niedermayer
This avoids unneeded operations and makes the code faster. Fixes: Timeout (12sec -> 9sec) Fixes: 15724/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PAF_VIDEO_fuzzer-5750842205929472 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-o

[FFmpeg-devel] [PATCH 2/5] [RFC] avcodec/hevcdec: Check for overread in hls_decode_entry()

2019-08-06 Thread Michael Niedermayer
This checks by adjusting the bytestream end pointer so as to detect overread without the need of additional code in the innermost loop. This should be safe as arrays have additional AV_INPUT_BUFFER_PADDING_SIZE at their end. This is simple and does not cause a slowdown but it is hackish. The clean

[FFmpeg-devel] [PATCH 4/5] avcodec/tiff: Detect infinite retry loop

2019-08-06 Thread Michael Niedermayer
Fixes: Timeout (Infinite->Finite) Fixes: 15706/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TIFF_fuzzer-5114674904825856 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/tiff.c | 6 +-

[FFmpeg-devel] [PATCH 3/5] avcodec/hevc_refs: Optimize 16bit generate_missing_ref()

2019-08-06 Thread Michael Niedermayer
Fixes: Timeout (86sec -> 8sec) Fixes: 15702/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HEVC_fuzzer-5657764929470464 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/hevc_refs.c | 12

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-08-06 Thread Juan De León
On Mon, Aug 5, 2019 at 1:07 PM Michael Niedermayer wrote > a macro would still require a major version bump as its a ABI change, if > it changes. > To fix this would probably require a deeper change, we could also > of course just leave it and accept that as the maximum till the next > ABI major

Re: [FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Nicolas George
Tomas Härdin (12019-08-06): > I think they should. See Documenting the coding standard and changing it are different matters. > https://www.imperialviolet.org/2014/02/22/applebug.html test_warn.c: In function ‘test’: test_warn.c:6:5: warning: this ‘if’ clause does not guard... [-Wmisleading-in

Re: [FFmpeg-devel] [PATCH v3 4/6] avformat/wavdec: s337m support

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 17:08 +0200 skrev Nicolas Gaullier: > Enabled by AVOption dolbyeprobe > Requires stereo > --- > libavformat/s337m.h | 1 + > libavformat/wavdec.c | 23 +++ > 2 files changed, 24 insertions(+) > > diff --git a/libavformat/s337m.h b/libavformat/s337m.h >

Re: [FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 18:33 +0200 skrev Nicolas George: > Nicolas Gaullier (12019-08-06): > > --- > > doc/developer.texi | 8 > > 1 file changed, 8 insertions(+) > > > > diff --git a/doc/developer.texi b/doc/developer.texi > > index 5c342c9106..1e95768364 100644 > > --- a/doc/developer.

Re: [FFmpeg-devel] [PATCH v3 3/6] avformat/s337m: Make available as subdemuxer

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 17:08 +0200 skrev Nicolas Gaullier: > +int avpriv_s337m_probe_stream(void *avc, AVIOContext *pb, AVStream **st, int > size) > +{ > +if ( size >= S337M_MIN_OFFSET && > +((*st)->codecpar->codec_id == AV_CODEC_ID_PCM_S16LE || > (*st)->codecpar->codec_id == AV_CODEC

Re: [FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Lynne
Aug 6, 2019, 5:33 PM by geo...@nsup.org: > Nicolas Gaullier (12019-08-06): > >> --- >> doc/developer.texi | 8 >> 1 file changed, 8 insertions(+) >> >> diff --git a/doc/developer.texi b/doc/developer.texi >> index 5c342c9106..1e95768364 100644 >> --- a/doc/developer.texi >> +++ b/doc/dev

Re: [FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Nicolas George
Nicolas Gaullier (12019-08-06): > --- > doc/developer.texi | 8 > 1 file changed, 8 insertions(+) > > diff --git a/doc/developer.texi b/doc/developer.texi > index 5c342c9106..1e95768364 100644 > --- a/doc/developer.texi > +++ b/doc/developer.texi > @@ -213,6 +213,14 @@ please use av_log(

[FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Nicolas Gaullier
--- doc/developer.texi | 8 1 file changed, 8 insertions(+) diff --git a/doc/developer.texi b/doc/developer.texi index 5c342c9106..1e95768364 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -213,6 +213,14 @@ please use av_log() instead. @item Casts should be used only when

[FFmpeg-devel] [PATCH 2/2] swscale: Fix AltiVec/VSX build with recent GCC

2019-08-06 Thread Daniel Kolesa
The argument to vec_splat_u16 must be a literal. By making the function always inline and marking the arguments const, gcc can turn those into literals, and avoid build errors like: swscale_vsx.c:165:53: error: argument 1 must be a 5-bit signed literal Signed-off-by: Daniel Kolesa --- libswscal

[FFmpeg-devel] [PATCH 1/2] swscale: Replace illegal vector keyword usage in altivec code

2019-08-06 Thread Daniel Kolesa
While this technically compiles in current ffmpeg, this is only because ffmpeg is compiled in strict ISO C mode, which disables the builtin 'vector' keyword for AltiVec/VSX. Instead this gets replaced with a macro inside altivec.h, which defines vector to be actually __vector, which accepts random

[FFmpeg-devel] [PATCH 0/2] AltiVec/VSX fixes in swscale

2019-08-06 Thread Daniel Kolesa
The first of these patches fixes the original report as described in https://trac.ffmpeg.org/ticket/7861. The second of the patches fixes the second report in the same ticket (by leigh123linux). Daniel Kolesa (2): swscale: Replace illegal vector keyword usage in altivec code swscale: Fix AltiV

[FFmpeg-devel] [PATCH v3 5/6] avformat/wavdec: reindent

2019-08-06 Thread Nicolas Gaullier
--- libavformat/wavdec.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c index 2ef2d9e549..b9a782bec5 100644 --- a/libavformat/wavdec.c +++ b/libavformat/wavdec.c @@ -712,14 +712,14 @@ smv_out: } if (!CONF

[FFmpeg-devel] [PATCH v3 4/6] avformat/wavdec: s337m support

2019-08-06 Thread Nicolas Gaullier
Enabled by AVOption dolbyeprobe Requires stereo --- libavformat/s337m.h | 1 + libavformat/wavdec.c | 23 +++ 2 files changed, 24 insertions(+) diff --git a/libavformat/s337m.h b/libavformat/s337m.h index 0f21a23a30..aff76373d3 100644 --- a/libavformat/s337m.h +++ b/libavfor

[FFmpeg-devel] [PATCH v3 0/6] avformat: Use s337m subdemux inside wav

2019-08-06 Thread Nicolas Gaullier
Corrected with tomas's feedback The guard band minimum size is now specified in bytes to simplify and restrict the test to an integer checking, it is no big deal for me, and anyway I have set it to 0 by default, just overridable if someone wants to set it. S337M_MAX_RECOMMENDED_PROBE_SIZE is no

[FFmpeg-devel] [PATCH v3 3/6] avformat/s337m: Make available as subdemuxer

2019-08-06 Thread Nicolas Gaullier
--- libavformat/s337m.c | 64 - libavformat/s337m.h | 64 + 2 files changed, 117 insertions(+), 11 deletions(-) create mode 100644 libavformat/s337m.h diff --git a/libavformat/s337m.c b/libavf

Re: [FFmpeg-devel] [PATCH v2 5/5] avformat/s337m: Test wav subdemux

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 13:09 + skrev Gaullier Nicolas: > > > +FATE_SAMPLES_AUDIO-$(call DEMDEMDEC, WAV, S337M, DOLBY_E) += > > > +fate-dolby-e-wav > > > +fate-dolby-e-wav: CMD = dolbye2pcm16 -i > > > +$(TARGET_SAMPLES)/dolby_e/512.wav > > > +fate-dolby-e-wav: CMP = oneoff > > > +fate-dolby-e-

Re: [FFmpeg-devel] [PATCH] web/index: add news entry for release 4.2

2019-08-06 Thread James Almer
On 8/6/2019 10:47 AM, Thomas Volkert wrote: > On 06.08.2019 15:41, James Almer wrote: >> Signed-off-by: James Almer >> --- >> src/index | 47 +++ >> 1 file changed, 47 insertions(+) >> >> diff --git a/src/index b/src/index >> index 0dcf6c1..fafb0e8 1006

Re: [FFmpeg-devel] [PATCH] web/index: add news entry for release 4.2

2019-08-06 Thread Thomas Volkert
On 06.08.2019 15:41, James Almer wrote: > Signed-off-by: James Almer > --- > src/index | 47 +++ > 1 file changed, 47 insertions(+) > > diff --git a/src/index b/src/index > index 0dcf6c1..fafb0e8 100644 > --- a/src/index > +++ b/src/index > @@ -37,6 +37

[FFmpeg-devel] [PATCH] web/index: add news entry for release 4.2

2019-08-06 Thread James Almer
Signed-off-by: James Almer --- src/index | 47 +++ 1 file changed, 47 insertions(+) diff --git a/src/index b/src/index index 0dcf6c1..fafb0e8 100644 --- a/src/index +++ b/src/index @@ -37,6 +37,53 @@ News + August 5th, 2019, FFmpeg 4.2 "Ada

Re: [FFmpeg-devel] FFmpeg 4.2

2019-08-06 Thread James Almer
On 8/6/2019 9:44 AM, Gyan wrote: > > > On 06-08-2019 03:19 AM, Michael Niedermayer wrote: >> On Sun, Aug 04, 2019 at 07:56:00AM +0200, Michael Niedermayer wrote: >>> On Sun, Jul 21, 2019 at 07:11:40PM +0200, Michael Niedermayer wrote: On Mon, May 20, 2019 at 08:39:45PM +0200, Michael Niederm

Re: [FFmpeg-devel] [PATCH v2 5/5] avformat/s337m: Test wav subdemux

2019-08-06 Thread Gaullier Nicolas
>> +FATE_SAMPLES_AUDIO-$(call DEMDEMDEC, WAV, S337M, DOLBY_E) += >> +fate-dolby-e-wav >> +fate-dolby-e-wav: CMD = dolbye2pcm16 -i >> +$(TARGET_SAMPLES)/dolby_e/512.wav >> +fate-dolby-e-wav: CMP = oneoff >> +fate-dolby-e-wav: REF = $(SAMPLES)/dolby_e/512.wav.pcm > >This does a floaty comparison, r

Re: [FFmpeg-devel] FFmpeg 4.2

2019-08-06 Thread Gyan
On 06-08-2019 03:19 AM, Michael Niedermayer wrote: On Sun, Aug 04, 2019 at 07:56:00AM +0200, Michael Niedermayer wrote: On Sun, Jul 21, 2019 at 07:11:40PM +0200, Michael Niedermayer wrote: On Mon, May 20, 2019 at 08:39:45PM +0200, Michael Niedermayer wrote: Hi Its quite some time since 4.1

Re: [FFmpeg-devel] [PATCH v2 1/5] avformat/s337m: Use base AVClass for av_log usage

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 13:50 +0200 skrev Nicolas Gaullier: > s337m_get_offset_and_codec does not make use of > AVFormatContext: AVClass is enough for logging. > Will facilitate further use from outside > --- > libavformat/s337m.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) >

Re: [FFmpeg-devel] [PATCH v2 5/5] avformat/s337m: Test wav subdemux

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 13:50 +0200 skrev Nicolas Gaullier: > diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak > index c41958ea2d..0e56d401ea 100644 > --- a/tests/fate/audio.mak > +++ b/tests/fate/audio.mak > @@ -24,6 +24,11 @@ fate-dolby-e: CMD = pcm -i $(TARGET_SAMPLES)/dolby_e/16-11 > fa

Re: [FFmpeg-devel] [PATCH v2 4/5] avformat/wavdec: s337m support

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 13:50 +0200 skrev Nicolas Gaullier: > Enabled by AVOption dolbyeprobe > Requires stereo > static void set_spdif(AVFormatContext *s, WAVDemuxContext *wav) > @@ -593,6 +599,10 @@ break_loop: > } else if (st->codecpar->codec_id == AV_CODEC_ID_ADPCM_MS && > st->codecpar->

Re: [FFmpeg-devel] [PATCH v2 2/5] avformat/s337m: Split read_packet/get_packet

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 13:50 +0200 skrev Nicolas Gaullier: > Prepare use of s337m_get_packet from outside > --- > libavformat/s337m.c | 29 +++-- > 1 file changed, 23 insertions(+), 6 deletions(-) Looks OK /Tomas ___ ffmpeg-dev

Re: [FFmpeg-devel] [PATCH v2 3/5] avformat/s337m: Make available as subdemuxer

2019-08-06 Thread Tomas Härdin
tis 2019-08-06 klockan 13:50 +0200 skrev Nicolas Gaullier: > +int avpriv_s337m_probe_stream(void *avc, AVIOContext *pb, AVStream > **st, int size) > +{ > +if ( size >= S337M_MIN_OFFSET && > +((*st)->codecpar->codec_id == AV_CODEC_ID_PCM_S16LE || > (*st)->codecpar->codec_id == AV_CODEC_I

[FFmpeg-devel] [PATCH] avutil/mips: refine msa macros CLIP_*.

2019-08-06 Thread gxw
Changing details as following: 1. Refine CLIP_SH, results are in placed to input vectors. 2. Reimplement the macro CLIP_SH/Wn_0_255. The new macro is more efficient than before. 3. Remove CLIP_SH/Wn_0_255_MAX_SATU. CLIP_SH/Wn_0_255_MAX_SATU and CLIP_SH/Wn_0_255 have the same function. It is n

[FFmpeg-devel] [PATCH v2 5/5] avformat/s337m: Test wav subdemux

2019-08-06 Thread Nicolas Gaullier
incl. 24bits 5.1+2 dolby_e decoding --- tests/Makefile | 1 + tests/fate-run.sh| 4 tests/fate/audio.mak | 5 + 3 files changed, 10 insertions(+) diff --git a/tests/Makefile b/tests/Makefile index 0ef571b067..ebaa465597 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -75,6

[FFmpeg-devel] [PATCH v2 1/5] avformat/s337m: Use base AVClass for av_log usage

2019-08-06 Thread Nicolas Gaullier
s337m_get_offset_and_codec does not make use of AVFormatContext: AVClass is enough for logging. Will facilitate further use from outside --- libavformat/s337m.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/s337m.c b/libavformat/s337m.c index 48ab66a6da

[FFmpeg-devel] [PATCH v2 2/5] avformat/s337m: Split read_packet/get_packet

2019-08-06 Thread Nicolas Gaullier
Prepare use of s337m_get_packet from outside --- libavformat/s337m.c | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git a/libavformat/s337m.c b/libavformat/s337m.c index 8956afb23f..22140297e6 100644 --- a/libavformat/s337m.c +++ b/libavformat/s337m.c

[FFmpeg-devel] [PATCH v2 4/5] avformat/wavdec: s337m support

2019-08-06 Thread Nicolas Gaullier
Enabled by AVOption dolbyeprobe Requires stereo --- libavformat/s337m.h | 1 + libavformat/wavdec.c | 41 ++--- 2 files changed, 35 insertions(+), 7 deletions(-) diff --git a/libavformat/s337m.h b/libavformat/s337m.h index bfe2a5d457..0c3fbf1990 100644 --- a/

[FFmpeg-devel] [PATCH v2 3/5] avformat/s337m: Make available as subdemuxer

2019-08-06 Thread Nicolas Gaullier
--- libavformat/s337m.c | 63 +++- libavformat/s337m.h | 64 + 2 files changed, 116 insertions(+), 11 deletions(-) create mode 100644 libavformat/s337m.h diff --git a/libavformat/s337m.c b/libavfo

[FFmpeg-devel] [PATCH v2 0/5] avformat: Use s337m subdemux inside wav

2019-08-06 Thread Nicolas Gaullier
As discussed with tomas, this second revision does not include MXF anymore as it raises broader questions yet. The following track would certainly be to developp a generic audio filter that could be used in the general case with any format. Tests samples for fate-suite/dolby_e uploaded here: htt

[FFmpeg-devel] [PATCH] avfilter/af_compand: change error path into warning

2019-08-06 Thread Paul B Mahol
Hi, patch attached. 0001-avfilter-af_compand-check-error-condition-into-warni.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or em

[FFmpeg-devel] [PATCH v2] avutil/mips: refactor msa SLDI_Bn_0 and SLDI_Bn macros.

2019-08-06 Thread gxw
Changing details as following: 1. The previous order of parameters are irregular and difficult to understand. Adjust the order of the parameters according to the rule: (RTYPE, input registers, input mask/input index/..., output registers). Most of the existing msa macros follow the rule. 2

Re: [FFmpeg-devel] [RFC] samples.ffmpeg.org

2019-08-06 Thread Michael Niedermayer
On Mon, Aug 05, 2019 at 09:03:32PM +0200, Paul B Mahol wrote: > On Mon, Aug 5, 2019 at 8:55 PM Michael Niedermayer > wrote: > > > On Mon, Aug 05, 2019 at 05:57:47PM +0200, Paul B Mahol wrote: > > > On Mon, Aug 5, 2019 at 5:45 PM Michael Niedermayer > > > > > wrote: > > > > > > > On Mon, Aug 05,

Re: [FFmpeg-devel] [PATCH] avutil/mips: refactor msa SLDI_Bn_0 and SLDI_Bn macros.

2019-08-06 Thread Shiyou Yin
>-Original Message- >From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] >On Behalf Of gxw >Sent: Tuesday, August 6, 2019 11:38 AM >To: ffmpeg-devel@ffmpeg.org >Subject: [FFmpeg-devel] [PATCH] avutil/mips: refactor msa SLDI_Bn_0 and >SLDI_Bn macros. > >Changing

Re: [FFmpeg-devel] [PATCH, v2 2/2] lavc/vaapi_decode: recreate hw_frames_ctx for vp9 without destroy va_context

2019-08-06 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Hendrik Leppkes > Sent: Tuesday, August 6, 2019 16:27 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH, v2 2/2] lavc/vaapi_decode: recrea

Re: [FFmpeg-devel] [PATCH v3] avformat/rtpdec_rfc4175: Fix incorrect copy_offset calculation

2019-08-06 Thread Michael Niedermayer
On Tue, Aug 06, 2019 at 05:42:22AM +, Jacob Siddall wrote: > > On Mon, Aug 05, 2019 at 06:05:29PM +0100, Kieran Kunhya wrote: > > > On Mon, 5 Aug 2019 at 17:10, Michael Niedermayer > > > wrote: > > > > > > > On Thu, Jun 27, 2019 at 06:06:22AM +, Jacob Siddall wrote: > > > > > The previous

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: fix PCR generation intervals

2019-08-06 Thread Andreas Håkon
Hi Marton, ‐‐‐ Original Message ‐‐‐ On Tuesday, 6 de August de 2019 0:20, Marton Balint wrote: > On Mon, 5 Aug 2019, Andreas Håkon wrote: > > > Hi Marton, > > ‐‐‐ Original Message ‐‐‐ > > On Sunday, 4 de August de 2019 22:30, Marton Balint c...@passwd.hu wrote: > > > > > PCR gen

Re: [FFmpeg-devel] [PATCH, v2 2/2] lavc/vaapi_decode: recreate hw_frames_ctx for vp9 without destroy va_context

2019-08-06 Thread Hendrik Leppkes
On Thu, Jul 11, 2019 at 10:20 AM Linjie Fu wrote: > > VP9 allows resolution changes per frame. Currently in VAAPI, resolution > changes leads to va context destroy and reinit. This will cause > reference frame surface lost and produce garbage. > > Though refs surface id could be passed to media dr

Re: [FFmpeg-devel] [PATCH, v2] lavc/vaapi_encode: grow packet if vaMapBuffer returns multiple buffers

2019-08-06 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Fu, Linjie > Sent: Tuesday, June 4, 2019 17:11 > To: ffmpeg-devel@ffmpeg.org > Cc: Mark Thompson > Subject: Re: [FFmpeg-devel] [PATCH, v2] lavc/vaapi_encode: grow packet if > vaMapBuffer retur

Re: [FFmpeg-devel] [PATCH 2/3] avformat/assenc: optimize writing dialogs

2019-08-06 Thread Paul B Mahol
On Mon, Aug 5, 2019 at 11:35 PM Marton Balint wrote: > An example where the preivously added API can be useful. > > Also fixes ticket #6390. > > Signed-off-by: Marton Balint > --- > libavformat/assenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/assenc.