[FFmpeg-devel] [PATCH] avdevice/libdc1394: silence -Wdiscarded-qualifiers

2015-09-19 Thread Ganesh Ajjanagadde
fmt, fps are non-const as they are needed to loop over a static const array. Thus the patch explicitly casts them to be non-const. This suppresses -Wdiscarded-qualifiers seen in e.g http://fate.ffmpeg.org/log.cgi?time=20150919100330=compile=x86_64-archlinux-gcc-enableshared. Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCHv3] avfilter/vf_chromakey: Add chromakey video filter

2015-09-19 Thread Lou Logan
Hi, I know the docs is similar to colorkey, but I have a few short suggestions. On Fri, 18 Sep 2015 16:27:54 +0200, Timo Rothenpieler wrote: [...] > diff --git a/doc/filters.texi b/doc/filters.texi > index 4a55e59..0446204 100644 > --- a/doc/filters.texi > +++ b/doc/filters.texi > @@ -3556,6

[FFmpeg-devel] [PATCH] avcodec/snowenc: fix discarded const warning

2015-09-19 Thread Ganesh Ajjanagadde
This fixes a -Wdiscarded-qualifiers in e.g http://fate.ffmpeg.org/log.cgi?time=20150919093218=compile=x86_64-archlinux-gcc-ddebug. It is quite clear from the code that the discard of constness was deliberate, so the cast should be fine. Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] avformat/format: silence -Wdiscarded-qualifiers

2015-09-19 Thread Ganesh Ajjanagadde
On Wed, Sep 16, 2015 at 9:08 PM, Michael Niedermayer wrote: > On Wed, Sep 16, 2015 at 06:50:54PM -0400, Ganesh Ajjanagadde wrote: >> lpd.buf is non-const and discards the const qualifier of zerobuffer. >> This fixes -Wdiscarded-qualifiers observed with GCC 5.2. >> >>

Re: [FFmpeg-devel] [PATCH] checkasm: add VP9 loopfilter tests.

2015-09-19 Thread Ronald S. Bultje
Hi, On Thu, Sep 17, 2015 at 5:58 PM, Ronald S. Bultje wrote: > The randomize_buffer() implementation assures that "most of the time", > we'll do a good mix of wide16/wide8/hev/regular/no filters for complete > code coverage. However, this is not mathematically assured

[FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread Ganesh Ajjanagadde
ffmpeg has not been using cvs for a long time. Signed-off-by: Ganesh Ajjanagadde --- doc/developer.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/developer.texi b/doc/developer.texi index 7324629..923ee69 100644 --- a/doc/developer.texi +++

[FFmpeg-devel] [PATCH 1/2] doc/developer: add note on patches that fix warnings

2015-09-19 Thread Ganesh Ajjanagadde
This adds some recommendations while submitting patches that fix warnings. Signed-off-by: Ganesh Ajjanagadde --- doc/developer.texi | 6 ++ 1 file changed, 6 insertions(+) diff --git a/doc/developer.texi b/doc/developer.texi index b8259e7..7324629 100644 ---

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/x86/hpeldsp_rnd_template: silence -Wunused-function on --disable-mmx

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 11:38:59AM -0400, Ganesh Ajjanagadde wrote: > This silences some of the -Wunused-function warnings when compiled with > --disable-mmx, e.g > http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. > Header guards are too brittle and

Re: [FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread James Almer
On 9/19/2015 8:36 PM, Ganesh Ajjanagadde wrote: > ffmpeg has not been using cvs for a long time. > > Signed-off-by: Ganesh Ajjanagadde > --- > doc/developer.texi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/doc/developer.texi

Re: [FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread James Almer
On 9/19/2015 8:54 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 7:48 PM, James Almer wrote: >> On 9/19/2015 8:36 PM, Ganesh Ajjanagadde wrote: >>> ffmpeg has not been using cvs for a long time. >>> >>> Signed-off-by: Ganesh Ajjanagadde >>> ---

Re: [FFmpeg-devel] [RFC] avformat/mxfenc: stop encoding if unfilled video packet

2015-09-19 Thread Tomas Härdin
On Wed, 2015-09-16 at 14:33 +0200, Tobias Rapp wrote: > Hi, > > attached patch fixes ticket #4759 but I guess it is a bit too hasty to > always abort transcoding if a single frame cannot be written. I guess it > would be better to check for some "exit_on_error" like flag set but > couldn't

Re: [FFmpeg-devel] [PATCH] avcodec/x86/mpegvideoenc: silence -Wunused-function on --disable-mmx

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 11:00:50AM -0400, Ganesh Ajjanagadde wrote: > This silences -Wunused-function when compiled with --disable-mmx, e.g > http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. > > Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] avresample/resample: remove unused variable

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 01:26:35PM -0400, Ganesh Ajjanagadde wrote: > This fixes a -Wunused-variable, see e.g > http://fate.ffmpeg.org/log.cgi?time=20150919162338=compile=x86_64-archlinux-gcc-threads. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavresample/resample.c

[FFmpeg-devel] [PATCH] avcodec/hevc_mp4toannexb_bsf: silence -Wdiscarded-qualifiers

2015-09-19 Thread Ganesh Ajjanagadde
*poutbuf is non-const, so this casts it explicitly. This suppresses -Wdiscarded-qualifiers seen in e.g http://fate.ffmpeg.org/log.cgi?time=20150919100330=compile=x86_64-archlinux-gcc-enableshared. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/hevc_mp4toannexb_bsf.c |

Re: [FFmpeg-devel] [PATCH] avcodec/mlpdec: fix a undefined left shift of negative number

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 01:06:46AM -0400, Ganesh Ajjanagadde wrote: > This fixes a -Wshift-negative-value reported with clang 3.7+, e.g > http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7. > > Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCHv2] configure: add -D_DEFAULT_SOURCE to silence -Wcpp

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 2:13 PM, Hendrik Leppkes wrote: > On Sat, Sep 19, 2015 at 6:07 PM, Ganesh Ajjanagadde wrote: >> On Thu, Sep 17, 2015 at 6:45 AM, Ganesh Ajjanagadde wrote: >>> On Thu, Sep 17, 2015 at 4:54 AM, Clément Bœsch

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/x86/hpeldsp_rnd_template: silence -Wunused-function on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 11:38 AM, Ganesh Ajjanagadde wrote: > This silences some of the -Wunused-function warnings when compiled with > --disable-mmx, e.g > http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. > Header guards are

Re: [FFmpeg-devel] [PATCH] avcodec/videotoolbox: fix -Wunused-but-set-variable

2015-09-19 Thread Hendrik Leppkes
On Sat, Sep 19, 2015 at 9:58 PM, Ganesh Ajjanagadde wrote: > pix_fmt was declared presumably to shorten the argument passed to the > function. > However, it is currently not being used for such a purpose. > This patch makes it used for that purpose. > This fixes

Re: [FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 8:30 PM, James Almer wrote: > On 9/19/2015 8:54 PM, Ganesh Ajjanagadde wrote: >> On Sat, Sep 19, 2015 at 7:48 PM, James Almer wrote: >>> On 9/19/2015 8:36 PM, Ganesh Ajjanagadde wrote: ffmpeg has not been using cvs for a long

Re: [FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 7:48 PM, James Almer wrote: > On 9/19/2015 8:36 PM, Ganesh Ajjanagadde wrote: >> ffmpeg has not been using cvs for a long time. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> doc/developer.texi | 2 +- >> 1 file changed, 1

[FFmpeg-devel] [PATCHv2] avcodec/videotoolbox: fix -Wunused-but-set-variable

2015-09-19 Thread Ganesh Ajjanagadde
pix_fmt was declared presumably to shorten the argument passed to the function. However, it is currently not being used for such a purpose. This patch simply removes it instead. This fixes -Wunused-but-set-variable reported at e.g:

Re: [FFmpeg-devel] [PATCH] avcodec/videotoolbox: fix -Wunused-but-set-variable

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 5:33 PM, Hendrik Leppkes wrote: > On Sat, Sep 19, 2015 at 9:58 PM, Ganesh Ajjanagadde > wrote: >> pix_fmt was declared presumably to shorten the argument passed to the >> function. >> However, it is currently not being used

Re: [FFmpeg-devel] [PATCH] checkasm: add jpeg2000dsp rct_int tests

2015-09-19 Thread Henrik Gramner
On Fri, Sep 18, 2015 at 4:55 AM, James Almer wrote: > Signed-off-by: James Almer > --- > tests/checkasm/Makefile | 1 + > tests/checkasm/checkasm.c| 3 ++ > tests/checkasm/checkasm.h| 1 + > tests/checkasm/jpeg2000dsp.c | 71 >

[FFmpeg-devel] [PATCH] avcodec/dvenc: silence -Wdiscarded-qualifiers

2015-09-19 Thread Ganesh Ajjanagadde
s->frame is non-const, so this casts them explicitly. This suppresses -Wdiscarded-qualifiers seen in e.g http://fate.ffmpeg.org/log.cgi?time=20150919100330=compile=x86_64-archlinux-gcc-enableshared. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/dvenc.c | 2 +- 1 file

Re: [FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 9:09 PM, James Almer wrote: > On 9/19/2015 10:03 PM, Ganesh Ajjanagadde wrote: >> On Sat, Sep 19, 2015 at 8:30 PM, James Almer wrote: >>> On 9/19/2015 8:54 PM, Ganesh Ajjanagadde wrote: On Sat, Sep 19, 2015 at 7:48 PM, James

[FFmpeg-devel] [PATCH] avcodec/videotoolbox: fix -Wunused-but-set-variable

2015-09-19 Thread Ganesh Ajjanagadde
pix_fmt was declared presumably to shorten the argument passed to the function. However, it is currently not being used for such a purpose. This patch makes it used for that purpose. This fixes -Wunused-but-set-variable reported at e.g:

Re: [FFmpeg-devel] [PATCHv2] avformat/format: silence -Wdiscarded-qualifiers

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 05:37:14PM -0400, Ganesh Ajjanagadde wrote: > lpd.buf is non-const and discards the const qualifier of zerobuffer. > This fixes -Wdiscarded-qualifiers observed with a variety of compilers, > including GCC 5.2. > Note that this does not change the type of zerobuffer, and

[FFmpeg-devel] [PATCH] avcodec/apedec: fix undefined left shifts of negative numbers

2015-09-19 Thread Ganesh Ajjanagadde
This fixes -Wshift-negative-value reported with clang 3.7+, e.g http://fate.ffmpeg.org/log.cgi?time=20150919172459=compile=x86_64-darwin-clang-polly-notiling-3.7. Note that the patch crucially depends on int >= 32 bits, an assumption made in many places in the codebase. Signed-off-by: Ganesh

[FFmpeg-devel] [PATCH] avfilter: add tremolo filter

2015-09-19 Thread Kyle Swanson
Signed-off-by: Kyle Swanson --- doc/filters.texi | 19 ++ libavfilter/Makefile | 1 + libavfilter/af_tremolo.c | 173 +++ libavfilter/allfilters.c | 1 + libavfilter/version.h| 2 +- 5 files changed, 195

Re: [FFmpeg-devel] [PATCHv2] configure: add -D_DEFAULT_SOURCE to silence -Wcpp

2015-09-19 Thread Hendrik Leppkes
On Sat, Sep 19, 2015 at 6:07 PM, Ganesh Ajjanagadde wrote: > On Thu, Sep 17, 2015 at 6:45 AM, Ganesh Ajjanagadde wrote: >> On Thu, Sep 17, 2015 at 4:54 AM, Clément Bœsch wrote: >>> On Wed, Sep 16, 2015 at 06:55:39PM -0400, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/vda_h264: use multichar literal portably

2015-09-19 Thread Hendrik Leppkes
On Sat, Sep 19, 2015 at 12:35 AM, Michael Niedermayer wrote: > On Fri, Sep 18, 2015 at 06:16:18PM -0400, Ganesh Ajjanagadde wrote: >> Multichar literals are implementation defined, and thus trigger -Wmultichar: >>

[FFmpeg-devel] [PATCH] hevc: properly handle no_rasl_output_flag when removing pictures from the DPB

2015-09-19 Thread Hendrik Leppkes
Fixes ticket #4185. Reviewed-By: Mickael Raulet --- libavcodec/hevc.c | 5 + libavcodec/hevc.h | 1 + libavcodec/hevc_refs.c | 2 +- 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index

Re: [FFmpeg-devel] [PATCH] avcodec/internal: silence -Wempty-body on clang

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 12:55:18AM -0400, Ganesh Ajjanagadde wrote: > This silences a -Wempty-body warning on clang 3.7+, e.g > http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7. > > Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] hevc: properly handle no_rasl_output_flag when removing pictures from the DPB

2015-09-19 Thread Carl Eugen Hoyos
Hendrik Leppkes gmail.com> writes: > Fixes ticket #4185. > > Reviewed-By: Mickael Raulet insa-rennes.fr> If he reviewed it (that's also how I remember it) please push! Thank you for looking into this issue, Carl Eugen ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/vda_h264: use multichar literal portably

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 2:27 AM, Hendrik Leppkes wrote: > On Sat, Sep 19, 2015 at 12:35 AM, Michael Niedermayer > wrote: >> On Fri, Sep 18, 2015 at 06:16:18PM -0400, Ganesh Ajjanagadde wrote: >>> Multichar literals are implementation defined, and thus

Re: [FFmpeg-devel] [PATCH] avcodec/imgconvert: Support non-planar colorspaces while padding

2015-09-19 Thread Przemysław Sobala
W dniu 19.09.2015 o 04:14, Michael Niedermayer pisze: On Fri, Sep 18, 2015 at 04:31:16PM +0200, Przemysław Sobala wrote: --- libavcodec/imgconvert.c | 99 - 1 file changed, 65 insertions(+), 34 deletions(-) diff --git a/libavcodec/imgconvert.c

Re: [FFmpeg-devel] [PATCH] hevc: properly handle no_rasl_output_flag when removing pictures from the DPB

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 11:46:09AM +0200, Hendrik Leppkes wrote: > Fixes ticket #4185. > > Reviewed-By: Mickael Raulet > --- > libavcodec/hevc.c | 5 + > libavcodec/hevc.h | 1 + > libavcodec/hevc_refs.c | 2 +- > 3 files changed, 7 insertions(+), 1

Re: [FFmpeg-devel] [PATCH] avcodec/g723_1: fix a undefined left shift of negative number

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 12:57:24AM -0400, Ganesh Ajjanagadde wrote: > This fixes a -Wshift-negative-value reported with clang 3.7+, e.g > http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7. > > Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] avcodec/x86/cavsdsp: silence -Wunuse-variable on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
This silences -Wunused-variable when compiled with --disable-mmx, e.g http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. The alternative of header guards will make it far too ugly. Signed-off-by: Ganesh Ajjanagadde ---

Re: [FFmpeg-devel] [PATCH] avcodec/x86/cavsdsp: silence -Wunuse-variable on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 10:34 AM, Ganesh Ajjanagadde wrote: > This silences -Wunused-variable when compiled with --disable-mmx, e.g > http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. > The alternative of header guards will make

[FFmpeg-devel] [PATCHv2] avcodec/ac3enc: use long long after switch to 64 bit bitrate

2015-09-19 Thread Ganesh Ajjanagadde
Commit 7404f3bdb switched bitrate to 64 bits. This triggers -Wabsolute-value on clang, e.g http://fate.ffmpeg.org/log.cgi?time=20150917122742=compile=x86_64-darwin-clang-3.7-O3. Therefore, usage of abs is changed to llabs, which is available on all of the platforms. Unfortunately, LLONG_MAX is

[FFmpeg-devel] [PATCH] cmdutils: silence unused warnings under --disable-swscale, --disable-swresample

2015-09-19 Thread Ganesh Ajjanagadde
This patch silences such warnings by placing initializations under a header guard, see e.g http://fate.ffmpeg.org/log.cgi?time=20150919095430=compile=x86_64-archlinux-gcc-disableswscale, http://fate.ffmpeg.org/log.cgi?time=20150919095048=compile=x86_64-archlinux-gcc-disableswresample for examples

[FFmpeg-devel] [PATCH] avcodec/x86/mpegaudiodsp: silence -Wunused-variable on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
This silences -Wunused-variable when compiled with --disable-mmx, e.g http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. The alternative of header guards will make it far too ugly. Signed-off-by: Ganesh Ajjanagadde ---

Re: [FFmpeg-devel] policy on "necro-bumping" patches

2015-09-19 Thread Ganesh Ajjanagadde
On Wed, Sep 16, 2015 at 8:29 PM, Michael Niedermayer wrote: > On Tue, Sep 15, 2015 at 04:54:19PM +0200, Michael Niedermayer wrote: >> On Tue, Sep 15, 2015 at 08:48:33AM -0400, Ganesh Ajjanagadde wrote: >> > On Tue, Sep 15, 2015 at 6:54 AM, Ronald S. Bultje

Re: [FFmpeg-devel] [PATCH] avcodec/ac3enc: use long long after switch to 64 bit bitrate

2015-09-19 Thread Ganesh Ajjanagadde
On Fri, Sep 18, 2015 at 5:52 PM, James Almer wrote: > On 9/18/2015 6:35 PM, Ganesh Ajjanagadde wrote: >> On Fri, Sep 18, 2015 at 5:30 PM, James Almer wrote: >>> On 9/17/2015 9:46 AM, Ganesh Ajjanagadde wrote: Commit 7404f3bdb switched bitrate to 64

[FFmpeg-devel] [PATCH] avcodec/x86/rv40dsp_init: silence -Wunused-variable on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
This silences -Wunused-variable when compiled with --disable-mmx, e.g http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. The alternative of header guards will make it far too ugly. Signed-off-by: Ganesh Ajjanagadde ---

[FFmpeg-devel] [PATCH] avcodec/x86/mpegvideoenc: silence -Wunused-function on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
This silences -Wunused-function when compiled with --disable-mmx, e.g http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/x86/mpegvideoenc.c | 4 1 file changed, 4

[FFmpeg-devel] [PATCH 1/2] avcodec/x86/hpeldsp_rnd_template: silence -Wunused-function on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
This silences some of the -Wunused-function warnings when compiled with --disable-mmx, e.g http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. Header guards are too brittle and ugly for this case. Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] avcodec/motion_est_template: fix undefined left shift of negative number

2015-09-19 Thread Ganesh Ajjanagadde
This fixes -Wshift-negative-value reported with clang 3.7+, e.g http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/motion_est_template.c | 2 +- 1 file changed, 1

[FFmpeg-devel] [PATCH] avresample/resample: remove unused variable

2015-09-19 Thread Ganesh Ajjanagadde
This fixes a -Wunused-variable, see e.g http://fate.ffmpeg.org/log.cgi?time=20150919162338=compile=x86_64-archlinux-gcc-threads. Signed-off-by: Ganesh Ajjanagadde --- libavresample/resample.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavresample/resample.c

Re: [FFmpeg-devel] [PATCH] avresample/resample: remove unused variable

2015-09-19 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 1:26 PM, Ganesh Ajjanagadde wrote: > This fixes a -Wunused-variable, see e.g > http://fate.ffmpeg.org/log.cgi?time=20150919162338=compile=x86_64-archlinux-gcc-threads. > > Signed-off-by: Ganesh Ajjanagadde > --- >

Re: [FFmpeg-devel] [PATCHv3] avfilter/vf_chromakey: Add chromakey video filter

2015-09-19 Thread Paul B Mahol
On 9/18/15, Timo Rothenpieler wrote: > --- > Changelog | 1 + > MAINTAINERS| 1 + > doc/filters.texi | 45 +++ > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/version.h |

[FFmpeg-devel] [PATCHv2] avformat/format: silence -Wdiscarded-qualifiers

2015-09-19 Thread Ganesh Ajjanagadde
lpd.buf is non-const and discards the const qualifier of zerobuffer. This fixes -Wdiscarded-qualifiers observed with a variety of compilers, including GCC 5.2. Note that this does not change the type of zerobuffer, and merely makes the intent explicit. Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] checkasm: add VP9 loopfilter tests.

2015-09-19 Thread Henrik Gramner
On Thu, Sep 17, 2015 at 5:58 PM, Ronald S. Bultje wrote: > The randomize_buffer() implementation assures that "most of the time", > we'll do a good mix of wide16/wide8/hev/regular/no filters for complete > code coverage. However, this is not mathematically assured because that

[FFmpeg-devel] [PATCH] x86/vp9mc: fix string concatenation of fullpel function names

2015-09-19 Thread James Almer
Fixes compilation with NASM Signed-off-by: James Almer --- See http://fate.ffmpeg.org/log.cgi?time=20150919155527=compile=x86_64-archlinux-gcc-nasm libavcodec/x86/vp9mc.asm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/x86/vp9mc.asm

Re: [FFmpeg-devel] [PATCH] swscale/output: fix undefined left shifts of negative numbers

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 01:10:44PM -0400, Ganesh Ajjanagadde wrote: > This fixes -Wshift-negative-value reported with clang 3.7+, e.g > http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7. > > Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 2/2] doc/developer: s/ffmpeg-cvslog/ffmpeg-devel

2015-09-19 Thread James Almer
On 9/19/2015 10:03 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 8:30 PM, James Almer wrote: >> On 9/19/2015 8:54 PM, Ganesh Ajjanagadde wrote: >>> On Sat, Sep 19, 2015 at 7:48 PM, James Almer wrote: On 9/19/2015 8:36 PM, Ganesh Ajjanagadde

[FFmpeg-devel] some thoughts on the website and warnings

2015-09-19 Thread Ganesh Ajjanagadde
Hi all, I have a suggestion for improving the FFmpeg website. I really like the way the current "About FFmpeg" webpage presents FFmpeg - the very first sentence is amazingly true. A few months back I thought this was just a PR push, but the proof of this lies in the variety of ways FFmpeg is used

Re: [FFmpeg-devel] [PATCH] avcodec/imgconvert: Support non-planar colorspaces while padding

2015-09-19 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 12:33:55PM +0200, Przemysław Sobala wrote: > W dniu 19.09.2015 o 04:14, Michael Niedermayer pisze: > >On Fri, Sep 18, 2015 at 04:31:16PM +0200, Przemysław Sobala wrote: > >>--- > >> libavcodec/imgconvert.c | 99 > >> - > >>

[FFmpeg-devel] [PATCH] swscale/output: fix undefined left shifts of negative numbers

2015-09-19 Thread Ganesh Ajjanagadde
This fixes -Wshift-negative-value reported with clang 3.7+, e.g http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7. Signed-off-by: Ganesh Ajjanagadde --- libswscale/output.c | 32

[FFmpeg-devel] [PATCH 2/3] avcodec/x86/hpeldsp_init: silence -Wunused-function on --disable-mmx

2015-09-19 Thread Ganesh Ajjanagadde
This silences some of the -Wunused-function warnings when compiled with --disable-mmx, e.g http://fate.ffmpeg.org/log.cgi?time=20150919094617=compile=x86_64-archlinux-gcc-disable-mmx. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/x86/hpeldsp_init.c | 4 1 file

Re: [FFmpeg-devel] some thoughts on the website and warnings

2015-09-19 Thread Lou Logan
On Sat, Sep 19, 2015, at 08:36 AM, Ganesh Ajjanagadde wrote: > Thus, FFmpeg can rightfully be proud of this accomplishment, and IMHO > should mention it publicly on the page. I can help/submit patches > regarding wording if others agree with this. The index page mentions "cross-platform", but if