Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread Anshul
On 09/29/2015 12:46 PM, Paul B Mahol wrote: On 9/29/15, Ray Cole wrote: I hope this is the appropriate place to propose an enhancement. This is the first time I've offered up code to any open source project...so be gentle :-) First, I love ffmpeg. Wonderful

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

2015-09-29 Thread Paul B Mahol
On 9/25/15, Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 10:18 PM, Ganesh Ajjanagadde > wrote: >> This fixes -Wshift-negative-value reported with clang 3.7+, e.g >>

Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread Paul B Mahol
On 9/29/15, Ray Cole wrote: > I hope this is the appropriate place to propose an enhancement. This is the > first time I've offered up code to any open source project...so be gentle > :-) > > First, I love ffmpeg. Wonderful software and thank you for your efforts. >

Re: [FFmpeg-devel] Adding Closed Caption support to Decklink output

2015-09-29 Thread Carl Eugen Hoyos
Deron pagestream.org> writes: > For this to work, I have to output 10 bit YUV ‘v210’ > 4:2:2 representation (twelve 10-bit unsigned components > packed into four 32-bit little-endian words). Can't you use the v210 encoder to produce this format? Carl Eugen

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-29 Thread Lou Logan
On Sun, Sep 27, 2015, at 08:55 AM, Stefano Sabatini wrote: > > + 0001-ffmpeg-web-index-add-news-describing-GSoC-2015-progr.patch > 10k (text/x-diff) LGTM. Thanks for writing this. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-29 Thread Stefano Sabatini
On date Monday 2015-09-28 22:20:53 -0800, Lou Logan encoded: > On Sun, Sep 27, 2015, at 08:55 AM, Stefano Sabatini wrote: > > > > + 0001-ffmpeg-web-index-add-news-describing-GSoC-2015-progr.patch > > 10k (text/x-diff) > > LGTM. Thanks for writing this. Thanks all for the reviews, finally

Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread Carl Eugen Hoyos
Ray Cole gmail.com> writes: > I hope this is the appropriate place to propose an enhancement. Yes but please read https://ffmpeg.org/developer.html Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH v2 3/8] libkvazaar: Remove unnecessary NULL checks

2015-09-29 Thread Arttu Ylä-Outinen
Signed-off-by: Arttu Ylä-Outinen --- libavcodec/libkvazaar.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c index 7430e0a..aaaf1f7 100644 --- a/libavcodec/libkvazaar.c +++

[FFmpeg-devel] [PATCH v2 7/8] libkvazaar: Fix setting framerate

2015-09-29 Thread Arttu Ylä-Outinen
The divisor and dividend in the equation had been swapped, making the result the inverse of the actual framerate. Signed-off-by: Arttu Ylä-Outinen --- v2: Fix error in the commit message (bitrate --> framerate). --- libavcodec/libkvazaar.c |2 +- 1 file changed, 1

[FFmpeg-devel] [PATCH v2 8/8] doc/encoders: Fix libkvazaar documentation

2015-09-29 Thread Arttu Ylä-Outinen
The -threads option is ignored with libkvazaar since it does not have any of the AV_CODEC_CAP_{FRAME,SLICE,AUTO}_THREADS capabilities. This commit removes the incorrect documentation as well as the no-op of setting the number of threads in libkvazaar encoder. Signed-off-by: Arttu Ylä-Outinen

[FFmpeg-devel] [PATCH v2 4/8] libkvazaar: Replace asserts with proper errors

2015-09-29 Thread Arttu Ylä-Outinen
Changes function libkvazaar_encode to return proper error codes instead of crashing when the video dimensions or pixel format change in the middle of encoding. Signed-off-by: Arttu Ylä-Outinen --- libavcodec/libkvazaar.c | 24 +--- 1 file changed,

Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread Ray Cole
Thank you. I know I have a number of coding style things to clean up before this could be accepted (as well as removing some output I'm logging as info) but perhaps those familiar with the decimate filter can see if the changes being proposed make sense. -- Ray On 09/29/2015 07:32 AM, compn

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: Fix memory leak

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 12:29:52PM +0200, Przemysław Sobala wrote: > W dniu 29.09.2015 o 12:22, Hendrik Leppkes pisze: > >On Tue, Sep 29, 2015 at 12:14 PM, Przemysław Sobala > > wrote: > >>Fixes #4886 > >>--- > >> libavformat/img2dec.c | 6 +- > >> 1 file

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

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 08:08:54AM -0400, Ganesh Ajjanagadde wrote: > On Tue, Sep 29, 2015 at 4:11 AM, Paul B Mahol wrote: > > On 9/25/15, Ganesh Ajjanagadde wrote: > >> On Sat, Sep 19, 2015 at 10:18 PM, Ganesh Ajjanagadde > >>

[FFmpeg-devel] [PATCH] lavf/img2dec: Fix memory leak

2015-09-29 Thread Przemysław Sobala
Fixes #4886 --- libavformat/img2dec.c | 32 +++- 1 file changed, 23 insertions(+), 9 deletions(-) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index 70f0b09..2c0fd9a 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -444,14 +444,17 @@ int

[FFmpeg-devel] [PATCH v2 6/8] libkvazaar: Use av_image_copy for copying pixels

2015-09-29 Thread Arttu Ylä-Outinen
Replaces a for loop for copying pixels by a call to av_image_copy. Signed-off-by: Arttu Ylä-Outinen --- libavcodec/libkvazaar.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/libavcodec/libkvazaar.c

[FFmpeg-devel] [PATCH v2 5/8] libkvazaar: Set pts and dts

2015-09-29 Thread Arttu Ylä-Outinen
Signed-off-by: Arttu Ylä-Outinen --- libavcodec/libkvazaar.c |9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c index a15700a..9c59cad 100644 --- a/libavcodec/libkvazaar.c +++

Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread compn
On Mon, 28 Sep 2015 21:31:59 -0500 Ray Cole wrote: > This is the first time I've offered up code to any open source > project...so be gentle :-) ehe, patches are what the devs want, either git diff or diff -u... i've downloaded and diff'ed your filter to ffmpeg

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

2015-09-29 Thread Ganesh Ajjanagadde
On Tue, Sep 29, 2015 at 9:08 AM, Michael Niedermayer wrote: > On Tue, Sep 29, 2015 at 08:08:54AM -0400, Ganesh Ajjanagadde wrote: >> On Tue, Sep 29, 2015 at 4:11 AM, Paul B Mahol wrote: >> > On 9/25/15, Ganesh Ajjanagadde wrote: >> >>

Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread wm4
On Tue, 29 Sep 2015 06:42:53 -0500 Stephen Cole wrote: > Thank you for the responses. I assumed the comments would suffice for > discussing it initially so one wouldn't have to interpret the patch. I'll > produce a diff and resubmit. A patch is _much_ easier to

[FFmpeg-devel] [PATCH v2 0/8] libkvazaar improvements

2015-09-29 Thread Arttu Ylä-Outinen
These patches fix some problems in libkvazaar encoder and update it to work with the latest git version of Kvazaar. Most notable changes are setting pts, dts and keyframe flag on the output packets and fixing the calculation of the framerate. v2: Add libkvazaar version check and fix descriptions

[FFmpeg-devel] [PATCH v2 1/8] libkvazaar: Update to work with the latest version

2015-09-29 Thread Arttu Ylä-Outinen
Function encoder_encode in Kvazaar API was changed to have new output parameters: source picture and frame info. Frame info is used to set the keyframe flag and source picture is ignored. Signed-off-by: Arttu Ylä-Outinen --- libavcodec/libkvazaar.c | 16

[FFmpeg-devel] [PATCH v2 2/8] configure: Add version check for libkvazaar

2015-09-29 Thread Arttu Ylä-Outinen
Signed-off-by: Arttu Ylä-Outinen --- v2: Add this patch. --- configure |2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index b572d59..30576c1 100755 --- a/configure +++ b/configure @@ -5272,6 +5272,8 @@ enabled libgsm&& { for

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

2015-09-29 Thread Hendrik Leppkes
On Sun, Sep 20, 2015 at 4:18 AM, Ganesh Ajjanagadde wrote: > 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

Re: [FFmpeg-devel] [PATCH v2 2/8] configure: Add version check for libkvazaar

2015-09-29 Thread Clément Bœsch
On Tue, Sep 29, 2015 at 04:29:09PM +0300, Arttu Ylä-Outinen wrote: > Signed-off-by: Arttu Ylä-Outinen > --- > v2: Add this patch. > --- > configure |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/configure b/configure > index b572d59..30576c1 100755 > ---

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

2015-09-29 Thread Ganesh Ajjanagadde
On Tue, Sep 29, 2015 at 4:11 AM, Paul B Mahol wrote: > On 9/25/15, Ganesh Ajjanagadde wrote: >> On Sat, Sep 19, 2015 at 10:18 PM, Ganesh Ajjanagadde >> wrote: >>> This fixes -Wshift-negative-value reported with clang 3.7+, e.g

Re: [FFmpeg-devel] [PATCH]select attribute of tee pseudo demuxer may contain multiple stream specifiers

2015-09-29 Thread Bodecs Bela
ping 2015.09.25. 13:28 keltezéssel, Bodecs Bela írta: Hi All, currently, select option of tee pseudo muxer may contain only one stream specifier. Sometimes I need to use more than one stream specifier. So I made the following patch. It makes possible to put multiple stream specifier into

[FFmpeg-devel] [PATCH] lavf/img2dec: Fix memory leak

2015-09-29 Thread Przemysław Sobala
Fixes #4886 --- libavformat/img2dec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index 70f0b09..fcd2b76 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -444,8 +444,12 @@ int

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: Fix memory leak

2015-09-29 Thread Hendrik Leppkes
On Tue, Sep 29, 2015 at 12:14 PM, Przemysław Sobala wrote: > Fixes #4886 > --- > libavformat/img2dec.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c > index 70f0b09..fcd2b76 100644 > ---

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: Fix memory leak

2015-09-29 Thread Przemysław Sobala
W dniu 29.09.2015 o 12:22, Hendrik Leppkes pisze: On Tue, Sep 29, 2015 at 12:14 PM, Przemysław Sobala wrote: Fixes #4886 --- libavformat/img2dec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/img2dec.c

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

2015-09-29 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 11:39 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] Proposed vf_decimate enhancement

2015-09-29 Thread Stephen Cole
Thank you for the responses. I assumed the comments would suffice for discussing it initially so one wouldn't have to interpret the patch. I'll produce a diff and resubmit. > On Sep 29, 2015, at 2:16 AM, Paul B Mahol wrote: > >> On 9/29/15, Ray Cole

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

2015-09-29 Thread Ronald S. Bultje
Hi, On Tue, Sep 29, 2015 at 11:04 AM, Ganesh Ajjanagadde wrote: > On Tue, Sep 29, 2015 at 9:24 AM, Hendrik Leppkes > wrote: > > On Sun, Sep 20, 2015 at 4:18 AM, Ganesh Ajjanagadde > > wrote: > >> This fixes -Wshift-negative-value

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

2015-09-29 Thread Ganesh Ajjanagadde
On Tue, Sep 29, 2015 at 9:24 AM, Hendrik Leppkes wrote: > On Sun, Sep 20, 2015 at 4:18 AM, Ganesh Ajjanagadde > wrote: >> This fixes -Wshift-negative-value reported with clang 3.7+, e.g >>

Re: [FFmpeg-devel] Proposed vf_decimate enhancement

2015-09-29 Thread Ray Cole
Here is an updated patch. I cleaned the code up to hopefully be closer to standards. It works well for me, but your mileage may vary... --- vf_decimate.c 2015-09-29 10:56:46.171698492 -0500 +++ vf_decimatex.c 2015-09-29 10:59:50.679695685 -0500 @@ -27,6 +27,7 @@ #define INPUT_MAIN

Re: [FFmpeg-devel] PATCH: gdigrab work for DPI in windows

2015-09-29 Thread wm4
On Wed, 23 Sep 2015 12:04:43 -0600 Roger Pack wrote: > From 6a972dda58bd5ab31524cd4e5326b4bcdeaeaa8c Mon Sep 17 00:00:00 2001 > From: rogerdpack > Date: Wed, 23 Sep 2015 12:03:27 -0600 > Subject: [PATCH] gdigrab: grab right desktop size if DPI in

Re: [FFmpeg-devel] [PATCH] fate: Add basic license header check

2015-09-29 Thread Michael Niedermayer
On Sun, Sep 27, 2015 at 02:06:54PM +0200, Michael Niedermayer wrote: > From: Michael Niedermayer > > --- > tests/Makefile |1 + > tests/fate-run.sh |5 + > tests/fate/source-check.sh | 20 > tests/fate/source.mak

Re: [FFmpeg-devel] [PATCH] avformat/movenc: suppress -Wstrict-overflow warnings

2015-09-29 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 9:39 PM, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 9:18 PM, Michael Niedermayer wrote: >> On Sun, Sep 27, 2015 at 01:23:03PM -0400, Ganesh Ajjanagadde wrote: >>> On Sun, Sep 27, 2015 at 12:58 PM, Michael Niedermayer

Re: [FFmpeg-devel] PATCH: gdigrab work for DPI in windows

2015-09-29 Thread Roger Pack
On 9/23/15, Roger Pack wrote: > See attached, tested locally. > Cheers! Bump ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] x86/hevc_sao: move 10/12bit functions into a separate file

2015-09-29 Thread James Almer
Sorry for the attachment. git send-email is giving me an unusual error when i try to send this. >From 9e8ec4d51566cdda677b15e50240e8842ec6cd34 Mon Sep 17 00:00:00 2001 From: James Almer Date: Mon, 28 Sep 2015 00:58:01 -0300 Subject: [PATCH] x86/hevc_sao: move 10/12bit functions

Re: [FFmpeg-devel] [PATCH] doc: fix spelling errors

2015-09-29 Thread Paul B Mahol
On 9/29/15, Moritz Barsnick wrote: > On Tue, Sep 29, 2015 at 20:34:09 +0200, Andreas Cadhalpun wrote: >> Signed-off-by: Andreas Cadhalpun >> --- >> doc/encoders.texi | 2 +- >> doc/filters.texi | 14

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

2015-09-29 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_setfps.c | 119 +++ 3 files changed, 121 insertions(+) create mode 100644 libavfilter/vf_setfps.c diff --git

Re: [FFmpeg-devel] [PATCH] videotoolbox: require hardware acceleration

2015-09-29 Thread wm4
On Mon, 28 Sep 2015 22:24:14 +0200 wm4 wrote: > From: Stefano Pigozzi > > VideoToolbox also implements a software decoder for h264, and will fallback to > using it if the file cannot be doceded on the CPU. In these cases though > we want the

Re: [FFmpeg-devel] [PATCH 6/7] dnxhddata: deduplicate table

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 08:29:32PM +0200, Christophe Gisquet wrote: > 2015-09-28 11:21 GMT+02:00 Christophe Gisquet : > > Please ignore for now. > > The specs explicitly says it is the same for luma and chroma. Here's a > patch that confidently reflects that. > > --

Re: [FFmpeg-devel] [PATCH] doc: fix spelling errors

2015-09-29 Thread Moritz Barsnick
On Tue, Sep 29, 2015 at 20:34:09 +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > doc/encoders.texi | 2 +- > doc/filters.texi | 14 +++--- > libavcodec/g726.c | 2 +- >

[FFmpeg-devel] [PATCH] libavformat/mov.c: Add parsing for DDTS atom for DTS audio

2015-09-29 Thread Shawn Singh
The DDTS atom is defined in ETSI TS 102 114, v1.4.1, Annex E. This is useful for DTS-HD formats, some of which cannot be decoded by dcadec.c or libdcadec. Signed-off-by: Shawn Singh --- libavformat/mov.c | 56 +++ 1 file

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

2015-09-29 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

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread James Darnley
On 2015-09-29 21:56, Clément Bœsch wrote: > On Tue, Sep 29, 2015 at 09:21:53PM +0200, Hendrik Leppkes wrote: >> I agree, we have patchcheck for typo checking. > > A lot of people do not run patchcheck (I personally never do, and given > that we fix typo on a regular basis I'm probably not the

[FFmpeg-devel] [PATCH] avcodec/apedec: fix bug introduced in commit d3e5fbb1406995e07fccbff3ca8c1e24f57a1f7b

2015-09-29 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavcodec/apedec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c index 7b34d26..4214ea6 100644 --- a/libavcodec/apedec.c +++ b/libavcodec/apedec.c @@ -1281,7 +1281,7 @@

Re: [FFmpeg-devel] [PATCHv4] ffplay: add support for interactive volume control

2015-09-29 Thread Ganesh Ajjanagadde
On Sun, Sep 27, 2015 at 5:09 PM, Marton Balint wrote: > > On Sun, 27 Sep 2015, Ganesh Ajjanagadde wrote: > >> This is a feature heavily inspired by the mpv player. At the moment, >> methods >> for adjusting volume in ffplay are rather clumsy: either one needs to set >> it >>

Re: [FFmpeg-devel] [PATCH] avcodec/apedec: fix bug introduced in commit d3e5fbb1406995e07fccbff3ca8c1e24f57a1f7b

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 07:27:03PM -0400, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/apedec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) breaks fate --- ./tests/ref/fate/lossless-monkeysaudio-399 2015-09-29

Re: [FFmpeg-devel] [PATCH] x86/hevc_sao: move 10/12bit functions into a separate file

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 01:40:51PM -0300, James Almer wrote: > Sorry for the attachment. git send-email is giving me an unusual error when > i try to send this. > Makefile |3 > hevc_sao.asm | 394 ++-- > hevc_sao_10bit.asm | 433

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: Add parsing for DDTS atom for DTS audio

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 02:44:38PM -0700, Shawn Singh wrote: > The DDTS atom is defined in ETSI TS 102 114, v1.4.1, Annex E. > This is useful for DTS-HD formats, some of which cannot be > decoded by dcadec.c or libdcadec. > > Signed-off-by: Shawn Singh > --- >

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: Add parsing for DDTS atom for DTS audio

2015-09-29 Thread Yusuke Nakamura
2015-09-30 6:44 GMT+09:00 Shawn Singh : > The DDTS atom is defined in ETSI TS 102 114, v1.4.1, Annex E. > This is useful for DTS-HD formats, some of which cannot be > decoded by dcadec.c or libdcadec. > How useful? DTS audio has scalability, and 'ddts' box indicates only

Re: [FFmpeg-devel] [PATCH] winrt: multithreading support

2015-09-29 Thread Michael Niedermayer
On Wed, Sep 30, 2015 at 02:18:29AM +0800, wang-bin wrote: > --- > compat/w32pthreads.h | 11 +++ > configure| 4 > libavutil/cpu.c | 11 ++- > 3 files changed, 25 insertions(+), 1 deletion(-) with minng64: compat/w32pthreads.h:40:55: error: missing binary

Re: [FFmpeg-devel] [PATCH] doc: fix spelling errors

2015-09-29 Thread Lou Logan
On Tue, 29 Sep 2015 20:34:09 +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > doc/encoders.texi | 2 +- > doc/filters.texi | 14 +++--- > libavcodec/g726.c | 2 +- >

Re: [FFmpeg-devel] [PATCH] avcodec/apedec: fix bug introduced in commit d3e5fbb1406995e07fccbff3ca8c1e24f57a1f7b

2015-09-29 Thread Ganesh Ajjanagadde
On Tue, Sep 29, 2015 at 9:08 PM, Michael Niedermayer wrote: > On Tue, Sep 29, 2015 at 07:27:03PM -0400, Ganesh Ajjanagadde wrote: >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/apedec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH] libx264: copy A53 closed captions from source

2015-09-29 Thread DeHackEd
Assumes 'GA94' format (ATSC standard) Signed-off-by: DHE --- doc/encoders.texi| 4 libavcodec/libx264.c | 45 + 2 files changed, 49 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 3550bcc..f2d46dc

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: Fix memory leak

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 03:25:07PM +0200, Przemysław Sobala wrote: > Fixes #4886 > --- > libavformat/img2dec.c | 32 +++- > 1 file changed, 23 insertions(+), 9 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] videotoolbox: require hardware acceleration

2015-09-29 Thread wm4
On Mon, 28 Sep 2015 22:24:14 +0200 wm4 wrote: > From: Stefano Pigozzi > > VideoToolbox also implements a software decoder for h264, and will fallback to > using it if the file cannot be doceded on the CPU. In these cases though > we want the

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

2015-09-29 Thread Michael Niedermayer
On Tue, Sep 29, 2015 at 08:09:44AM -0400, Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 11:39 AM, Ganesh Ajjanagadde > wrote: > > This silences some of the -Wunused-function warnings when compiled with > > --disable-mmx, e.g > >

[FFmpeg-devel] [PATCH] doc: fix spelling errors

2015-09-29 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- doc/encoders.texi | 2 +- doc/filters.texi | 14 +++--- libavcodec/g726.c | 2 +- libavcodec/vaapi_hevc.c| 2 +- libavfilter/x86/vf_removegrain.asm |

[FFmpeg-devel] [PATCH] winrt: multithreading support

2015-09-29 Thread wang-bin
--- compat/w32pthreads.h | 11 +++ configure| 4 libavutil/cpu.c | 11 ++- 3 files changed, 25 insertions(+), 1 deletion(-) diff --git a/compat/w32pthreads.h b/compat/w32pthreads.h index 87e816f..9828e8a 100644 --- a/compat/w32pthreads.h +++

Re: [FFmpeg-devel] [PATCH 6/7] dnxhddata: deduplicate table

2015-09-29 Thread Christophe Gisquet
2015-09-28 11:21 GMT+02:00 Christophe Gisquet : > Please ignore for now. The specs explicitly says it is the same for luma and chroma. Here's a patch that confidently reflects that. -- Christophe From bfbb0bdde4eee8d44d9c158e04b50fb4177c428f Mon Sep 17 00:00:00

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

2015-09-29 Thread Ganesh Ajjanagadde
On Tue, Sep 29, 2015 at 11:22 AM, Ronald S. Bultje wrote: > Hi, > > On Tue, Sep 29, 2015 at 11:04 AM, Ganesh Ajjanagadde > wrote: > >> On Tue, Sep 29, 2015 at 9:24 AM, Hendrik Leppkes >> wrote: >> > On Sun, Sep 20, 2015 at 4:18 AM,

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread Hendrik Leppkes
On Tue, Sep 29, 2015 at 9:08 PM, wm4 wrote: > On Tue, 29 Sep 2015 20:57:31 +0200 > Michael Niedermayer wrote: > >> From: Michael Niedermayer >> >> Signed-off-by: Michael Niedermayer >> --- >>

[FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- tests/fate/source-check.sh |8 tests/ref/fate/source |3 +++ 2 files changed, 11 insertions(+) diff --git a/tests/fate/source-check.sh

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread wm4
On Tue, 29 Sep 2015 20:57:31 +0200 Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > tests/fate/source-check.sh |8 > tests/ref/fate/source |3 +++ > 2

[FFmpeg-devel] [PATCH 1/2] compat/avisynth/avisynth_c.h: fix typos

2015-09-29 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- compat/avisynth/avisynth_c.h |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/compat/avisynth/avisynth_c.h b/compat/avisynth/avisynth_c.h index

Re: [FFmpeg-devel] [PATCH v2 2/8] configure: Add version check for libkvazaar

2015-09-29 Thread Carl Eugen Hoyos
Clément Bœsch pkh.me> writes: > > + { check_cpp_condition kvazaar.h "KVZ_API_VERSION >= 7" || > > + die "ERROR: kvazaar API version must be at least 7."; } > > Can you do something like require_pkg_config "kvazaar >= 1.2.3" instead? Please don't! This only adds an

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Force one stream for hevc and m4v

2015-09-29 Thread Carl Eugen Hoyos
Michael Niedermayer gmx.at> writes: > > rawenc.c |2 ++ > > 1 file changed, 2 insertions(+) > > 152253d1707fbee45355abd6364a9ecc592942c8 patchonestream.diff > > LGTM Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread Clément Bœsch
On Tue, Sep 29, 2015 at 09:21:53PM +0200, Hendrik Leppkes wrote: > On Tue, Sep 29, 2015 at 9:08 PM, wm4 wrote: > > On Tue, 29 Sep 2015 20:57:31 +0200 > > Michael Niedermayer wrote: > > > >> From: Michael Niedermayer > >> > >>

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread Hendrik Leppkes
On Tue, Sep 29, 2015 at 9:56 PM, Clément Bœsch wrote: > On Tue, Sep 29, 2015 at 09:21:53PM +0200, Hendrik Leppkes wrote: >> On Tue, Sep 29, 2015 at 9:08 PM, wm4 wrote: >> > On Tue, 29 Sep 2015 20:57:31 +0200 >> > Michael Niedermayer wrote:

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread James Almer
On 9/29/2015 5:14 PM, Hendrik Leppkes wrote: > Typos should not be fixed in the merge. Either review and fix them > before they get commited in Libav, or fix them in a separate commit > afterwards. > If I have to start fixing every typo in every commit that I merge from > Libav, then I'm going to

Re: [FFmpeg-devel] [PATCH v2 2/8] configure: Add version check for libkvazaar

2015-09-29 Thread Hendrik Leppkes
On Tue, Sep 29, 2015 at 9:39 PM, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > >> > + { check_cpp_condition kvazaar.h "KVZ_API_VERSION >= 7" || >> > + die "ERROR: kvazaar API version must be at least 7."; } >> >> Can you do something like

Re: [FFmpeg-devel] [PATCH v2 2/8] configure: Add version check for libkvazaar

2015-09-29 Thread Clément Bœsch
On Tue, Sep 29, 2015 at 07:39:05PM +, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > > + { check_cpp_condition kvazaar.h "KVZ_API_VERSION >= 7" || > > > + die "ERROR: kvazaar API version must be at least 7."; } > > > > Can you do something like

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-09-29 Thread Timo Rothenpieler
> Signed-off-by: Timo Rothenpieler > --- > doc/filters.texi | 5 + > libavfilter/chromakey_opencl_kernel.h | 98 +++ > libavfilter/opencl_allkernels.c | 2 + > libavfilter/vf_chromakey.c| 179 >

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread James Almer
On 9/29/2015 4:21 PM, Hendrik Leppkes wrote: > On Tue, Sep 29, 2015 at 9:08 PM, wm4 wrote: >> On Tue, 29 Sep 2015 20:57:31 +0200 >> Michael Niedermayer wrote: >> >>> From: Michael Niedermayer >>> >>> Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread Hendrik Leppkes
On Tue, Sep 29, 2015 at 10:11 PM, James Almer wrote: > On 9/29/2015 4:21 PM, Hendrik Leppkes wrote: >> On Tue, Sep 29, 2015 at 9:08 PM, wm4 wrote: >>> On Tue, 29 Sep 2015 20:57:31 +0200 >>> Michael Niedermayer wrote: >>> From: