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

Paul B Mahol gmail.com> writes: > +Scale (resize) the input video, using the z.img library. Please add a link here to the git repository of z.img Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

At present it only converts global metadata as that is what I wanted to do. It should be possible to extend it so that the conversion can be different for different files or streams. --- doc/ffmpeg.texi | 6 +++ ffmpeg.c| 15 ++ ffmpeg.h| 1 + ffmpeg_opt.c| 149

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

On Tue, 22 Sep 2015 17:20:13 + Lucas Andrade wrote: > Sorry about top posting.. That's the gmail default. Anyway, I've double > checked and think you're right. I'll add that to open_url, but I need to > set the right HLSContext cookies. Right now it isn't passed as

Re: [FFmpeg-devel] [DECISION] Server "move"

On Tue, 22 Sep 2015 14:21:22 +0200 Clément Bœsch wrote: > On Mon, Sep 21, 2015 at 08:27:47PM +0100, Kieran Kunhya wrote: > > > This "decision" thingy, is intended to confirm that > > > * Stuff should be moved to the telepoint server, possibly > > > duplicates should be left at/moved

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

On Tue, 22 Sep 2015 15:35:06 -0400 Calvin Walton wrote: > Please never enable "-Werror" by default in a release version of a > package - where that's defined as whatever version you recommend > arbitrary users pick up. +1, and please not for development versions

Re: [FFmpeg-devel] [PATCH] hls: only seek if there is an offset

On 22.09.2015 18:18, wm4 wrote: > On Tue, 22 Sep 2015 14:04:48 +0200 > Andreas Cadhalpun wrote: > >> If there is no #EXT-X-BYTERANGE specified, there is no need to seek. >> Seeking fails anyway for rtmp, because this protocol does not support >> url_seek. >> >>

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

On Sat, 2015-09-19 at 12:36 -0400, Ganesh Ajjanagadde wrote: > Regarding the large array of platforms, as can be seen from the > patches I have submitted over the past few weeks, one long term goal > I > have is getting to a "nearly -Werror" state at least on the most > recent clang/gcc. Already

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

"I'm only here for the nitpicking." I noticed quite a lot of errors in the various documentation recently, and joined the list to submit patches. Let me start though by noting my observations here: > + --enable-libzimg enable zimg, needed for zscale filter [no] So, is it "z.img" or

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

Hi Calvin, On Tue, Sep 22, 2015 at 9:35 PM, Calvin Walton wrote: > On Sat, 2015-09-19 at 12:36 -0400, Ganesh Ajjanagadde wrote: > > Regarding the large array of platforms, as can be seen from the > > patches I have submitted over the past few weeks, one long term goal

Re: [FFmpeg-devel] [PATCH] avfilter/af_tremolo: fix envelope phase

On 9/22/15, Kyle Swanson wrote: > Signed-off-by: Kyle Swanson > --- > libavfilter/af_tremolo.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tremolo.c > index 6335401..69eaa34 100644 > ---

Re: [FFmpeg-devel] [PATCH 1/4] lavc/aacdec: handle case where extradata is set between init and first packet

On Tue, Sep 22, 2015 at 4:50 AM, Rodger Combs wrote: > --- > libavcodec/aacdec_template.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/libavcodec/aacdec_template.c b/libavcodec/aacdec_template.c > index c2d7d05..48717c7 100644 > ---

Re: [FFmpeg-devel] [WIP][RFC][PATCH] avfilter: add zscale filter

On Mon, Sep 21, 2015 at 23:53:26 +0200, Paul B Mahol wrote: > On 9/21/15, Carl Eugen Hoyos wrote: > > Paul B Mahol gmail.com> writes: > >> www://github.com/sekrit-twc/zimg > > I fear that a working link is needed in the > > documentation of the filter. > This link works fine.

Re: [FFmpeg-devel] [PATCH 4/4] lavc: deprecate aac_adtstoasc BSF; remove warnings referencing it

On Mon, 21 Sep 2015 21:50:34 -0500 Rodger Combs wrote: > --- > Changelog | 1 + > doc/bitstream_filters.texi | 11 --- > libavcodec/aac_adtstoasc_bsf.c | 73 > ++ > libavformat/flvenc.c | 6

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

Signed-off-by: Paul B Mahol --- Now with X -> gbrp path fixed. --- configure| 4 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_zscale.c | 581 +++ 4 files changed, 587

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/dvdsub: fix partial packet assembly

On Mon, 21 Sep 2015 18:25:30 +0200 wm4 wrote: > Assuming the first and second packets are partial, this would append the > reassembly buffer (ctx->buf) to itself with the second > append_to_cached_buf() call, because buf is set to ctx->buf. > > I do not know a valid

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

Le primidi 1er vendémiaire, an CCXXIV, James Darnley a écrit : > At present it only converts global metadata as that is what I wanted to do. > It > should be possible to extend it so that the conversion can be different for > different files or streams. > --- > doc/ffmpeg.texi | 6 +++ >

Re: [FFmpeg-devel] [DECISION] Server "move"

Hi, On Tue, Sep 22, 2015 at 7:55 PM, compn wrote: > On Tue, 22 Sep 2015 14:21:22 +0200 > Clément Bœsch wrote: > > > On Mon, Sep 21, 2015 at 08:27:47PM +0100, Kieran Kunhya wrote: > > > > This "decision" thingy, is intended to confirm that > > > > * Stuff should be

[FFmpeg-devel] [PATCH] checkasm: add vp9dsp.itxfm_add tests.

--- tests/checkasm/vp9dsp.c | 273 1 file changed, 273 insertions(+) diff --git a/tests/checkasm/vp9dsp.c b/tests/checkasm/vp9dsp.c index e94daf3..4a7bc96 100644 --- a/tests/checkasm/vp9dsp.c +++ b/tests/checkasm/vp9dsp.c @@ -18,12 +18,15 @@ *

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

Sorry about top posting.. That's the gmail default. Anyway, I've double checked and think you're right. I'll add that to open_url, but I need to set the right HLSContext cookies. Right now it isn't passed as parameter to open_url, that way, I'll have to pass it ahead. About the opts, it isn't

[FFmpeg-devel] [PATCH] avfilter/af_tremolo: fix envelope phase

Signed-off-by: Kyle Swanson --- libavfilter/af_tremolo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tremolo.c index 6335401..69eaa34 100644 --- a/libavfilter/af_tremolo.c +++ b/libavfilter/af_tremolo.c @@ -138,7

Re: [FFmpeg-devel] [PATCH] avformat/hls: fix some cases of HLS streams which require cookies

On Mon, 21 Sep 2015 14:46:14 +0200 wm4 wrote: > Broken by commit ba12ba859aabfa7153ba397d869db13acdaba340. This only > happens with HLS streams which use encryption and require preserving > cookies sent by the server. > > Fixes trac issue #4846. > --- > Really stupid... >

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

I've tested and this works. Cookies is being set correctly. As you can see, I've removed the opts2 as it isn't needed anymore. I've added the cookie update to open_url (setting the HLSContext->cookies and the opts (maybe used on ciphered stream). diff --git a/libavformat/hls.c b/libavformat/hls.c

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

On Tue, Sep 22, 2015 at 4:03 PM, wm4 wrote: > On Tue, 22 Sep 2015 15:35:06 -0400 > Calvin Walton wrote: > >> Please never enable "-Werror" by default in a release version of a >> package - where that's defined as whatever version you recommend >>

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

On Tue, Sep 22, 2015 at 3:56 PM, Ronald S. Bultje wrote: > Hi Calvin, > > On Tue, Sep 22, 2015 at 9:35 PM, Calvin Walton > wrote: > >> On Sat, 2015-09-19 at 12:36 -0400, Ganesh Ajjanagadde wrote: >> > Regarding the large array of platforms, as can be

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

On 9/22/15, Moritz Barsnick wrote: > "I'm only here for the nitpicking." I noticed quite a lot of errors in > the various documentation recently, and joined the list to submit > patches. Let me start though by noting my observations here: > >> + --enable-libzimg enable

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

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

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

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

Re: [FFmpeg-devel] [PATCH 1/3] configure: Force mingw's ld to keep the reloc section

On Tue, Sep 22, 2015 at 7:43 PM, Michael Niedermayer wrote: > > > this is ugly, but ASLR is important thus applied > i did not apply the nxcompat dynamicbase merge as its unrelated > > Thanks > > PS: if this breaks some version of mingw then this should be reverted > > [...] > >

Re: [FFmpeg-devel] [PATCH 1/3] configure: Force mingw's ld to keep the reloc section

On Mon, Sep 21, 2015 at 12:41:36AM -0400, Alex Smith wrote: > From: Alex Smith > > Binutils will always strip the relocation information from executable > files even if it needs it (dynamicbase/ASLR). We can work around this > by using the pic-executable flag combined

Re: [FFmpeg-devel] [PATCH 2/3] configure: Support for HEASLR on mingw targets

On 9/21/2015 1:41 AM, Alex Smith wrote: > From: Alex Smith > > The appropriate flag for HEASLR (--high-entropy-va) is scheduled for > inclusion in the next version of binutils (2.25), doesn't hurt to > include it a little early. 2.25 has been out for a while, so i

Re: [FFmpeg-devel] [PATCH 1/3] configure: Force mingw's ld to keep the reloc section

On Tue, Sep 22, 2015 at 07:59:38PM -0400, Alex Smith wrote: > On Tue, Sep 22, 2015 at 7:43 PM, Michael Niedermayer > wrote: > > > > > > this is ugly, but ASLR is important thus applied > > i did not apply the nxcompat dynamicbase merge as its unrelated > > > > Thanks > > > > PS:

Re: [FFmpeg-devel] [PATCH] hls: only seek if there is an offset

On Tue, 22 Sep 2015 14:04:48 +0200 Andreas Cadhalpun wrote: > If there is no #EXT-X-BYTERANGE specified, there is no need to seek. > Seeking fails anyway for rtmp, because this protocol does not support > url_seek. > > This fixes CNN.m3u from trac ticket 4797

Re: [FFmpeg-devel] Update my email address

On Tue, Sep 22, 2015 at 10:43:27AM +0200, Jean Delvare wrote: > My old address no longer works. > --- > MAINTAINERS |2 +- > libavfilter/vf_delogo.c |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

On Tue, 22 Sep 2015 04:37:48 + Lucas Andrade wrote: > This fixes the https://trac.ffmpeg.org/ticket/4808, as it sets the cookie > on playlist response with setcookie header. > > Update Cookies on Setcookie playlist response > --- > libavformat/hls.c | 2 ++ > 1 file

[FFmpeg-devel] [PATCH] avcodec/dvdsubdec: reject some broken packets

If cmd_pos is broken, this would just keep accumulating packets in the reassembly buffer, until it fails and flushes the buffer on overflow. Since packets are usually rather small, this will take a lot of subtitle packets. The perceived effect is that subtitles are not displayed anymore after the

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/dvdsubdec: reject some broken packets

On Mon, 21 Sep 2015 23:33:16 +0200 Michael Niedermayer wrote: > On Mon, Sep 21, 2015 at 06:25:31PM +0200, wm4 wrote: > > If cmd_pos is broken, this would just keep accumulating packets in the > > reassembly buffer, until it fails and flushes the buffer on overflow. > > Since

[FFmpeg-devel] Update my email address

My old address no longer works. --- MAINTAINERS |2 +- libavfilter/vf_delogo.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- ffmpeg.orig/MAINTAINERS 2015-09-16 08:51:37.861444714 +0200 +++ ffmpeg/MAINTAINERS 2015-09-22 10:38:01.760054763 +0200 @@ -365,7

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

You're right, I should at least check if ret == 0. I'll update the patch and check if it works. Em ter, 22 de set de 2015 às 08:09, Lucas Andrade escreveu: > I think the other code paths already sets the cookie. > > Em Ter, 22 de set de 2015 04:41, wm4

Re: [FFmpeg-devel] [WIP][RFC][PATCH] avfilter: add zscale filter

On 9/21/2015 8:22 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > configure| 4 + > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_zscale.c | 546 > +++ > 4

Re: [FFmpeg-devel] [PATCH 1/3] tests/checkasm: make randomize_buffers a function for easier debugging

Hi, On Mon, Sep 21, 2015 at 4:55 AM, Rodger Combs wrote: > --- > tests/checkasm/vp9dsp.c | 108 > +--- > 1 file changed, 57 insertions(+), 51 deletions(-) LGTM. Sorry for the initial long macro, that was kind of ugly, yes.

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

I think the other code paths already sets the cookie. Em Ter, 22 de set de 2015 04:41, wm4 escreveu: > On Tue, 22 Sep 2015 04:37:48 + > Lucas Andrade wrote: > > > This fixes the https://trac.ffmpeg.org/ticket/4808, as it sets the > cookie > >

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

On 9/21/15, Timo Rothenpieler wrote: >> Benefit would be to save 2 branching and 2 div (the div can already be >> replaced by a shift here though - which would have a benefit since x & y >> are signed) and keep the code generic. > > Changed it to a shift and moved the if()

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

still lgtm If nobody objects, i'll push later today. Timo signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

Signed-off-by: Paul B Mahol --- configure| 4 + doc/filters.texi | 110 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_zscale.c | 614 +++ 5 files changed, 730

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

On Tue, 22 Sep 2015 13:47:37 + Lucas Andrade wrote: > You're right, I should at least check if ret == 0. I'll update the patch > and check if it works. > Well, I meant, maybe this should be in open_url(), instead of adding it on every code path. Right now everything

[FFmpeg-devel] [PATCH] hls: only seek if there is an offset

If there is no #EXT-X-BYTERANGE specified, there is no need to seek. Seeking fails anyway for rtmp, because this protocol does not support url_seek. This fixes CNN.m3u from trac ticket 4797 (i.e. Debian bug #798189). --- libavformat/hls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [FFmpeg-devel] [DECISION] Server "move"

On 21/09/15 20:44, Carl Eugen Hoyos wrote: > Michael Niedermayer gmx.at> writes: > >> This "decision" thingy, is intended to confirm that >> * Stuff should be moved to the telepoint server, possibly duplicates >> should be left at/moved to the hetzner server for redundancy, if >> this is

Re: [FFmpeg-devel] [DECISION] Server "move"

On Mon, Sep 21, 2015 at 08:27:47PM +0100, Kieran Kunhya wrote: > > This "decision" thingy, is intended to confirm that > > * Stuff should be moved to the telepoint server, possibly duplicates > > should be left at/moved to the hetzner server for redundancy, if > > this is technically possible

Re: [FFmpeg-devel] [PATCH] Update Cookies on Setcookie playlist response

On Tue, 22 Sep 2015 04:37:48 + Lucas Andrade wrote: > This fixes the https://trac.ffmpeg.org/ticket/4808, as it sets the cookie > on playlist response with setcookie header. > > Update Cookies on Setcookie playlist response > --- > libavformat/hls.c | 2 ++ > 1 file