Re: [FFmpeg-devel] [PATCH 1/2] lavu/intmath.h: Add msvc/icl ctzll optimisations.

2015-10-15 Thread James Almer
On 10/15/2015 7:25 PM, Matt Oliver wrote: > ICC defines __GNUC__ but ICL does not! ICC (being the linux variant) > supports all the same functions as gcc so theres no need in this case for > special handling. However ICL defines _MSC_VER so this is correct. Ah, good to know then. >> > >>> > >

[FFmpeg-devel] [PATCH][RFC] avutil/time: add av_warn_unused_result to av_usleep

2015-10-15 Thread Ganesh Ajjanagadde
av_usleep can fail. Forcing users to check the error code here may be somewhat controversial. Signed-off-by: Ganesh Ajjanagadde --- libavutil/time.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/time.h b/libavutil/time.h index dc169b0..f55167b 100644 ---

[FFmpeg-devel] [PATCH] avutil/file: add av_warn_unused_result to av_file_map

2015-10-15 Thread Ganesh Ajjanagadde
The return code here should be checked. Signed-off-by: Ganesh Ajjanagadde --- libavutil/file.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/file.h b/libavutil/file.h index 1cae295..e931be7 100644 --- a/libavutil/file.h +++ b/libavutil/file.h @@ -40,6

[FFmpeg-devel] [PATCH] avutil/opencl_internal: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavutil/opencl_internal.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/opencl_internal.h b/libavutil/opencl_internal.h index dacd930..a4f5885 100644 --- a/libavutil/opencl_internal.h +++

[FFmpeg-devel] [PATCH] avfilter/transform: add av_warn_unused_result to avfilter_transform

2015-10-15 Thread Ganesh Ajjanagadde
This adds av_warn_unused_result to avfilter_transform to trigger warnings for non-checked return codes, currently none. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/transform.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/transform.h

[FFmpeg-devel] [PATCH] avutil/buffer: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavutil/buffer.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/buffer.h b/libavutil/buffer.h index b4399fd..6f03677 100644 --- a/libavutil/buffer.h +++ b/libavutil/buffer.h @@ -178,6 +178,7 @@ int

[FFmpeg-devel] [PATCH] avutil/internal: add av_warn_unused_result to avpriv_open

2015-10-15 Thread Ganesh Ajjanagadde
The open syscall can obviously fail, and its return code needs to be checked. Signed-off-by: Ganesh Ajjanagadde --- libavutil/internal.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/internal.h b/libavutil/internal.h index 13dbd3b..5c2cd99 100644 ---

[FFmpeg-devel] [PATCH] avutil/avstring: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavutil/avstring.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/avstring.h b/libavutil/avstring.h index a306e89..a46d012 100644 --- a/libavutil/avstring.h +++ b/libavutil/avstring.h @@ -339,6 +339,7 @@ enum

[FFmpeg-devel] [PATCH] avcodec: loongson optimize h264dsp idct and loop filter with mmi

2015-10-15 Thread 周晓勇
Hi, i am making rpm package on fedora21-loongson but find out the master contain this patch, but neither in release/2.8 or n2.8.1 git log master: commit f7e9b9d260f9c7f065791c70d9ca6b89b5164742 Author: 周晓勇 Date: Wed Sep 2 18:02:24 2015 +0800 avcodec: loongson

Re: [FFmpeg-devel] [PATCH 2/2] lavu/intmath.h: Move x86 only msvc/icl functions to x86 specific header.

2015-10-15 Thread James Almer
On 10/15/2015 10:26 PM, Matt Oliver wrote: > On 16 October 2015 at 01:10, Matt Oliver wrote: > >> > Some of the existing msvc/icl optimised versions for ctz and log2 use >> > intrinsics that are only usable on x86 systems. This patch moves them from >> > intmath.h to

Re: [FFmpeg-devel] [PATCH 1/2] lavu/intmath.h: Add msvc/icl ctzll optimisations.

2015-10-15 Thread James Almer
On 10/15/2015 11:07 AM, Matt Oliver wrote: > This patch just adds the msvc and icl equivalent ctzll optimisations to > correspond with the recently added gcc variant > > > 0001-lavu-intmath.h-Add-msvc-icl-ctzll-optimisations.patch > > > From 2b8e7757cdb8595181a01bb18756d60662c41fde Mon Sep 17

Re: [FFmpeg-devel] [PATCH] vp9: use AVFrame.buf[0] to check if a frame is valid

2015-10-15 Thread wm4
On Thu, 15 Oct 2015 08:52:33 -0400 "Ronald S. Bultje" wrote: > Hi, > > On Thu, Oct 15, 2015 at 8:02 AM, Hendrik Leppkes > wrote: > > > AVFrame.data[0] is not guaranteed to be set with a HWAccel > > --- > > libavcodec/vp9.c | 46

[FFmpeg-devel] [PATCH] swresample: slightly nicer debug output for auto matrix

2015-10-15 Thread wm4
This is the matrix that will be used for up/downmixing. --- libswresample/rematrix.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libswresample/rematrix.c b/libswresample/rematrix.c index 54ebb96..932088f 100644 --- a/libswresample/rematrix.c +++

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Paul B Mahol
Dana 15. 10. 2015. 15:44 osoba "Nicholas Robbins" < nickrobbins-at-yahoo@ffmpeg.org> napisala je: > > > On Thursday, October 15, 2015 7:18 AM, Paul B Mahol wrote: > > > > On 10/15/15, Nicolas George wrote: > >> Le quartidi 24 vendemiaire, an CCXXIV, Paul B

Re: [FFmpeg-devel] [PATCH] mips: disable all features in configure if no cpu is matched

2015-10-15 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 01:22:39PM +, Nedeljko Babic wrote: > >> > >> On the other hand, if cpu option is used with some cpu that is not > >> supported in > >> the list, configuration will break since “-march” is set to that cpu > >> value. > > > >Are you sure? What happen if you pass

Re: [FFmpeg-devel] [PATCH] swresample: slightly nicer debug output for auto matrix

2015-10-15 Thread wm4
On Thu, 15 Oct 2015 19:50:13 +0200 Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 07:07:48PM +0200, wm4 wrote: > > This is the matrix that will be used for up/downmixing. > > --- > > libswresample/rematrix.c | 9 +++-- > > 1 file changed, 7 insertions(+), 2

Re: [FFmpeg-devel] [PATCH] swresample: slightly nicer debug output for auto matrix

2015-10-15 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 07:07:48PM +0200, wm4 wrote: > This is the matrix that will be used for up/downmixing. > --- > libswresample/rematrix.c | 9 +++-- > 1 file changed, 7 insertions(+), 2 deletions(-) LGTM thanks [...] -- Michael GnuPG fingerprint:

<    1   2