Re: [FFmpeg-devel] [PATCH] libavformat/oggdec: Free stream private when header parsing fails.

2016-05-10 Thread Michael Niedermayer
On Mon, May 09, 2016 at 03:27:29PM -0700, Chris Cunningham wrote: > Leaking this private structure opens up the possibility that it may > be re-used when parsing later packets in the stream. This is > problematic if the later packets are not the same codec type (e.g. > private allocated during Vorb

Re: [FFmpeg-devel] [PATCH] avcodec/rscc: check deflated input pixel buffer size

2016-05-10 Thread James Almer
On 5/10/2016 9:30 PM, Michael Niedermayer wrote: > On Tue, May 10, 2016 at 04:30:03PM -0300, James Almer wrote: >> Prevents overreads as reported by valgrind >> >> Signed-off-by: James Almer >> --- >> See >> http://fate.ffmpeg.org/report.cgi?time=20160509191458&slot=x86_64-archlinux-gcc-valgrindu

Re: [FFmpeg-devel] [PATCH] avcodec/rscc: check deflated input pixel buffer size

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 04:30:03PM -0300, James Almer wrote: > Prevents overreads as reported by valgrind > > Signed-off-by: James Almer > --- > See > http://fate.ffmpeg.org/report.cgi?time=20160509191458&slot=x86_64-archlinux-gcc-valgrindundef > > libavcodec/rscc.c | 5 + > 1 file changed

Re: [FFmpeg-devel] [PATCH] lavfi/drawutils: support NV12 and NV21

2016-05-10 Thread Michael Niedermayer
On Mon, May 09, 2016 at 08:52:30PM -0500, Rodger Combs wrote: > --- > libavfilter/drawutils.c | 33 - > tests/ref/fate/filter-pixfmts-pad | 2 ++ > 2 files changed, 18 insertions(+), 17 deletions(-) should be ok thx [...] -- Michael GnuPG fingerpr

[FFmpeg-devel] [PATCH] IFF ANIM decoder

2016-05-10 Thread Paul B Mahol
Hi, patch attached. 0001-lavc-add-IFF-ANIM-decoder.patch.bz2 Description: BZip2 compressed data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 3/3] avcodec/dca: add DTS Express (LBR) decoder

2016-05-10 Thread James Almer
On 5/1/2016 12:43 PM, foo86 wrote: > --- > Changelog|1 + > libavcodec/Makefile |2 +- > libavcodec/dca_lbr.c | 1825 > ++ > libavcodec/dca_lbr.h | 130 > libavcodec/dcadata.c | 466 + > libavcodec/dcadata.h |

Re: [FFmpeg-devel] [PATCH] FFmpeg Decklink input connection options

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 11:44:00PM +0200, Deti Fliegl wrote: > Hi, > > thank you. Patch approved. applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door even though the window is wide open a

[FFmpeg-devel] [PATCH] avformat/tee: Move to new BSF API

2016-05-10 Thread sebechlebskyjan
From: Jan Sebechlebsky Signed-off-by: Jan Sebechlebsky --- libavformat/tee.c | 171 -- 1 file changed, 139 insertions(+), 32 deletions(-) diff --git a/libavformat/tee.c b/libavformat/tee.c index 806beaa..ff0918b 100644 --- a/libavformat/tee.c

[FFmpeg-devel] [PATCH] avcodec/rscc: check deflated input pixel buffer size

2016-05-10 Thread James Almer
Prevents overreads as reported by valgrind Signed-off-by: James Almer --- See http://fate.ffmpeg.org/report.cgi?time=20160509191458&slot=x86_64-archlinux-gcc-valgrindundef libavcodec/rscc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/rscc.c b/libavcodec/rscc.c index a47

Re: [FFmpeg-devel] [PATCH] -- Convert decklink input module to use codecpar

2016-05-10 Thread Felt, Patrick
Bump On 5/7/16, 10:02 PM, "ffmpeg-devel on behalf of Felt, Patrick" wrote: >This patch converts decklink input to use codecpar. There are still a couple >of deprecated calls that I’m not sure what to do with. They are both related >to some logic around AVCodecContext.coded_frame. I cou

Re: [FFmpeg-devel] [PATCH] FFmpeg Decklink input connection options

2016-05-10 Thread Deti Fliegl
Hi, thank you. Patch approved. Deti On 10.05.16 19:34, Felipe Astroza wrote: Hello, I thought you could have interest in this: The patch adds additional input options: video input connection (vinput), audio input connection (ainput). *Usage: ffmpeg -f decklink -i 'Card name'@fmt

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread wm4
On Tue, 10 May 2016 22:21:12 +0200 (CEST) Marton Balint wrote: > On Tue, 10 May 2016, Michael Niedermayer wrote: > > > On Tue, May 10, 2016 at 03:53:31PM +0200, wm4 wrote: > >> On Tue, 10 May 2016 14:35:02 +0200 > >> Michael Niedermayer wrote: > >> > >>> On Tue, May 10, 2016 at 02:02:52PM +

Re: [FFmpeg-devel] Add ARM64 NEON optimization for HEVC decoder about Fate test

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 04:46:03PM +0800, 章军海 wrote: > Hi Michael: > > > Sorry for long time no reply to this email, recently I own time to > solve the error > during fate test for ARM64 NEON optimization for HEVC decoder. > > > I have seen fate test document on https://ffmpe

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Marton Balint
On Tue, 10 May 2016, Michael Niedermayer wrote: On Tue, May 10, 2016 at 03:53:31PM +0200, wm4 wrote: On Tue, 10 May 2016 14:35:02 +0200 Michael Niedermayer wrote: On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer wrote:

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Paul B Mahol
On 5/10/16, Kyle Swanson wrote: > On Tue, May 10, 2016 at 4:33 AM, Paul B Mahol wrote: >> On 5/10/16, Kyle Swanson wrote: >>> Hi, >>> >>> Updated patch attached. Thanks! >>> >> >> >> [...] >> >>> >>> +@section loudnorm >>> + >>> +EBU R128 loudness normalization. Includes both dynamic and linear

Re: [FFmpeg-devel] libavdevice/decklink: advanced options for selecting an input connection

2016-05-10 Thread Matthias Hunstock
Am 10.05.2016 um 07:54 schrieb Felipe Astroza: > This patch add additional input options: video input connection (vinput), > audio input connection (ainput). > > Usage: > ffmpeg -f decklink -i 'Card name'@fmt :vinput:ainput > Where fmt, vinput and ainput are optional > >

Re: [FFmpeg-devel] [PATCH v2] hevc: Fix memory leak related to a53_caption data

2016-05-10 Thread Will Kelleher
On 04/08, Will Kelleher wrote: > > The leak is...continuous. We only noticed it during long running decode > operations because the amount is small, but it definitely adds up. > Hi all, I finally got some time to revisit this. In the simple case where we are not decoding the HEVC bitstream, the

Re: [FFmpeg-devel] CONFIG_W64_DEMUXER and dead-code elimination

2016-05-10 Thread Bruce Dawson
BTW, the reason I suggested the CONFIG_W64_DEMUXER patch that started this email thread is because the pre-release version of VS 2015 Update 3 can't handle that code as-is. Microsoft might change their compiler before the officially release of Update 3, but currently ffmpeg (in Chromium at least) d

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Robert Krüger
On Mon, May 9, 2016 at 9:40 PM, Kyle Swanson wrote: > On Tue, Apr 5, 2016 at 7:01 PM, Kyle Swanson wrote: > > > > Here's another audio filter. I hinted at this a few months ago, but I > found out that > > finishing the last 5% took almost as long as the first 95%. This is an > EBU R128 > > dynam

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread wm4
On Tue, 10 May 2016 14:59:18 +0100 Derek Buitenhuis wrote: > On 5/10/2016 2:53 PM, wm4 wrote: > > Indeed, av_opt calls on structs which don't support it compile without > > warning, but mysteriously crash at runtime. That's a violation of this > > principle. And no, it doesn't mean that every str

[FFmpeg-devel] Add ARM64 NEON optimization for HEVC decoder about Fate test

2016-05-10 Thread ??????
Hi Michael: Sorry for long time no reply to this email, recently I own time to solve the error during fate test for ARM64 NEON optimization for HEVC decoder. I have seen fate test document on https://ffmpeg.org/fate.html, but still don't know how to finish the test myself o

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Kyle Swanson
On Tue, May 10, 2016 at 4:33 AM, Paul B Mahol wrote: > On 5/10/16, Kyle Swanson wrote: >> Hi, >> >> Updated patch attached. Thanks! >> > > > [...] > >> >> +@section loudnorm >> + >> +EBU R128 loudness normalization. Includes both dynamic and linear >> normalization modes. >> +Support for both si

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Check that logo area is inside the picture

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 01:16:59PM +, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > > > lgtm > > Then please commit. applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just b

[FFmpeg-devel] [PATCH 1/3] lavfi/framepool: rename FFVideoFramePool to FFFramePool

2016-05-10 Thread Matthieu Bouron
From: Matthieu Bouron --- libavfilter/avfilter.c | 2 +- libavfilter/avfilter.h | 4 ++-- libavfilter/framepool.c | 24 libavfilter/framepool.h | 32 libavfilter/video.c | 20 ++-- 5 files changed, 41 insertions(+)

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 03:08:25PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes an incorrect error message and an incorrect failure on > xerror as reported by applemax82 on both user mailing lists. > > Please comment, Carl Eugen > rmdec.c |4 +++- > 1 file changed, 3 inser

[FFmpeg-devel] [PATCH 2/3] lavfi/framepool: add audio support

2016-05-10 Thread Matthieu Bouron
From: Matthieu Bouron --- libavfilter/framepool.c | 106 libavfilter/framepool.h | 36 +++- 2 files changed, 141 insertions(+), 1 deletion(-) diff --git a/libavfilter/framepool.c b/libavfilter/framepool.c index 36c6e8f..4a63fe9 10064

[FFmpeg-devel] [PATCH 3/3] lavfi: use an audio frame pool for each link of the filtergraph

2016-05-10 Thread Matthieu Bouron
From: Matthieu Bouron --- libavfilter/audio.c | 51 +-- 1 file changed, 37 insertions(+), 14 deletions(-) diff --git a/libavfilter/audio.c b/libavfilter/audio.c index 51fef03..dbc92d6 100644 --- a/libavfilter/audio.c +++ b/libavfilter/audio.c @@ -

[FFmpeg-devel] (no subject)

2016-05-10 Thread Matthieu Bouron
Hello, The following patchset add an audio frame pool for each link of the filtergraph. It extends the FFVideoFramePool API to support audio (and renames it to FFFramePool). The performance gain on a rpi2 is very little. malloc+free goes from 2.50% to 1.84% cpu time with the following command lin

[FFmpeg-devel] additive frame change detection algorithm

2016-05-10 Thread Dave Hess
I'm working on a post-process that will create thumbnails of a computer screen capture. Is there a frame change detection algorithm similar to "scene" that will detect changes from a previously detected frame or from x frames prior? The current scene algorithm doesn't detect subtle changes in fra

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Kyle Swanson
On Tue, May 10, 2016 at 4:35 AM, Rostislav Pehlivanov wrote: > Then just copy the logic from libebur128 into the filter, the library > itself is smaller than some filters we have and it's under the MIT license. I think this is probably a good way to go. I'd need to rewrite the resampling bit to u

Re: [FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 10:21 PM, Michael Niedermayer wrote: > On Tue, May 03, 2016 at 08:58:07PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Tue, May 3, 2016 at 8:29 PM, Michael Niedermayer > > > wrote: > > > > > On Tue, May 03, 2016 at 03:21:00PM -0400, Ronald S. Bultje wrote: > > >

Re: [FFmpeg-devel] [PATCH] fate: add matroska-remux test.

2016-05-10 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 6:33 PM, Michael Niedermayer wrote: > On Fri, May 06, 2016 at 09:27:20AM -0400, Ronald S. Bultje wrote: > > This tests automatic insertion of the vp9_superframe BSF as well as > > ensuring that the colorspace properties in the video header can be > > modified when remu

Re: [FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2016 at 6:47 AM, Paul B Mahol wrote: > On 5/6/16, Ronald S. Bultje wrote: > > Also add von kries whitepoint adaptation, and add 'identity' to turn > > whitepoint adaptation off. > > --- > > doc/filters.texi| 15 +++ > > libavfilter/vf_colorspace.c |

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 03:39:09PM +0200, Hendrik Leppkes wrote: > On Tue, May 10, 2016 at 2:35 PM, Michael Niedermayer > wrote: > > On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: > >> On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer > >> wrote: > >> > On Tue, May 10, 2016 a

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 03:53:31PM +0200, wm4 wrote: > On Tue, 10 May 2016 14:35:02 +0200 > Michael Niedermayer wrote: > > > On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: > > > On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer > > > wrote: > > > > On Tue, May 10, 2016 at

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 03:56:18PM +0200, Hendrik Leppkes wrote: > On Tue, May 10, 2016 at 2:10 PM, Michael Niedermayer > wrote: > > > > libavformat gathers information about streams in av_find_stream_info() > > and that is all information completely unneeded by decoders > > no decoder needs the w

Re: [FFmpeg-devel] [PATCH 1/2] dshow: Add missing library dependency (dxguid)

2016-05-10 Thread Roger Pack
did you have a failure with this? in what environment? On 5/6/16, Moriyoshi Koizumi wrote: > From: Moriyoshi koizumi > > Signed-off-by: Moriyoshi koizumi > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 1f684d2..fb8d7d3 10

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Derek Buitenhuis
On 5/10/2016 2:53 PM, wm4 wrote: > Indeed, av_opt calls on structs which don't support it compile without > warning, but mysteriously crash at runtime. That's a violation of this > principle. And no, it doesn't mean that every struct should have an > AVClass, because that would be insane. Is there

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Hendrik Leppkes
On Tue, May 10, 2016 at 2:10 PM, Michael Niedermayer wrote: > > libavformat gathers information about streams in av_find_stream_info() > and that is all information completely unneeded by decoders > no decoder needs the width and height from the parser run by > av_find_stream_info() or other field

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread wm4
On Tue, 10 May 2016 14:35:02 +0200 Michael Niedermayer wrote: > On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: > > On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer > > wrote: > > > On Tue, May 10, 2016 at 11:17:12AM +0100, Derek Buitenhuis wrote: > > >> On 5/10/2016 7:2

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > > How is the fact that the device produces damaged files > > (be it because of wireless or other issues) related to > > fixing broken audio for the given samples? > > Either too big predictors should be skipped or clipped. > How decoded output sounds when played

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Paul B Mahol
On 5/10/16, Hendrik Leppkes wrote: > On Tue, May 10, 2016 at 2:35 PM, Michael Niedermayer > wrote: >> On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: >>> On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer >>> wrote: >>> > On Tue, May 10, 2016 at 11:17:12AM +0100, Derek Buitenh

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Hendrik Leppkes
On Tue, May 10, 2016 at 2:35 PM, Michael Niedermayer wrote: > On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: >> On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer >> wrote: >> > On Tue, May 10, 2016 at 11:17:12AM +0100, Derek Buitenhuis wrote: >> >> On 5/10/2016 7:24 AM, Hendr

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Paul B Mahol
On 5/10/16, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> OT but I remembered seeing overread messages from OP >> when using your patch on his files. > > How is the fact that the device produces damaged files > (be it because of wireless or other issues) related to > fixing broke

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > OT but I remembered seeing overread messages from OP > when using your patch on his files. How is the fact that the device produces damaged files (be it because of wireless or other issues) related to fixing broken audio for the given samples? Carl Eugen __

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Paul B Mahol
On 5/10/16, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> Again not patch but useless diff file. > > Again no useful comment;-( > > Did you forget my question about the Babyphon? OT but I remembered seeing overread messages from OP when using your patch on his files. ___

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Check that logo area is inside the picture

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > lgtm Then please commit. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > Again not patch but useless diff file. Again no useful comment;-( Did you forget my question about the Babyphon? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ff

Re: [FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Paul B Mahol
On 5/10/16, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes an incorrect error message and an incorrect failure on > > xerror as reported by applemax82 on both user mailing lists. > > Please comment, Carl Eugen > Again not patch but useless diff file. __

[FFmpeg-devel] [PATCH]lavf/rmdec: Do not return EIO on EOF

2016-05-10 Thread Carl Eugen Hoyos
Hi! Attached patch fixes an incorrect error message and an incorrect failure on xerror as reported by applemax82 on both user mailing lists. Please comment, Carl Eugen diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c index cb0481e..a633582 100644 --- a/libavformat/rmdec.c +++ b/libavformat

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Check that logo area is inside the picture

2016-05-10 Thread Paul B Mahol
On 5/10/16, Jean Delvare wrote: > We can only remove the logo if it is inside the picture. We need at > least one pixel around the logo area for interpolation. > > Fixes ticket #5527 (Delogo crash with x=0 and/or y=0). > > Signed-off-by: Jean Delvare > --- > libavfilter/vf_delogo.c | 15 ++

[FFmpeg-devel] [PATCH] avfilter/delogo: Check that logo area is inside the picture

2016-05-10 Thread Jean Delvare
We can only remove the logo if it is inside the picture. We need at least one pixel around the logo area for interpolation. Fixes ticket #5527 (Delogo crash with x=0 and/or y=0). Signed-off-by: Jean Delvare --- libavfilter/vf_delogo.c | 15 +++ 1 file changed, 15 insertions(+) --

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Clément Bœsch
On Mon, May 09, 2016 at 02:40:09PM -0500, Kyle Swanson wrote: [...] > I thought about extracting the FFmpeg ebur128 functions so they could be > reused elsewhere in FFmpeg, but the R128 logic from the ebur128 filter > is quite tangled, tied to filter functions, internal states are spread > out betw

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 02:02:52PM +0200, Hendrik Leppkes wrote: > On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer > wrote: > > On Tue, May 10, 2016 at 11:17:12AM +0100, Derek Buitenhuis wrote: > >> On 5/10/2016 7:24 AM, Hendrik Leppkes wrote: > >> > I don't like this, the struct is pretty cl

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > libavformat gathers information about streams in av_find_stream_info() > and that is all information completely unneeded by decoders > no decoder needs the width and height from the parser run by > av_find_stream_info() or other fields. Didn't Hendri

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 08:24:35AM +0200, Hendrik Leppkes wrote: > On Tue, May 10, 2016 at 2:07 AM, Michael Niedermayer > wrote: > > Allow enumeration, read and write of fields without requiring #if on > > versions > > All other public structures can be accessed through AVOptions > > > > TODO: ad

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Derek Buitenhuis
On 5/10/2016 12:43 PM, Michael Niedermayer wrote: > This patch adds an AVClass field to AVCodecParameters like we use in > other public structures. > i fail to see how this patch is related to your reply I apologize. I thought it was the start of an attempt to address the "issue" from the other pa

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Hendrik Leppkes
On Tue, May 10, 2016 at 1:43 PM, Michael Niedermayer wrote: > On Tue, May 10, 2016 at 11:17:12AM +0100, Derek Buitenhuis wrote: >> On 5/10/2016 7:24 AM, Hendrik Leppkes wrote: >> > I don't like this, the struct is pretty cleanly defined and unlikely >> > to be extended much over time. >> > Most ot

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Michael Niedermayer
On Tue, May 10, 2016 at 11:17:12AM +0100, Derek Buitenhuis wrote: > On 5/10/2016 7:24 AM, Hendrik Leppkes wrote: > > I don't like this, the struct is pretty cleanly defined and unlikely > > to be extended much over time. > > Most other structs have AVOptions so the CLI can interact with it, but > >

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread wm4
On Tue, 10 May 2016 08:24:35 +0200 Hendrik Leppkes wrote: > On Tue, May 10, 2016 at 2:07 AM, Michael Niedermayer > wrote: > > Allow enumeration, read and write of fields without requiring #if on > > versions > > All other public structures can be accessed through AVOptions > > > > TODO: add all

Re: [FFmpeg-devel] [PATCHv2] add signature filter for MPEG7 video signature

2016-05-10 Thread Gerion Entrup
On Dienstag, 3. Mai 2016 19:01:51 CEST Gerion Entrup wrote: > On Sonntag, 24. April 2016 01:44:51 CEST Michael Niedermayer wrote: > > On Tue, Apr 19, 2016 at 04:37:16PM +0200, Gerion Entrup wrote: > > > On Dienstag, 19. April 2016 13:25:53 CEST Moritz Barsnick wrote: > > > +static int request_frame

Re: [FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-10 Thread Paul B Mahol
On 5/6/16, Ronald S. Bultje wrote: > Also add von kries whitepoint adaptation, and add 'identity' to turn > whitepoint adaptation off. > --- > doc/filters.texi| 15 +++ > libavfilter/vf_colorspace.c | 39 --- > 2 files changed, 47 insert

Re: [FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2016 at 8:36 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, May 6, 2016 at 9:40 AM, Ronald S. Bultje > wrote: > >> Also add von kries whitepoint adaptation, and add 'identity' to turn >> whitepoint adaptation off. >> --- >> doc/filters.texi| 15 +++ >>

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-10 Thread Derek Buitenhuis
On 5/10/2016 7:24 AM, Hendrik Leppkes wrote: > I don't like this, the struct is pretty cleanly defined and unlikely > to be extended much over time. > Most other structs have AVOptions so the CLI can interact with it, but > this struct is not meant to be modified by users, its just a direct > line

Re: [FFmpeg-devel] [PATCH]: add DST decoder

2016-05-10 Thread Paul B Mahol
On 5/9/16, Piotr Bandurski wrote: >> > this eating up memory: >> >> Another one attached. > > similar issue with this file: Fixed. 0001-avcodec-add-Direct-Stream-Transfer-DST-decoder.patch.bz2 Description: BZip2 compressed data ___ ffmpeg-devel mailin

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Rostislav Pehlivanov
On 9 May 2016 at 20:40, Kyle Swanson wrote: > On Tue, Apr 5, 2016 at 7:01 PM, Kyle Swanson wrote: > > > > Here's another audio filter. I hinted at this a few months ago, but I > found out that > > finishing the last 5% took almost as long as the first 95%. This is an > EBU R128 > > dynamic loudn

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Paul B Mahol
On 5/10/16, Kyle Swanson wrote: > Hi, > > Updated patch attached. Thanks! > [...] > > +@section loudnorm > + > +EBU R128 loudness normalization. Includes both dynamic and linear > normalization modes. > +Support for both single pass (livestreams, files) and double pass (files) > modes. > +Thi

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-10 Thread Gaullier Nicolas
>Most broadcasters are now specifying mandatory loudness targets for source >material, which is usually supplied as a target integrated loudness value and >a target maximum true peak. Loudness normalization is easy for files with >sufficient headroom for linear gain adjustments, difficult for fi