Re: [FFmpeg-devel] [PATCH 1/4] blowfish: add av_blowfish_alloc()

2015-08-28 Thread James Almer
On 8/1/2015 1:31 PM, James Almer wrote: > On 01/08/15 7:15 AM, Nicolas George wrote: >> Le quartidi 14 thermidor, an CCXXIII, Reimar Döffinger a écrit : >>> If the goal is consistency, wouldn't an API like av_hash be the better >>> solution? >> >> Hear, hear! >> >> (Actually, when reading that disc

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-28 Thread Yayoi Ukai
>> >> rstrip_spaces_buf(dst); >> av_bprintf(dst, "\\N"); >> line_start = 1; >> @@ -90,6 +105,15 @@ void ff_htmlmarkup_to_ass(AVCodecContext *avctx, >> AVBPrint *dst, const char *in) >> av_bprint_chars(dst, *in, 1); >> break; >>

[FFmpeg-devel] [PATCH 3/4] avformat/samidec:do not include trailing sami

2015-08-28 Thread Yayoi
footer in the last packet --- libavformat/samidec.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavformat/samidec.c b/libavformat/samidec.c index 948e1ed..bc9b745 100644 --- a/libavformat/samidec.c +++ b/libavformat/samidec.c @@ -68,11 +68,17 @@ static int sami_rea

[FFmpeg-devel] [PATCH 4/4] fate/subtitles: add a new test for SAMI demuxer

2015-08-28 Thread Yayoi
and decoder --- tests/fate/subtitles.mak | 3 ++ tests/ref/fate/sub-sami2 | 91 2 files changed, 94 insertions(+) create mode 100644 tests/ref/fate/sub-sami2 diff --git a/tests/fate/subtitles.mak b/tests/fate/subtitles.mak index d8b2034..f5270ea

[FFmpeg-devel] [PATCH 2/4] avcodec/samidec: use ff_htmlmarkup_to_ass()

2015-08-28 Thread Yayoi
--- libavcodec/Makefile | 2 +- libavcodec/samidec.c| 38 ++ tests/ref/fate/sub-sami | 18 +- 3 files changed, 36 insertions(+), 22 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index d1ffb12..1045d7a 100644 --- a/

[FFmpeg-devel] [PATCH 1/4] avcodec/srtdec: factor out HTML parsing code

2015-08-28 Thread Yayoi
This code will be used in SAMI decoder in a later commit. --- libavcodec/Makefile| 2 +- libavcodec/htmlsubtitles.c | 177 + libavcodec/htmlsubtitles.h | 29 libavcodec/srtdec.c| 149 +- 4 f

Re: [FFmpeg-devel] FFmpeg 2.8

2015-08-28 Thread Ganesh Ajjanagadde
On Fri, Aug 28, 2015 at 10:28 PM, Michael Niedermayer wrote: > Hi all > > Its about 2 and a half month since 2.7 > if there are no objections then ill branch of release/2.8 from > some revission prior to the next API bump and will then make a 2.8 > release from that, after testing > if you want so

[FFmpeg-devel] FFmpeg 2.8

2015-08-28 Thread Michael Niedermayer
Hi all Its about 2 and a half month since 2.7 if there are no objections then ill branch of release/2.8 from some revission prior to the next API bump and will then make a 2.8 release from that, after testing if you want something in the release, push it prior to the bump OR cherry pick it in rele

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-28 Thread Philip Langdale
On Fri, 28 Aug 2015 20:44:24 +0200 Michael Niedermayer wrote: > On Fri, Aug 28, 2015 at 08:17:41AM -0700, Philip Langdale wrote: > > At least for vdpau, the hwaccel init code tries to check the video > > profile and ensure that there is a matching vdpau profile available. > > > > If it can't fin

Re: [FFmpeg-devel] [PATCH 3/6] lavc: deprecate -ab as part of FF_API_OLD_AVOPTIONS.

2015-08-28 Thread Ronald S. Bultje
On Fri, Aug 28, 2015 at 6:03 PM, Michael Niedermayer wrote: > On Fri, Aug 28, 2015 at 05:23:44PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, Aug 28, 2015 at 5:15 PM, Michael Niedermayer > > > wrote: > > > > > On Fri, Aug 28, 2015 at 03:02:47PM -0400, Ronald S. Bultje wrote: > > > > Hi

Re: [FFmpeg-devel] [PATCH 3/6] lavc: deprecate -ab as part of FF_API_OLD_AVOPTIONS.

2015-08-28 Thread Michael Niedermayer
On Fri, Aug 28, 2015 at 05:23:44PM -0400, Ronald S. Bultje wrote: > Hi, > > On Fri, Aug 28, 2015 at 5:15 PM, Michael Niedermayer > wrote: > > > On Fri, Aug 28, 2015 at 03:02:47PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Mon, Aug 17, 2015 at 9:56 PM, Michael Niedermayer > > > > >

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: introduce compatibility -ab option.

2015-08-28 Thread Ronald S. Bultje
Hi, On Fri, Aug 28, 2015 at 4:42 PM, Michael Niedermayer wrote: > On Mon, Aug 17, 2015 at 06:24:57PM -0400, Ronald S. Bultje wrote: > > --- > > ffmpeg_opt.c | 4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > > index e923a19..effeaf1 100644 > > --- a

Re: [FFmpeg-devel] [PATCH 3/6] lavc: deprecate -ab as part of FF_API_OLD_AVOPTIONS.

2015-08-28 Thread Ronald S. Bultje
Hi, On Fri, Aug 28, 2015 at 5:15 PM, Michael Niedermayer wrote: > On Fri, Aug 28, 2015 at 03:02:47PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Aug 17, 2015 at 9:56 PM, Michael Niedermayer > > > wrote: > > > > > On Mon, Aug 17, 2015 at 01:38:19PM -0400, Ronald S. Bultje wrote: > > >

Re: [FFmpeg-devel] [PATCH 3/6] lavc: deprecate -ab as part of FF_API_OLD_AVOPTIONS.

2015-08-28 Thread Michael Niedermayer
On Fri, Aug 28, 2015 at 03:02:47PM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, Aug 17, 2015 at 9:56 PM, Michael Niedermayer > wrote: > > > On Mon, Aug 17, 2015 at 01:38:19PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Mon, Aug 17, 2015 at 1:12 PM, Michael Niedermayer > > > > >

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: introduce compatibility -ab option.

2015-08-28 Thread Michael Niedermayer
On Mon, Aug 17, 2015 at 06:24:57PM -0400, Ronald S. Bultje wrote: > --- > ffmpeg_opt.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > index e923a19..effeaf1 100644 > --- a/ffmpeg_opt.c > +++ b/ffmpeg_opt.c > @@ -3328,6 +3328,10 @@ const OptionDef option

Re: [FFmpeg-devel] ffplay buffer

2015-08-28 Thread Dan Gordon
I’m working on a frame accurate stream switch. My current iteration is using vapoursynth to serve frames to ffplay. I have the vapoursynth script set up to take a keyboard input and then change to stream without skipping forward to the next iframe. There is a 500 ms delay between the keyboard ac

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: introduce compatibility -ab option.

2015-08-28 Thread Ronald S. Bultje
Hi, On Mon, Aug 17, 2015 at 6:24 PM, Ronald S. Bultje wrote: > --- > ffmpeg_opt.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > index e923a19..effeaf1 100644 > --- a/ffmpeg_opt.c > +++ b/ffmpeg_opt.c > @@ -3328,6 +3328,10 @@ const OptionDef options[]

Re: [FFmpeg-devel] [PATCH 3/6] lavc: deprecate -ab as part of FF_API_OLD_AVOPTIONS.

2015-08-28 Thread Ronald S. Bultje
Hi, On Mon, Aug 17, 2015 at 9:56 PM, Michael Niedermayer wrote: > On Mon, Aug 17, 2015 at 01:38:19PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Aug 17, 2015 at 1:12 PM, Michael Niedermayer > > > wrote: > > > > > On Mon, Aug 17, 2015 at 11:52:16AM -0400, Ronald S. Bultje wrote: > > >

Re: [FFmpeg-devel] [PATCH] ffserver: use -b instead of -ab for setting audio bitrate.

2015-08-28 Thread Ronald S. Bultje
Hi, On Thu, Aug 20, 2015 at 3:07 PM, Paul B Mahol wrote: > On 8/18/15, Ronald S. Bultje wrote: > > --- > > ffserver_config.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/ffserver_config.c b/ffserver_config.c > > index 06bd8ac..de8a454 100644 > > --- a/ff

Re: [FFmpeg-devel] [PATCH] Put remaining pieces of CODEC_FLAG_EMU_EDGE under FF_API_EMU_EDGE.

2015-08-28 Thread Ronald S. Bultje
Hi, On Fri, Aug 28, 2015 at 2:33 PM, Michael Niedermayer wrote: > On Mon, Aug 17, 2015 at 12:25:47PM -0400, Ronald S. Bultje wrote: > > The amv one probably looks suspicious, but since it's an intra-only > > codec, I couldn't possibly imagine what it would use the edge for, > > and the vsyncht f

[FFmpeg-devel] [PATCH] avfilter/vf_waveform: implement various filters

2015-08-28 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 18 ++ libavfilter/vf_waveform.c | 787 +- 2 files changed, 663 insertions(+), 142 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 5bef4b1..983ae74 100644 --- a/doc/filter

Re: [FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-28 Thread Michael Niedermayer
On Fri, Aug 28, 2015 at 08:17:41AM -0700, Philip Langdale wrote: > At least for vdpau, the hwaccel init code tries to check the video > profile and ensure that there is a matching vdpau profile available. > > If it can't find a match, it will fail to initialise. > > In the case of wmv3/vc1, I obs

Re: [FFmpeg-devel] [PATCH] Put remaining pieces of CODEC_FLAG_EMU_EDGE under FF_API_EMU_EDGE.

2015-08-28 Thread Michael Niedermayer
On Mon, Aug 17, 2015 at 12:25:47PM -0400, Ronald S. Bultje wrote: > The amv one probably looks suspicious, but since it's an intra-only > codec, I couldn't possibly imagine what it would use the edge for, > and the vsyncht fate result doesn't change, so it's probably OK. > --- > ffmpeg_opt.c

Re: [FFmpeg-devel] ffplay buffer

2015-08-28 Thread Marton Balint
On Thu, 27 Aug 2015, Dan Gordon wrote: I am trying to reduce the size of the buffer for ffplay, it looks like the default is around 500 ms. I have been looking through the code but unfortunately, I’m struggling with understanding it as I’m not too familiar with c yet. Can somebody point me in

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-28 Thread Robert Krüger
On Fri, Aug 28, 2015 at 5:50 PM, Carl Eugen Hoyos wrote: > Robert Krüger lesspain.de> writes: > > > I still think the text is fine as it will > > hopefully lead to people adding something like > > yadif before the filter > > Which - if I am correct - is plain wrong (or at > least only the second

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-28 Thread Carl Eugen Hoyos
Robert Krüger lesspain.de> writes: > I still think the text is fine as it will > hopefully lead to people adding something like > yadif before the filter Which - if I am correct - is plain wrong (or at least only the second-best option). Of course I may be wrong (I cannot test atm) but if d

[FFmpeg-devel] [PATCH] avcodec/vc1dec: Re-order init to avoid initting hwaccel too early

2015-08-28 Thread Philip Langdale
At least for vdpau, the hwaccel init code tries to check the video profile and ensure that there is a matching vdpau profile available. If it can't find a match, it will fail to initialise. In the case of wmv3/vc1, I observed initialisation to fail all the time. It turns out that this is due to t

Re: [FFmpeg-devel] Enhancing/extending OpenEXR 'gamma' feature

2015-08-28 Thread Kevin Wheatley
On Fri, Aug 28, 2015 at 3:54 PM, Paul B Mahol wrote: > On 8/28/15, Kevin Wheatley wrote: > Feel free to do it. > > Note however that those various curves really belong to avfilter once > swscale supports float pixel format and exr decoder make use of it. Absolutely, most of the tools I normally

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-28 Thread Robert Krüger
On Fri, Aug 28, 2015 at 10:55 AM, Carl Eugen Hoyos wrote: > Robert Krüger lesspain.de> writes: > > > > > > Shouldn't this be: > > > > > ... required to deinterleave before and > > > > > interleave after this filter. > > > > > ? > > > From my understanding what the filter does and what > > the vi

Re: [FFmpeg-devel] Enhancing/extending OpenEXR 'gamma' feature

2015-08-28 Thread Paul B Mahol
On 8/28/15, Kevin Wheatley wrote: > HI, > > I was wondering what others think about the option of extending the > OpenEXR reader's gamma option to also include various transfer > characteristic curves, potentially corresponding to the various > AVCOL_TRC_* options, though I don't immediately have

Re: [FFmpeg-devel] [PATCH 4/5] lavfi: drop the requirement that request_frame returns a frame.

2015-08-28 Thread Paul B Mahol
On 8/26/15, Paul B Mahol wrote: > On 8/26/15, Nicolas George wrote: >> Le nonidi 9 fructidor, an CCXXIII, Paul B Mahol a ecrit : >>> I'm interested. >> >> Glad to read it. So here it is: >> >> Foremost, I want to remove the recursiveness. Currently, when a frame is >> requested on buffersink, it

Re: [FFmpeg-devel] [PATCHv2] configure: colorize warning messages

2015-08-28 Thread Ganesh Ajjanagadde
On Fri, Aug 28, 2015 at 8:36 AM, Clément Bœsch wrote: > On Thu, Aug 27, 2015 at 11:33:52PM -0400, Ganesh Ajjanagadde wrote: >> This patch introduces color warning messages (yellow). >> They are enabled whenever available. >> Tested with zsh, bash, and sh (bash under --posix). >> >> Signed-off-by:

[FFmpeg-devel] [PATCHv3] configure: colorize warning messages

2015-08-28 Thread Ganesh Ajjanagadde
This patch introduces color warning messages (yellow). They are enabled whenever available. Tested with zsh, bash, and sh (bash under --posix). Signed-off-by: Ganesh Ajjanagadde --- configure | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/configure b/configure inde

[FFmpeg-devel] [PATCH] png: Fully support the tRNS chunk

2015-08-28 Thread Donny Yang
Signed-off-by: Donny Yang --- libavcodec/pngdec.c | 98 - 1 file changed, 89 insertions(+), 9 deletions(-) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 6e7eae0..1153d65 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c

[FFmpeg-devel] Enhancing/extending OpenEXR 'gamma' feature

2015-08-28 Thread Kevin Wheatley
HI, I was wondering what others think about the option of extending the OpenEXR reader's gamma option to also include various transfer characteristic curves, potentially corresponding to the various AVCOL_TRC_* options, though I don't immediately have a need for all of them. In addition I could s

Re: [FFmpeg-devel] [PATCHv2] configure: colorize warning messages

2015-08-28 Thread Clément Bœsch
On Thu, Aug 27, 2015 at 11:33:52PM -0400, Ganesh Ajjanagadde wrote: > This patch introduces color warning messages (yellow). > They are enabled whenever available. > Tested with zsh, bash, and sh (bash under --posix). > > Signed-off-by: Ganesh Ajjanagadde > --- > configure | 9 - > 1 fil

Re: [FFmpeg-devel] [PATCH 1/2] better demo ffmpeg filters; demos chaining the filters

2015-08-28 Thread Michael Niedermayer
On Thu, Aug 06, 2015 at 07:53:20PM +, harshit mittal wrote: > Hi ffmpeg developers, > I want to submit the following patch. It's a minor thing that improves the > filtering_video.c to demo chaining of vfilters. applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133

Re: [FFmpeg-devel] [PATCH 4/5] lavfi: drop the requirement that request_frame returns a frame.

2015-08-28 Thread Paul B Mahol
On 8/27/15, Michael Niedermayer wrote: > On Wed, Aug 26, 2015 at 04:46:21PM +0200, Nicolas George wrote: >> Le nonidi 9 fructidor, an CCXXIII, Paul B Mahol a ecrit : >> > I'm interested. >> >> Glad to read it. So here it is: >> >> Foremost, I want to remove the recursiveness. Currently, when a fra

Re: [FFmpeg-devel] [PATCH]Remove framerates from dnxhd encoder help

2015-08-28 Thread Kevin Wheatley
Though not definitive in terms of a specification: http://www.avid.com/static/resources/us/documents/dnxhd.pdf pages 9/10 has the table of options. Kevin ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-de

Re: [FFmpeg-devel] [PATCH]Remove framerates from dnxhd encoder help

2015-08-28 Thread Carl Eugen Hoyos
Hendrik Leppkes gmail.com> writes: > As i understand it, the frame rate is an integral part > of the DNxHD profile, and even though its not validated > and only shown in the output, I think this information > should be kept. I don't know if that's true or not (it absolutely may be) but if yo

Re: [FFmpeg-devel] [PATCH]Remove framerates from dnxhd encoder help

2015-08-28 Thread Hendrik Leppkes
On Fri, Aug 28, 2015 at 11:05 AM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch removes the framerate from the output shown > by the dnxhd encoder if bad properties were chosen: > The framerate is ignored and the table seems very difficult > to maintain and is therefore incomplete, see ticket #

Re: [FFmpeg-devel] [PATCH 1/5] lavfi/buffersink: loop over ff_request_frame().

2015-08-28 Thread Paul B Mahol
Dana 26. 8. 2015. 13:19 osoba "Nicolas George" napisala je: > > Do not assume that ff_request_frame() returning success > implies a frame has arrived in the FIFO. > Instead, just loop until a frame is in the FIFO. > It does not change anything since the same loop is present > in ff_request_frame()

[FFmpeg-devel] [PATCH]Remove framerates from dnxhd encoder help

2015-08-28 Thread Carl Eugen Hoyos
Hi! Attached patch removes the framerate from the output shown by the dnxhd encoder if bad properties were chosen: The framerate is ignored and the table seems very difficult to maintain and is therefore incomplete, see ticket #3342. Please comment, Carl Eugen diff --git a/libavcodec/dnxhddata.

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-28 Thread Carl Eugen Hoyos
Robert Krüger lesspain.de> writes: > > > > Shouldn't this be: > > > > ... required to deinterleave before and > > > > interleave after this filter. > > > > ? > From my understanding what the filter does and what > the visual results of each option would be You mean you tested deinterleaving a