Re: [FFmpeg-devel] [PATCH] ffserver: fix memory leaks pointed out by valgrind.

2017-04-15 Thread Zalewa PL
Hello, The plan is that if ffserver is not fixed before the next bump, it will be removed. ffserver has been serving our purpose in a 24/7 fashion for about 7 years now. It will probably be serving us for the foreseeable future, whatever the community decides. I prefer to share any fix I mak

Re: [FFmpeg-devel] [PATCH] web/index: add news entry for release 3.3

2017-04-15 Thread Reto Kromer
James Almer wrote: >IMO, since the "we strongly recommend" line has been a constant >in all releases, removing it now will (for those that notice >it) rise quite a few red flags and make people come to the >conclusion they should probably skip it altogether. +1 And as for bug report/fixing there

Re: [FFmpeg-devel] [PATCH] web/index: add news entry for release 3.3

2017-04-15 Thread Reto Kromer
Michael Niedermayer wrote: >> + April 13th, 2017, FFmpeg 3.3 "Hilbert" >> + >> +FFmpeg 3.3 "Hilbert", >>a new >> +major release, is now available! Some of the highlights: >> + > >> + >> + > >is this valid html ? No! Either a list: . . . or simply the text: . . . is cor

Re: [FFmpeg-devel] [PATCH] web/index: add news entry for release 3.3

2017-04-15 Thread Carl Eugen Hoyos
2017-04-15 1:48 GMT+02:00 James Almer : > And really, the tree wasn't any more "unstable" when we branched it > for 3.3 than how it had been when we branched previous releases. That's not how I remember it. We had a relatively high number of open regressions when 3.2 was released but I don't thi

Re: [FFmpeg-devel] [PATCHv3 2/3] ffplay: convert to new decode API

2017-04-15 Thread Marton Balint
On Fri, 7 Apr 2017, Marton Balint wrote: Signed-off-by: Marton Balint --- ffplay.c | 101 ++- 1 file changed, 54 insertions(+), 47 deletions(-) Pushed the rest of the series. Regards, Marton

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-15 Thread Marton Balint
On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 3:40 PM, Marton Balint wrote: On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 2:12 AM, Hendrik Leppkes wrote: On Thu, Apr 13, 2017 at 10:36 AM, Aaron Levinson wrote: Give it some time for the other changes to be reviewed by

Re: [FFmpeg-devel] [PATCH] Made minor changes to get the decklink avdevice code to build using Visual C++

2017-04-15 Thread Marton Balint
On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 1:23 PM, Hendrik Leppkes wrote: [...] From 00fdc9d15414a92a155eb7d1394bac3736dc9405 Mon Sep 17 00:00:00 2001 From: Aaron Levinson Date: Thu,

Re: [FFmpeg-devel] Added require fallback for libmfx in the case that pkg-config cannot find libmfx

2017-04-15 Thread Ricardo Constantino
On 15 April 2017 at 02:51, Aaron Levinson wrote: > From e0c73c054add0137901d0bf7a7893e42e7e566c8 Mon Sep 17 00:00:00 2001 > From: Aaron Levinson > Date: Fri, 14 Apr 2017 18:38:37 -0700 > Subject: [PATCH] Added require fallback for libmfx in the case that > pkg-config cannot find libmfx > > Purp

[FFmpeg-devel] [PATCH] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-15 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 1 + doc/general.texi| 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/mscc.c | 220

Re: [FFmpeg-devel] [PATCHv3] avfilter/vf_framerate: always request input if no output is provided in request_frame

2017-04-15 Thread Marton Balint
On Wed, 12 Apr 2017, Nicolas George wrote: Le tridi 23 germinal, an CCXXV, Marton Balint a écrit : Note that the above code which I moved returns 0 if ff_request_frame returns AVERROR_EOF. As far as I see, this can only happen a few times, not infinitely, so I am not sure if this is an error

Re: [FFmpeg-devel] [PATCH 2/6] avcodec/h264: change some labels to be macro-local

2017-04-15 Thread Ronald S. Bultje
Hi, On Fri, Apr 14, 2017 at 9:46 PM, James Darnley wrote: > The labels get stripped leading to (slightly) nicer disassembly from > objdump. > --- > libavcodec/x86/h264_idct.asm | 24 > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/libavcodec/x86/h26

Re: [FFmpeg-devel] Added require fallback for libmfx in the case that pkg-config cannot find libmfx

2017-04-15 Thread Aaron Levinson
On 4/15/2017 4:25 AM, Ricardo Constantino wrote: > On 15 April 2017 at 02:51, Aaron Levinson wrote: > >> From e0c73c054add0137901d0bf7a7893e42e7e566c8 Mon Sep 17 00:00:00 2001 >> From: Aaron Levinson >> Date: Fri, 14 Apr 2017 18:38:37 -0700 >> Subject: [PATCH] Added require fallback for libmfx i

Re: [FFmpeg-devel] [PATCH] Made minor changes to get the decklink avdevice code to build using Visual C++

2017-04-15 Thread Aaron Levinson
On 4/15/2017 4:19 AM, Marton Balint wrote: On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 1:23 PM, Hendrik Leppkes wrote: [...] From 00fdc9d15414a92a155eb7d1394bac3736dc9405 Mon Sep 17 00:

Re: [FFmpeg-devel] [PATCH] ffserver: fix memory leaks pointed out by valgrind.

2017-04-15 Thread wm4
On Sat, 15 Apr 2017 09:47:58 +0200 Zalewa PL wrote: > Hello, > > > The plan is that if ffserver is not fixed before the next bump, it will > > be removed. > > ffserver has been serving our purpose in a 24/7 fashion for about 7 > years now. It will probably be serving us for the foreseeable f

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread wm4
On Sat, 15 Apr 2017 07:55:50 +0200 Carl Eugen Hoyos wrote: > 2017-04-15 2:35 GMT+02:00 wm4 : > > > Legally, all these copyright headers are meaningless, as long as the > > license is correct. > > You are joking, right? Well, are you joking? While it may seem so, you apparently aren't. > > I

[FFmpeg-devel] [PATCH] add Falcom Xanadu demuxer

2017-04-15 Thread James Darnley
--- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/falcom_xa.c | 98 3 files changed, 100 insertions(+) create mode 100644 libavformat/falcom_xa.c diff --git a/libavformat/Makefile b/libavformat/Makefile index 6bdfb

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread Jean-Baptiste Kempf
Hi, On Sat, 15 Apr 2017, at 02:35, wm4 wrote: > > I do remember that in 2014 Ronald was pressuring Libav (e.g. Vittorio) > > about proper attribution to the changes done by multiple Libav developers > > when > > merging ffvp9. (AKA who exactly changed what). > > Legally, all these copyright head

Re: [FFmpeg-devel] Added require fallback for libmfx in the case that pkg-config cannot find libmfx

2017-04-15 Thread Ricardo Constantino
On 15 April 2017 at 13:41, Aaron Levinson wrote: > Doesn't require() already do that? Here is the contents of require(): > > require(){ > log require "$@" > name_version="$1" > headers="$2" > func="$3" > shift 3 > check_lib "$headers" $func "$@" || die "ERROR: $name_versi

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread Carl Eugen Hoyos
2017-04-15 15:06 GMT+02:00 wm4 : > On Sat, 15 Apr 2017 07:55:50 +0200 > Carl Eugen Hoyos wrote: > >> 2017-04-15 2:35 GMT+02:00 wm4 : >> >> > Legally, all these copyright headers are meaningless, as long as the >> > license is correct. >> >> You are joking, right? > > Well, are you joking? While it

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread Carl Eugen Hoyos
2017-04-15 15:53 GMT+02:00 Jean-Baptiste Kempf : > Hi, > > On Sat, 15 Apr 2017, at 02:35, wm4 wrote: >> > I do remember that in 2014 Ronald was pressuring Libav (e.g. Vittorio) >> > about proper attribution to the changes done by multiple Libav developers >> > when >> > merging ffvp9. (AKA who exa

Re: [FFmpeg-devel] [PATCH 2/6] avcodec/h264: change some labels to be macro-local

2017-04-15 Thread James Darnley
On 2017-04-15 14:29, Ronald S. Bultje wrote: > Hi, > > On Fri, Apr 14, 2017 at 9:46 PM, James Darnley wrote: > >> The labels get stripped leading to (slightly) nicer disassembly from >> objdump. >> --- >> libavcodec/x86/h264_idct.asm | 24 >> 1 file changed, 12 insertio

Re: [FFmpeg-devel] [PATCH] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-15 Thread Carl Eugen Hoyos
2017-04-15 13:49 GMT+02:00 Paul B Mahol : > +case 32: avctx->pix_fmt = AV_PIX_FMT_BGRA; break; (I am thankful for this patch and, yes, I believe we had this discussion before.) If this is a screen capture codec, I believe BGR0 is the correct colorspace. I understand that you tested that the

Re: [FFmpeg-devel] [PATCH] add Falcom Xanadu demuxer

2017-04-15 Thread James Darnley
On 2017-04-15 15:36, James Darnley wrote: > add Falcom Xanadu demuxer I mean Xanadu Next, not the original one. signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listi

Re: [FFmpeg-devel] [PATCH 2/6] avcodec/h264: change some labels to be macro-local

2017-04-15 Thread Henrik Gramner
What about just using strip -x on the assembly files to discard the local symbols? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] add Falcom Xanadu demuxer

2017-04-15 Thread James Almer
On 4/15/2017 10:36 AM, James Darnley wrote: > --- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/falcom_xa.c | 98 > > 3 files changed, 100 insertions(+) > create mode 100644 libavformat/falcom_xa.c Changelog

Re: [FFmpeg-devel] [PATCH] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-15 Thread Paul B Mahol
On 4/15/17, Carl Eugen Hoyos wrote: > 2017-04-15 13:49 GMT+02:00 Paul B Mahol : > >> +case 32: avctx->pix_fmt = AV_PIX_FMT_BGRA; break; > > (I am thankful for this patch and, yes, I believe we had this > discussion before.) > If this is a screen capture codec, I believe BGR0 is the correct >

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread wm4
On Sat, 15 Apr 2017 16:33:41 +0200 Carl Eugen Hoyos wrote: > 2017-04-15 15:06 GMT+02:00 wm4 : > > On Sat, 15 Apr 2017 07:55:50 +0200 > > Carl Eugen Hoyos wrote: > > > >> 2017-04-15 2:35 GMT+02:00 wm4 : > >> > >> > Legally, all these copyright headers are meaningless, as long as the > >> > li

Re: [FFmpeg-devel] [PATCH] add Falcom Xanadu demuxer

2017-04-15 Thread James Darnley
On 2017-04-15 17:56, James Almer wrote: > On 4/15/2017 10:36 AM, James Darnley wrote: >> --- >> libavformat/Makefile | 1 + >> libavformat/allformats.c | 1 + >> libavformat/falcom_xa.c | 98 >> >> 3 files changed, 100 insertions(+) >> crea

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread wm4
On Sat, 15 Apr 2017 15:53:36 +0200 "Jean-Baptiste Kempf" wrote: > > Recently, I contacted the same author about relicensing the > > same MPlayer code (of which af_pan.c was a subset of) to LGPL, and he > > explicitly denied relicensing. So if I had been cehoyos, I probably > > wouldn't shut up ab

[FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-15 Thread Marton Balint
Fixes Coverity CID 1404841. Signed-off-by: Marton Balint --- ffmpeg.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index e4b94b2..dd179b0 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -1904,8 +1904,6 @@ static void flush_encoders(void)

Re: [FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

2017-04-15 Thread Ronald S. Bultje
Hi, On Fri, Apr 14, 2017 at 8:14 PM, Ivan Kalvachev wrote: > I do remember that in 2014 Ronald was pressuring Libav (e.g. Vittorio) [..] Keep me out of this discussion, I want nothing to do with this. Ronald ___ ffmpeg-devel mailing list ffmpeg-deve

Re: [FFmpeg-devel] [PATCH 1/6] avformat/movenc: always check for new extradata on a packet

2017-04-15 Thread Michael Niedermayer
On Thu, Apr 13, 2017 at 04:54:12PM -0300, James Almer wrote: > Don't just look at zero sized packets, and also check for AAC extradata > updates, in preparation for the following patches. > > Signed-off-by: James Almer > --- > libavformat/movenc.c | 34 ++ > 1 fil

[FFmpeg-devel] [PATCH v2] web/index: add news entry for release 3.3

2017-04-15 Thread Rostislav Pehlivanov
Signed-off-by: Rostislav Pehlivanov --- src/index | 43 +++ 1 file changed, 43 insertions(+) diff --git a/src/index b/src/index index c203676..79c6627 100644 --- a/src/index +++ b/src/index @@ -37,6 +37,49 @@ News + April 13th, 2017, FFmpeg 3.3

Re: [FFmpeg-devel] [PATCH v2] web/index: add news entry for release 3.3

2017-04-15 Thread Michael Niedermayer
On Sun, Apr 16, 2017 at 12:40:16AM +0100, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > src/index | 43 +++ > 1 file changed, 43 insertions(+) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC7

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-15 Thread James Almer
On 4/15/2017 7:41 AM, Marton Balint wrote: > > On Thu, 13 Apr 2017, Aaron Levinson wrote: > >> On 4/13/2017 3:40 PM, Marton Balint wrote: >>> >>> On Thu, 13 Apr 2017, Aaron Levinson wrote: >>> On 4/13/2017 2:12 AM, Hendrik Leppkes wrote: > On Thu, Apr 13, 2017 at 10:36 AM, Aaron Levinson