Re: [FFmpeg-devel] [PATCH] lavc/mjpegdec: use ptrdiff_t instead of ssize_t

2016-01-28 Thread Matthieu Bouron
On Wed, Jan 27, 2016 at 11:31:04PM +0100, Michael Niedermayer wrote: > On Wed, Jan 27, 2016 at 10:01:39PM +0100, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > Fixes build on msvc. > > --- > > libavcodec/mjpegdec.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > shou

Re: [FFmpeg-devel] [PATCH] avcodec: add dvaudio parser

2016-01-28 Thread wm4
On Wed, 27 Jan 2016 14:48:17 +0100 Paul B Mahol wrote: > Hi, > > patch attached. Lots of magic numbers and I didn't test it, but looks ok otherwise. Not fascinated by putting a function implementation in a header, but it's probably ok. ___ ffmpeg-deve

Re: [FFmpeg-devel] GSoC 2016

2016-01-28 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > Is there anyone who wants to be (backup) admin for GSoC 2016 ? I'd like to help. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Carl Eugen Hoyos
Hi! Attached patch tries to address ticket #4947. (only tested with libx264) Please review, Carl Eugen diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 88406a3..c0e327e 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -781,6 +781,8 @@ FF_ENABLE_DEPRECATION_WARNINGS

Re: [FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Nicolas George
Le nonidi 9 pluviôse, an CCXXIV, Carl Eugen Hoyos a écrit : > Hi! > > Attached patch tries to address ticket #4947. > (only tested with libx264) > > Please review, Carl Eugen > diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c > index 88406a3..c0e327e 100644 > --- a/libavcodec/libx264.c >

Re: [FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Carl Eugen Hoyos
Nicolas George nsup.org> writes: > > +if (avctx->strict_std_compliance >= FF_COMPLIANCE_STRICT) > > +return AVERROR_INVALIDDATA; > > AVERROR(EINVAL) seems more correct. Same below. Changed locally. Thank you, Carl Eugen ___ ffmpe

Re: [FFmpeg-devel] GSoC 2016

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 10:22:00AM +, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > > > Is there anyone who wants to be (backup) admin for GSoC 2016 ? > > I'd like to help. ok, then please make sure that we register for GSoC & Outreachy in time. That is make sure s

[FFmpeg-devel] [PATCH 2/3] ffmpeg_dxva2: support hevc main10 decoding

2016-01-28 Thread Hendrik Leppkes
--- ffmpeg_dxva2.c | 26 +- 1 file changed, 21 insertions(+), 5 deletions(-) diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c index 905bf89..1b62bb9 100644 --- a/ffmpeg_dxva2.c +++ b/ffmpeg_dxva2.c @@ -53,6 +53,7 @@ DEFINE_GUID(DXVADDI_Intel_ModeH264_E, 0x604F8E68, 0x4951,0x4

[FFmpeg-devel] [PATCH 1/3] hevc: support Main10 decoding through dxva2

2016-01-28 Thread Hendrik Leppkes
--- libavcodec/hevc.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index c245d3b..5677f0f 100644 --- a/libavcodec/hevc.c +++ b/libavcodec/hevc.c @@ -346,7 +346,9 @@ static int set_sps(HEVCContext *s, const HEVCSPS *sps, enu

[FFmpeg-devel] [PATCH 3/3] ffmpeg_dxva2: add a profile check for hevc

2016-01-28 Thread Hendrik Leppkes
--- ffmpeg_dxva2.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c index 1b62bb9..35bedec 100644 --- a/ffmpeg_dxva2.c +++ b/ffmpeg_dxva2.c @@ -637,6 +637,12 @@ int dxva2_init(AVCodecContext *s) return AVERROR(EINVAL); } +if (s->codec_id

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

2016-01-28 Thread Paul B Mahol
Hi, 2nd version attached. From bd6be4dcf42e426eeeadc7738aa26354f6876c88 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 23 Jan 2016 17:15:53 +0100 Subject: [PATCH] avfilter: add nnedi filter Port of nnedi3 vapoursynth filter. Signed-off-by: Paul B Mahol --- configure|

Re: [FFmpeg-devel] [PATCH 2/3] ffmpeg_dxva2: support hevc main10 decoding

2016-01-28 Thread Hendrik Leppkes
2016-01-28 12:58 GMT+01:00 Hendrik Leppkes : > --- > ffmpeg_dxva2.c | 26 +- > 1 file changed, 21 insertions(+), 5 deletions(-) > > diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c > index 905bf89..1b62bb9 100644 > --- a/ffmpeg_dxva2.c > +++ b/ffmpeg_dxva2.c > @@ -53,6 +53,7 @@

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

2016-01-28 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > 2nd version attached. > +AVFilter ff_vf_nnedi = { > +.name = "nnedi", > +.description = NULL_IF_CONFIG_SMALL("Apply neural > network edge directed interpolation."), Am I correct that this is a deinterlacer? If yes, please add this to the des

Re: [FFmpeg-devel] [PATCH] Add TargetTypeValue in Matroska tag prefix

2016-01-28 Thread Michael Niedermayer
On Wed, Jan 27, 2016 at 08:00:23PM -0500, Pierre Choffet wrote: > Previously, the Matroska tag names were structured like this: > [/] > This lead to an issue when is not available: the different > levels tags overwrite each > other when they have the same name. > > This patch transforms the na

[FFmpeg-devel] mxfenc: Writes incomplete Video Line Map for progressive content

2016-01-28 Thread Sebastian Dröge
Hi everybody, since the following commit https://git.videolan.org/?p=ffmpeg.git;a=commit;h=8ed82d8174a666f80ab8834e3617cbe91ae740a9 mxfenc is only writing one value for progressive content. Unfortunately the commit comes without any explanation at all. SMPTE S377-1-2009c (the version I have) def

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-01-28 Thread Michael Niedermayer
On Wed, Jan 27, 2016 at 11:52:28PM +0100, Andreas Cadhalpun wrote: > On 27.01.2016 12:09, Michael Niedermayer wrote: > > On Tue, Jan 26, 2016 at 11:07:43PM +0100, Andreas Cadhalpun wrote: > >> configure |2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> 1edecc11203a53b6dfa3f9816583

Re: [FFmpeg-devel] [PATCH 1/3] hevc: support Main10 decoding through dxva2

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 12:58:25PM +0100, Hendrik Leppkes wrote: > --- > libavcodec/hevc.c | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay dictionary, page 1 "Used onl

Re: [FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 11:35:41AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch tries to address ticket #4947. > (only tested with libx264) > > Please review, Carl Eugen why should the errors be conditional? is there a case where the user wants her parameters to be ignored? [...] --

Re: [FFmpeg-devel] [PATCH 3/3] ffmpeg_dxva2: add a profile check for hevc

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 12:58:27PM +0100, Hendrik Leppkes wrote: > --- > ffmpeg_dxva2.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c > index 1b62bb9..35bedec 100644 > --- a/ffmpeg_dxva2.c > +++ b/ffmpeg_dxva2.c > @@ -637,6 +637,12 @@ int dxva2_ini

Re: [FFmpeg-devel] [PATCH 1/3] hevc: support Main10 decoding through dxva2

2016-01-28 Thread wm4
On Thu, 28 Jan 2016 12:58:25 +0100 Hendrik Leppkes wrote: > --- > libavcodec/hevc.c | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c > index c245d3b..5677f0f 100644 > --- a/libavcodec/hevc.c > +++ b/libavcodec/hevc.c >

Re: [FFmpeg-devel] [PATCH] x86/imdct36: use extractps inside the STORE macro

2016-01-28 Thread Michael Niedermayer
On Mon, Jan 25, 2016 at 03:41:19PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/x86/imdct36.asm | 7 +++ > 1 file changed, 7 insertions(+) fate passes with this on mingw32/64 and linux32/64 [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78

[FFmpeg-devel] [PATCH] msvc: Fix libx264 linking

2016-01-28 Thread Henrik Gramner
--- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 0936b69..ead0d2c 100755 --- a/configure +++ b/configure @@ -3552,6 +3552,7 @@ msvc_common_flags(){ -lz) echo zlib.lib ;; -lavifil32) echo vfw32.lib ;;

Re: [FFmpeg-devel] [PATCH 1/4] configure: fix mmal build dependencies

2016-01-28 Thread wm4
On Wed, 27 Jan 2016 17:17:53 +0100 wm4 wrote: > The mmal decoders do not depend on the software decoders. > --- All patches pushed. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] x86/imdct36: use extractps inside the STORE macro

2016-01-28 Thread Henrik Gramner
On Mon, Jan 25, 2016 at 7:41 PM, James Almer wrote: > +%if cpuflag(sse4) > +extractps [%3 ], %1, 0 > +extractps [%3 + %4], %1, 1 > +extractps [%3 + 2*%4], %1, 2 > +extractps [%3 + 3*%4], %1, 3 extractps with imm=0 should be replaced with movss.

Re: [FFmpeg-devel] [libav-devel] [PATCHv3] avcodec: Cineform HD Decoder

2016-01-28 Thread Vittorio Giovara
On Mon, Jan 25, 2016 at 6:15 PM, Kieran Kunhya wrote: >>> +{ >>> +CFHDContext *s = avctx->priv_data; >>> + >>> +avctx->pix_fmt = AV_PIX_FMT_YUV422P10; >> >> if the decoder supports multiple pixel formats it's better not to >> initialize the pixel format here, and wait until dec

Re: [FFmpeg-devel] [PATCH] x86/imdct36: use extractps inside the STORE macro

2016-01-28 Thread James Almer
On 1/28/2016 1:25 PM, Henrik Gramner wrote: > On Mon, Jan 25, 2016 at 7:41 PM, James Almer wrote: >> +%if cpuflag(sse4) >> +extractps [%3 ], %1, 0 >> +extractps [%3 + %4], %1, 1 >> +extractps [%3 + 2*%4], %1, 2 >> +extractps [%3 + 3*%4], %1, 3 > > extractps with imm=0 shou

Re: [FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > On Thu, Jan 28, 2016 at 11:35:41AM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch tries to address ticket #4947. > > (only tested with libx264) > > > > Please review, Carl Eugen > > why should the errors be conditional? Because beha

Re: [FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Derek Buitenhuis
On 1/28/2016 10:35 AM, Carl Eugen Hoyos wrote: > Attached patch tries to address ticket #4947. > (only tested with libx264) I don't think STRICT_STD_COMPLIANCE is the correct thing to use here. It looks like explode mode is better suited. - Derek ___ f

Re: [FFmpeg-devel] [PATCH]lavc/libx26x: Allow to fail if wrong options were passed

2016-01-28 Thread Hendrik Leppkes
Am 28.01.2016 18:00 schrieb "Derek Buitenhuis" : > > On 1/28/2016 10:35 AM, Carl Eugen Hoyos wrote: > > Attached patch tries to address ticket #4947. > > (only tested with libx264) > > I don't think STRICT_STD_COMPLIANCE is the correct thing to use here. > > It looks like explode mode is better sui

Re: [FFmpeg-devel] [libav-devel] [PATCHv3] avcodec: Cineform HD Decoder

2016-01-28 Thread Kieran Kunhya
On Thu, 28 Jan 2016 at 16:26 Vittorio Giovara wrote: > On Mon, Jan 25, 2016 at 6:15 PM, Kieran Kunhya > wrote: > >>> +{ > >>> +CFHDContext *s = avctx->priv_data; > >>> + > >>> +avctx->pix_fmt = AV_PIX_FMT_YUV422P10; > >> > >> if the decoder supports multiple pixel formats it'

Re: [FFmpeg-devel] [PATCH] Add TargetTypeValue in Matroska tag prefix

2016-01-28 Thread Nicolas George
L'octidi 8 pluviôse, an CCXXIV, Pierre Choffet a écrit : > Previously, the Matroska tag names were structured like this: > [/] > This lead to an issue when is not available: the different > levels tags overwrite each > other when they have the same name. > > This patch transforms the name pref

[FFmpeg-devel] [PATCH] dashenc support for 'live' webm

2016-01-28 Thread Ram Natarajan
Folks, This patch makes dashenc.c handle vp8/vp9 video codecs and vorbis codec for audio. The current webm_chunk and webm_dash_manifest is a two stage process and not really suitable for realti me "live" dash mpd creation. I have tested this for both webm and mp4 chunk creation - both for live and

Re: [FFmpeg-devel] [libav-devel] [PATCHv3] avcodec: Cineform HD Decoder

2016-01-28 Thread Vittorio Giovara
On Thu, Jan 28, 2016 at 1:15 PM, Kieran Kunhya wrote: > On Thu, 28 Jan 2016 at 16:26 Vittorio Giovara > wrote: > >> On Mon, Jan 25, 2016 at 6:15 PM, Kieran Kunhya >> wrote: >> >>> +{ >> >>> +CFHDContext *s = avctx->priv_data; >> >>> + >> >>> +avctx->pix_fmt = AV_PIX_FMT_YUV42

Re: [FFmpeg-devel] [PATCH 2/2] avformat: add protocol_whitelist

2016-01-28 Thread Nicolas George
L'octidi 8 pluviôse, an CCXXIV, Michael Niedermayer a écrit : > a fix, good or not that isnt implemented is useless > > I am not really attracted to the design you suggest, to me its worse > in several ways but above all its alot more work. So I dont volunteer > > to elaborate and avoid misuderst

[FFmpeg-devel] lavfi format negotiation at the "code source" seminar in Paris

2016-01-28 Thread Nicolas George
Hi. I am proud to announce that I have been asked to give a talk at the "code source" seminar at the Jussieu Campus in Paris on February the 17th 2016. This is a computer science / philosophy seminar where people present and explain an interesting program they know well by commenting directly the

Re: [FFmpeg-devel] [PATCHv2 1/2] lavf/segment: add support for specifying clock time offset

2016-01-28 Thread Marton Balint
On Thu, 28 Jan 2016, Stefano Sabatini wrote: On date Thursday 2016-01-28 00:49:27 +0100, Marton Balint encoded: Signed-off-by: Marton Balint --- doc/muxers.texi | 10 ++ libavformat/segment.c | 11 ++- 2 files changed, 20 insertions(+), 1 deletion(-) [...] LGTM, thank

Re: [FFmpeg-devel] [PATCHv2 2/2] lavf/segment: add new option segment_clocktime_wrap_duration

2016-01-28 Thread Marton Balint
On Thu, 28 Jan 2016, Stefano Sabatini wrote: On date Thursday 2016-01-28 00:49:28 +0100, Marton Balint encoded: This option can force the segmenter to only start a new segment if a packet reaches the muxer whithin the specified duration after the segmenting clock nit: whithin => within Fixe

Re: [FFmpeg-devel] [PATCH 2/3] lavd/decklink_dec: add support for teletext

2016-01-28 Thread Marton Balint
On Thu, 21 Jan 2016, Marton Balint wrote: On Thu, 14 Jan 2016, Marton Balint wrote: It uses the libzvbi slicer, therefore teletext capture requires libzvbi. Signed-off-by: Marton Balint --- doc/indevs.texi | 10 + libavdevice/decklink_common.h | 2 + libavdevice/decklin

[FFmpeg-devel] [PATCH v4] avcodec: Cineform HD Decoder

2016-01-28 Thread Kieran Kunhya
Decodes YUV 4:2:2 10-bit and RGB 12-bit files. Older files with more subbands, skips, Bayer, alpha not supported. Alpha requires addition of GBRAP12 pixel format. --- libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/cfhd.c | 746 ++

Re: [FFmpeg-devel] [PATCH 2/3] lavd/decklink_dec: add support for teletext

2016-01-28 Thread Deti Fliegl
Hi, I am sorry for the delay. Your patch seems ok as far as I could test. Deti On 28.01.16 21:43, Marton Balint wrote: On Thu, 21 Jan 2016, Marton Balint wrote: On Thu, 14 Jan 2016, Marton Balint wrote: It uses the libzvbi slicer, therefore teletext capture requires libzvbi. Signed-off-b

[FFmpeg-devel] [PATCH] libkvazaar: add support for interlaced coding

2016-01-28 Thread Ari Koivula
Tell libkvazaar to use field coding when -flags +ildct is detected, resulting in two pictures being output for every frame instead of one. Signed-off-by: Ari Koivula --- configure | 2 +- libavcodec/libkvazaar.c | 12 2 files changed, 13 insertions(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH] libkvazaar: add support for interlaced coding

2016-01-28 Thread Ari Koivula
I'm not quite sure if I'm going about this interlacing thing correctly, so I figured I should send in a patch and ask. Is AV_CODEC_FLAG_INTERLACED_DCT the correct flag to look at? There is also AV_CODEC_FLAG_INTERLACED_ME, but libx264 uses only DCT. Is outputting more than one picture per frame o

Re: [FFmpeg-devel] [PATCH] libkvazaar: add support for interlaced coding

2016-01-28 Thread Carl Eugen Hoyos
Ari Koivula koivu.la> writes: > Tell libkvazaar to use field coding when -flags +ildct > is detected, resulting in two pictures being output for > every frame instead of one. Is this really how it should be done? And is this useful? Imo, decoding should be fixed first, but that absolutely ma

Re: [FFmpeg-devel] Workaround for VS 2015 Update 1 bug

2016-01-28 Thread Hendrik Leppkes
Am 28.01.2016 22:14 schrieb "Bruce Dawson" : > > I hit a VS 2015 Update 1 code-gen bug that causes an string array to not be > null-terminated. I've already landed a patch in Chromium's downstream copy, > to unblock the failing test and I wanted to offer up the same patch for the > main ffmpeg repo

Re: [FFmpeg-devel] [PATCH] msvc: Fix libx264 linking

2016-01-28 Thread Moritz Barsnick
On Thu, Jan 28, 2016 at 17:18:47 +0100, Henrik Gramner wrote: > +-lx264) echo libx264.lib ;; Is this related to this issue?: http://ffmpeg.org/pipermail/ffmpeg-user/2016-January/030408.html http://ffmpeg.org/pipermail/ffmpeg-user/2016-January/030422.html Moritz _

Re: [FFmpeg-devel] Workaround for VS 2015 Update 1 bug

2016-01-28 Thread Bruce Dawson
> Can you elaborate which test should be failing? Sure. The test that is failing is not in the ffmpeg repo, but it uses the ffmpeg repo. The Chromium test that fails (debug and release because we always compile ffmpeg optimized) is this one: media_unittests.exe --single-process-tests --gtest_fil

Re: [FFmpeg-devel] [PATCH v4] avcodec: Cineform HD Decoder

2016-01-28 Thread Moritz Barsnick
Cosmetic nerd nit: > +av_log(avctx, AV_LOG_DEBUG, "large len %x \n", ((tagu & 0xff) << > 16) | data); > +} else if (tag == 20) { > +av_log(avctx, AV_LOG_DEBUG, "Width %"PRIu16" \n", data); ^ You're logging

Re: [FFmpeg-devel] [PATCH] msvc: Fix libx264 linking

2016-01-28 Thread Henrik Gramner
On Thu, Jan 28, 2016 at 10:57 PM, Moritz Barsnick wrote: > On Thu, Jan 28, 2016 at 17:18:47 +0100, Henrik Gramner wrote: >> +-lx264) echo libx264.lib ;; > > Is this related to this issue?: > http://ffmpeg.org/pipermail/ffmpeg-user/2016-January/030408.html > http://ffmpeg.

[FFmpeg-devel] [PATCH 3/3] Update demuxers and protocols for protocol whitelist support

2016-01-28 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavdevice/lavfi.c |7 ++- libavformat/async.c |2 +- libavformat/cache.c |3 ++- libavformat/concat.c |5 +++-- libavformat/crypto.c |5

[FFmpeg-devel] [PATCH 2/3] avformat: add protocol_whitelist

2016-01-28 Thread Michael Niedermayer
From: Michael Niedermayer TODO: Docs TODO: version bump Note to maintainers: update tools Note to maintainers: set a default whitelist for your protocol If that makes no sense then consider to set "none" and thus require the user to specify a white-list for sub-protocols to be opened Note, te

[FFmpeg-devel] [PATCH 1/3] avformat/avio: free url/avio options

2016-01-28 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavformat/avio.c|1 + libavformat/aviobuf.c |1 + 2 files changed, 2 insertions(+) diff --git a/libavformat/avio.c b/libavformat/avio.c index 05d0557..96b18fd 100644 --- a/libavformat/avio.c +++ b/libavformat/avio.c

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add protocol_whitelist

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 12:12:01AM +0100, Andreas Cadhalpun wrote: > On 27.01.2016 14:22, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > TODO: Docs > > TODO: version bump > > > > Note to maintainers: update tools > > > > Note to maintainers: set a default whitelist for your pr

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

2016-01-28 Thread Andy Furniss
Paul B Mahol wrote: Hi, 2nd version attached. Nice - fields support :-) I think bf and tf are the wrong way round. ie. ffmpeg -i something-tff -vf nnedi :field=tf gives wrong output on same file field=bf looks correct. idet confirms input is tff (which I knew anyway) ___

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add protocol_whitelist

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 11:58:37PM +0100, Michael Niedermayer wrote: > On Thu, Jan 28, 2016 at 12:12:01AM +0100, Andreas Cadhalpun wrote: > > On 27.01.2016 14:22, Michael Niedermayer wrote: > > > From: Michael Niedermayer > > > > > > TODO: Docs > > > TODO: version bump > > > > > > Note to mainta

Re: [FFmpeg-devel] [PATCH 2/3] lavd/decklink_dec: add support for teletext

2016-01-28 Thread Marton Balint
On Thu, 28 Jan 2016, Deti Fliegl wrote: Hi, I am sorry for the delay. Your patch seems ok as far as I could test. Great, thanks. Pushed with libavdevice version micro bump. Regards, Marton ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org ht

Re: [FFmpeg-devel] [PATCH v2 3/3] configure: remove libzvbi GPL dependency

2016-01-28 Thread Marton Balint
On Fri, 15 Jan 2016, Marton Balint wrote: The COPYING.LIB file in the zvbi source tree as well as libzvbi.h references the GNU Library General Public License version 2 since version 0.2.28. Signed-off-by: Marton Balint Pushed. Regards, Marton ___

Re: [FFmpeg-devel] [PATCH 1/3] configure: use -ldl for decklink

2016-01-28 Thread Marton Balint
On Thu, 14 Jan 2016, Marton Balint wrote: Signed-off-by: Marton Balint --- configure | 2 ++ 1 file changed, 2 insertions(+) Pushed this as well. Regards, Marton ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listi

[FFmpeg-devel] [PATCH 4/4] tests: Add test for proper header guard

2016-01-28 Thread Timothy Gu
--- tests/fate/source-check.sh | 11 +++ tests/ref/fate/source | 12 2 files changed, 23 insertions(+) diff --git a/tests/fate/source-check.sh b/tests/fate/source-check.sh index fb7af98..5524236 100755 --- a/tests/fate/source-check.sh +++ b/tests/fate/source-check.sh @@

[FFmpeg-devel] [PATCH 1/4] all: Add missing header guards

2016-01-28 Thread Timothy Gu
--- compat/va_copy.h | 5 + libavcodec/dcamath.h | 5 + libavcodec/dvaudio.h | 5 + libavcodec/frame_thread_encoder.h | 4 libavcodec/xface.h| 5 + libavdevice/decklink_common.h | 4 libavdevice/decklink_commo

[FFmpeg-devel] [PATCH 2/4] all: Make header guard names consistent

2016-01-28 Thread Timothy Gu
--- compat/aix/math.h | 6 +++--- compat/msvcrt/snprintf.h| 6 +++--- compat/os2threads.h | 6 +++--- compat/tms470/math.h| 6 +++--- compat/w32pthreads.h| 6 +++--- libavcodec/aacps.h | 6 +++--- libavc

[FFmpeg-devel] [PATCH 3/4] amrwbdec_mips: Add missing ff_ prefix

2016-01-28 Thread Timothy Gu
--- libavcodec/mips/amrwbdec_mips.c | 2 +- libavcodec/mips/amrwbdec_mips.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/mips/amrwbdec_mips.c b/libavcodec/mips/amrwbdec_mips.c index 1d6ed2d..e933b8a 100644 --- a/libavcodec/mips/amrwbdec_mips.c +++ b/libavcode

Re: [FFmpeg-devel] [PATCH v4] avcodec: Cineform HD Decoder

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 08:53:00PM +, Kieran Kunhya wrote: > Decodes YUV 4:2:2 10-bit and RGB 12-bit files. > Older files with more subbands, skips, Bayer, alpha not supported. > Alpha requires addition of GBRAP12 pixel format. [...] > +static inline int dequant_and_decompand(int level, int qu

Re: [FFmpeg-devel] [PATCH 1/4] all: Add missing header guards

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 06:33:53PM -0800, Timothy Gu wrote: > --- > compat/va_copy.h | 5 + > libavcodec/dcamath.h | 5 + > libavcodec/dvaudio.h | 5 + > libavcodec/frame_thread_encoder.h | 4 > libavcodec/xface.h| 5 +

[FFmpeg-devel] [PATCH] configure: make sure check_func_headers tests work with dead code stripping

2016-01-28 Thread Rodger Combs
OSX's ld's -Wl,-dead_strip is applied before symbols are resolved, so if the check functions aren't called, they're stripped out and the test is useless. This calls the functions, so they won't be stripped. --- configure | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/conf

Re: [FFmpeg-devel] [PATCH 3/4] amrwbdec_mips: Add missing ff_ prefix

2016-01-28 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 06:33:55PM -0800, Timothy Gu wrote: > --- > libavcodec/mips/amrwbdec_mips.c | 2 +- > libavcodec/mips/amrwbdec_mips.h | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB W

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libaacplus

2016-01-28 Thread Timothy Gu
On Mon, Jan 25, 2016 at 01:16:56PM +0100, Michael Niedermayer wrote: On Sun, Jan 24, 2016 at 09:50:01AM -0800, Timothy Gu wrote: > It's inferior to fdk-aac and has an arguably more problematic > license. a reference to the comparision finding its inferriority should be added to the commit mess

Re: [FFmpeg-devel] [PATCH 1/4] all: Add missing header guards

2016-01-28 Thread Timothy Gu
On Fri, Jan 29, 2016 at 04:08:28AM +0100, Michael Niedermayer wrote: > LGTM > > thx Pushed. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libaacplus

2016-01-28 Thread Timothy Gu
On Mon, Jan 25, 2016 at 10:23:57AM +, Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > > > It's inferior to fdk-aac and has an arguably more problematic > > license. > > Needs a Changelog entry and a news entry together with > libvo. Changelog entry added locally. What do you mean

Re: [FFmpeg-devel] [PATCH 3/4] amrwbdec_mips: Add missing ff_ prefix

2016-01-28 Thread Timothy Gu
On Fri, Jan 29, 2016 at 04:43:21AM +0100, Michael Niedermayer wrote: > > LGTM > thx Pushed. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libvo-aacenc support.

2016-01-28 Thread Timothy Gu
On Sun, Jan 24, 2016 at 04:33:31PM +, Kieran Kunhya wrote: > The internal encoder is superior to libvo-aacenc. > --- > configure | 6 -- > doc/encoders.texi | 25 -- > doc/general.texi | 8 -- > libavcodec/Makefile | 1 - > libavcodec/allcodecs

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libvo-aacenc support.

2016-01-28 Thread James Almer
On 1/29/2016 1:12 AM, James Almer wrote: > On 1/29/2016 1:08 AM, Timothy Gu wrote: >> On Sun, Jan 24, 2016 at 04:33:31PM +, Kieran Kunhya wrote: >>> The internal encoder is superior to libvo-aacenc. >>> --- >>> configure | 6 -- >>> doc/encoders.texi | 25 -- >>>

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libvo-aacenc support.

2016-01-28 Thread James Almer
On 1/29/2016 1:08 AM, Timothy Gu wrote: > On Sun, Jan 24, 2016 at 04:33:31PM +, Kieran Kunhya wrote: >> The internal encoder is superior to libvo-aacenc. >> --- >> configure | 6 -- >> doc/encoders.texi | 25 -- >> doc/general.texi | 8 -- >> libavcode