Re: [FFmpeg-devel] [PATCH] ffmpeg: Don't offer H.264 compatibility warning for NV12 input

2017-06-14 Thread Gyan
On Thu, Jun 15, 2017 at 4:59 AM, wm4 wrote: > > > av_log(NULL, AV_LOG_WARNING, > > "No pixel format specified, %s for H.264 encoding > chosen.\n" > > "Use -pix_fmt yuv420p for compatibility with > outdated media players.\n", > > This warning sh

Re: [FFmpeg-devel] [PATCH] avcodec/aacps: move checks for valid length outside the stereo_interpolate dsp function

2017-06-14 Thread James Almer
On 6/3/2017 2:04 PM, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/aacps.c | 7 --- > libavcodec/x86/aacpsdsp.asm | 6 -- > 2 files changed, 4 insertions(+), 9 deletions(-) I'll push this soon if nobody comments. _

[FFmpeg-devel] [PATCH] avcodec/lpc: signed integer overflow in compute_lpc_coefs() (aacdec_fixed)

2017-06-14 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: -1575818955 + -915383657 cannot be represented in type 'int' Fixes: 2224/clusterfuzz-testcase-minimized-6208559949807616 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niede

Re: [FFmpeg-devel] [PATCH] ffmpeg: Don't offer H.264 compatibility warning for NV12 input

2017-06-14 Thread wm4
On Wed, 14 Jun 2017 23:03:39 +0100 Mark Thompson wrote: > It's also 8-bit YUV 4:2:0. > --- > Most visible with streams downloaded from hardware to encode properly with > libx264, which will typically be NV12. > > > ffmpeg.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff -

Re: [FFmpeg-devel] [PATCH 1/3] checkasm: add a checkasm_checked_call function that doesn't issue emms

2017-06-14 Thread James Almer
On 6/6/2017 12:05 PM, James Almer wrote: > On 6/1/2017 8:10 PM, James Almer wrote: >> Meant for DSP functions returning a float or double, as they'd fail if emms >> is called after every run on x86_32. >> >> Signed-off-by: James Almer >> --- >> tests/checkasm/checkasm.h | 11 +++ >>

Re: [FFmpeg-devel] [PATCH] hevc: Fix scaling list prediction delta for the 32x32 inter matrix

2017-06-14 Thread Mark Thompson
On 15/05/17 23:44, Mark Thompson wrote: > On 15/05/17 22:46, James Almer wrote: >> On 5/15/2017 5:29 PM, Mark Thompson wrote: >>> On 30/04/17 13:34, Mark Thompson wrote: Fixes ticket #6356. --- This was broken in 627c044f50da3046809314f7cc742b8a10cf26a1, which changed the array

Re: [FFmpeg-devel] [PATCH 00/24] Generic hardware device setup and miscellaneous related merges

2017-06-14 Thread Mark Thompson
On 13/06/17 13:07, wm4 wrote: > On Mon, 12 Jun 2017 23:40:17 +0100 > Mark Thompson wrote: > >> This merges a set of stuff from libav to do with hardware codecs/processing. > > All patches LGTM. I don't think it makes sense to delay pushing those > either. Set applied (with some fixups from Mich

[FFmpeg-devel] [PATCH] ffmpeg: Don't offer H.264 compatibility warning for NV12 input

2017-06-14 Thread Mark Thompson
It's also 8-bit YUV 4:2:0. --- Most visible with streams downloaded from hardware to encode properly with libx264, which will typically be NV12. ffmpeg.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index 6170bd453c..e6e8b9e119 100644 --- a/ffmpeg.c

Re: [FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix build with Xcode 7.2

2017-06-14 Thread Matthieu Bouron
On Wed, Jun 14, 2017 at 06:17:54PM +0200, Matthieu Bouron wrote: > On Wed, Jun 14, 2017 at 01:39:12PM +0200, Matthieu Bouron wrote: > > --- > > libavcodec/aarch64/simple_idct_neon.S | 28 ++-- > > 1 file changed, 14 insertions(+), 14 deletions(-) > > > > diff --git a/libav

[FFmpeg-devel] [PATCH 3/3] avcodec/vorbisenc: Stop tracking number of samples per frame

2017-06-14 Thread Tyler Jones
Each frame is now padded with 0 values if not enough samples are present, and all frames are guaranteed to have exactly 1 << (venc->log2_blocksize[1] - 1) samples. Signed-off-by: Tyler Jones --- libavcodec/vorbisenc.c | 33 - 1 file changed, 16 insertions(+), 17 d

[FFmpeg-devel] [PATCH 2/3] avcodec/vorbisenc: Apply and output correct length window and mdct

2017-06-14 Thread Tyler Jones
Usage of blocksize, window, mode, and mdct indexes are switched from default 0 to a default of 1 to better align with specs. A flag of 0 should correspond with short windows, a flag of 1 with long. Signed-off-by: Tyler Jones --- libavcodec/vorbisenc.c | 25 ++--- 1 file chang

[FFmpeg-devel] [PATCH 1/3] avcodec/vorbisenc: Separate copying audio samples from windowing

2017-06-14 Thread Tyler Jones
Audio samples are shifted around when copying from the frame queue so that analysis can be done without negatively impacting calculation of the MDCT. Window coefficients are applied to the current two overlapped windows simultaneously instead of applying overlap for the next frame ahead of time. T

[FFmpeg-devel] How to handle incoming RTMP objects?

2017-06-14 Thread Ilya Evseev
Hi all! There is an incoming RTMP stream containing metainfo: $ rtmpdump -r rtmp://10.20.30.40/test1 --live -o /dev/null -V ... DEBUG: RTMP_ClientPacket, received: notify 330 bytes DEBUG: (object begin) DEBUG: (object begin) DEBUG: Property: DEBUG: Property: DEBUG: Property: DEBUG: Property:

Re: [FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix build with Xcode 7.2

2017-06-14 Thread Matthieu Bouron
On Wed, Jun 14, 2017 at 01:39:12PM +0200, Matthieu Bouron wrote: > --- > libavcodec/aarch64/simple_idct_neon.S | 28 ++-- > 1 file changed, 14 insertions(+), 14 deletions(-) > > diff --git a/libavcodec/aarch64/simple_idct_neon.S > b/libavcodec/aarch64/simple_idct_neon.S >

Re: [FFmpeg-devel] [PATCH] vp9: fix overwrite in ff_vp9_ipred_dr_16x16_16_avx2.

2017-06-14 Thread Ronald S. Bultje
Hi, On Wed, Jun 14, 2017 at 11:34 AM, James Almer wrote: > On 6/14/2017 9:46 AM, Ronald S. Bultje wrote: > > Fixes trac issue 6459. > > --- > > libavcodec/x86/vp9intrapred_16bpp.asm | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/libavcodec/x86/vp9intrapred_16bpp.asm > b/libavcode

Re: [FFmpeg-devel] [PATCH] vp9: fix overwrite in ff_vp9_ipred_dr_16x16_16_avx2.

2017-06-14 Thread James Almer
On 6/14/2017 9:46 AM, Ronald S. Bultje wrote: > Fixes trac issue 6459. > --- > libavcodec/x86/vp9intrapred_16bpp.asm | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavcodec/x86/vp9intrapred_16bpp.asm > b/libavcodec/x86/vp9intrapred_16bpp.asm > index 764f704..c651c08 100644 > --- a/lib

[FFmpeg-devel] [PATCH] vp9: fix overwrite in ff_vp9_ipred_dr_16x16_16_avx2.

2017-06-14 Thread Ronald S. Bultje
Fixes trac issue 6459. --- libavcodec/x86/vp9intrapred_16bpp.asm | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/x86/vp9intrapred_16bpp.asm b/libavcodec/x86/vp9intrapred_16bpp.asm index 764f704..c651c08 100644 --- a/libavcodec/x86/vp9intrapred_16bpp.asm +++ b/libavcodec/x86/vp9intra

[FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix build with Xcode 7.2

2017-06-14 Thread Matthieu Bouron
--- libavcodec/aarch64/simple_idct_neon.S | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/libavcodec/aarch64/simple_idct_neon.S b/libavcodec/aarch64/simple_idct_neon.S index 5bd31e5be9..5e4d021a97 100644 --- a/libavcodec/aarch64/simple_idct_neon.

Re: [FFmpeg-devel] [PATCH 13/24] vaapi_encode: Use gop_size consistently in RC parameters

2017-06-14 Thread Mark Thompson
On 14/06/17 04:12, Jun Zhao wrote: > On 2017/6/13 6:40, Mark Thompson wrote: >> The non-H.26[45] codecs already use this form. Since we don't >> currently generate I frames for codecs which support them separately >> to IDR, the p_per_i variable is set to infinity by default so that it >> doesn't

Re: [FFmpeg-devel] [PATCH] Add FITS Decoder

2017-06-14 Thread Paras Chadha
On Tue, Jun 13, 2017 at 7:45 PM, Paul B Mahol wrote: > On 6/13/17, Paras Chadha wrote: > > Above mentioned changes have been done. > > > > Signed-off-by: Paras Chadha > > --- > > Changelog | 1 + > > doc/general.texi| 2 + > > libavcodec/Makefile | 1 + > > liba