Re: [FFmpeg-devel] [FFmpeg-cvslog] Revert "mjpegenc: disable huffman coding with AMV"

2017-04-10 Thread Tobias Rapp
On 09.04.2017 22:38, Michael Niedermayer wrote: ffmpeg | branch: master | Michael Niedermayer | Sun Apr 9 21:51:33 2017 +0200| [724bb805ef8a6de0d9ef27a083ef501bdef2d453] | committer: Michael Niedermayer Revert "mjpegenc: disable huffman coding with AMV" This was the wrong patch This revert

Re: [FFmpeg-devel] [PATCH 2/2] configure: disable stripping and memory_poisoning in various toolchains

2017-04-10 Thread Clément Bœsch
On Mon, Apr 10, 2017 at 08:53:09AM +0200, wm4 wrote: > On Mon, 10 Apr 2017 07:54:41 +0200 > Clément Bœsch wrote: > > > On Mon, Apr 10, 2017 at 03:51:40AM +0200, Michael Niedermayer wrote: > > > On Sun, Apr 09, 2017 at 06:46:51PM +0200, Clément Bœsch wrote: > > > > Toolchains which target debugg

Re: [FFmpeg-devel] [PATCH 2/2] configure: disable stripping and memory_poisoning in various toolchains

2017-04-10 Thread Hendrik Leppkes
On Mon, Apr 10, 2017 at 8:53 AM, wm4 wrote: > On Mon, 10 Apr 2017 07:54:41 +0200 > Clément Bœsch wrote: > >> On Mon, Apr 10, 2017 at 03:51:40AM +0200, Michael Niedermayer wrote: >> > On Sun, Apr 09, 2017 at 06:46:51PM +0200, Clément Bœsch wrote: >> > > Toolchains which target debugging are meanin

Re: [FFmpeg-devel] [PATCH 2/2] configure: disable stripping and memory_poisoning in various toolchains

2017-04-10 Thread wm4
On Mon, 10 Apr 2017 09:28:06 +0200 Clément Bœsch wrote: > On Mon, Apr 10, 2017 at 08:53:09AM +0200, wm4 wrote: > > On Mon, 10 Apr 2017 07:54:41 +0200 > > Clément Bœsch wrote: > > > > > On Mon, Apr 10, 2017 at 03:51:40AM +0200, Michael Niedermayer wrote: > > > > On Sun, Apr 09, 2017 at 06:46

[FFmpeg-devel] [PATCH] avfilter/vf_paletteuse: silence warning about misaligned indentation

2017-04-10 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_paletteuse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_paletteuse.c b/libavfilter/vf_paletteuse.c index e8dde57..b25c6a9 100644 --- a/libavfilter/vf_paletteuse.c +++ b/libavfilter/vf_paletteuse.c @@ -444,7 +44

Re: [FFmpeg-devel] [PATCH] avfilter/vf_paletteuse: silence warning about misaligned indentation

2017-04-10 Thread Clément Bœsch
On Mon, Apr 10, 2017 at 11:27:19AM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_paletteuse.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/vf_paletteuse.c b/libavfilter/vf_paletteuse.c > index e8dde57..b25c6a9 100644 > ---

Re: [FFmpeg-devel] [RFC] Merge of Libav's bitstream API

2017-04-10 Thread Clément Bœsch
On Sun, Apr 02, 2017 at 06:58:39PM +0200, Clément Bœsch wrote: [...] > Here is a list of con arguments for merging: > > - speedlosses have been observed in various decoders, in particular on >32-bits platforms https://lists.libav.org/pipermail/libav-devel/2017-April/083283.html [...] -- C

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mjpegenc_huffman: Assert length in ff_mjpegenc_huffman_compute_bits()

2017-04-10 Thread Michael Niedermayer
On Sat, Apr 08, 2017 at 03:34:44AM +0200, Michael Niedermayer wrote: > This should help coverity see that the issues this leads to cannot occur > > Signed-off-by: Michael Niedermayer > --- > libavcodec/mjpegenc_huffman.c | 2 ++ > 1 file changed, 2 insertions(+) applied [...] -- Michael G

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/pixlet: Reorder rlen check

2017-04-10 Thread Michael Niedermayer
On Sat, Apr 08, 2017 at 03:34:45AM +0200, Michael Niedermayer wrote: > This changes nothing but is nicer looking as this checks rlen > > Maybe this helps coverity remove CID1397743 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/pixlet.c | 2 +- > 1 file changed, 1 insertion(+), 1 del

Re: [FFmpeg-devel] [PATCH] avformat/movenc: auto insert vp9_superframe bsf when needed

2017-04-10 Thread Ronald S. Bultje
Hi, On Mon, Apr 10, 2017 at 12:33 AM, James Almer wrote: > Experimental VP9 support was added to the muxer recently. > --- > libavformat/movenc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/movenc.c b/libavformat/movenc.c > index a54aa879e9..9280dc8d23 100644 > --- a/

Re: [FFmpeg-devel] [RFC] Merge of Libav's bitstream API

2017-04-10 Thread Ronald S. Bultje
Hi, On Mon, Apr 10, 2017 at 6:45 AM, Clément Bœsch wrote: > On Sun, Apr 02, 2017 at 06:58:39PM +0200, Clément Bœsch wrote: > [...] > > Here is a list of con arguments for merging: > > > > - speedlosses have been observed in various decoders, in particular on > >32-bits platforms > > https:/

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Michael Niedermayer
On Mon, Apr 10, 2017 at 08:02:34AM +0200, Clément Bœsch wrote: > On Sun, Apr 09, 2017 at 11:10:16PM -0300, James Almer wrote: > [...] > > > is this a regression ? > > > > Fate didn't always complain about this, so it's either something > > introduced by a change in our tree, or a valgrind bug intr

Re: [FFmpeg-devel] Match source video timestamp

2017-04-10 Thread Eran Kornblau
Pinging again... Thanks Eran From: Eran Kornblau Sent: Monday, April 3, 2017 12:29 PM To: 'FFmpeg development discussions and patches' Subject: RE: Match source video timestamp Ping From: Eran Kornblau Sent: Tuesday, March 28, 2017 3:13 PM To: FFmpeg development discussions and patches mail

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Clément Bœsch
On Mon, Apr 10, 2017 at 01:37:54PM +0200, Michael Niedermayer wrote: > On Mon, Apr 10, 2017 at 08:02:34AM +0200, Clément Bœsch wrote: > > On Sun, Apr 09, 2017 at 11:10:16PM -0300, James Almer wrote: > > [...] > > > > is this a regression ? > > > > > > Fate didn't always complain about this, so it'

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Michael Niedermayer
On Mon, Apr 10, 2017 at 02:04:30PM +0200, Clément Bœsch wrote: > On Mon, Apr 10, 2017 at 01:37:54PM +0200, Michael Niedermayer wrote: > > On Mon, Apr 10, 2017 at 08:02:34AM +0200, Clément Bœsch wrote: > > > On Sun, Apr 09, 2017 at 11:10:16PM -0300, James Almer wrote: > > > [...] > > > > > is this a

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Clément Bœsch
On Mon, Apr 10, 2017 at 02:19:54PM +0200, Michael Niedermayer wrote: > On Mon, Apr 10, 2017 at 02:04:30PM +0200, Clément Bœsch wrote: > > On Mon, Apr 10, 2017 at 01:37:54PM +0200, Michael Niedermayer wrote: > > > On Mon, Apr 10, 2017 at 08:02:34AM +0200, Clément Bœsch wrote: > > > > On Sun, Apr 09,

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Michael Niedermayer
On Mon, Apr 10, 2017 at 02:42:42PM +0200, Clément Bœsch wrote: > On Mon, Apr 10, 2017 at 02:19:54PM +0200, Michael Niedermayer wrote: > > On Mon, Apr 10, 2017 at 02:04:30PM +0200, Clément Bœsch wrote: > > > On Mon, Apr 10, 2017 at 01:37:54PM +0200, Michael Niedermayer wrote: > > > > On Mon, Apr 10,

Re: [FFmpeg-devel] [PATCH] avformat/movenc: auto insert vp9_superframe bsf when needed

2017-04-10 Thread James Almer
On 4/10/2017 8:17 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Apr 10, 2017 at 12:33 AM, James Almer wrote: > >> Experimental VP9 support was added to the muxer recently. >> --- >> libavformat/movenc.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/libavformat/movenc.c b/libavfo

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Clément Bœsch
On Mon, Apr 10, 2017 at 02:48:51PM +0200, Michael Niedermayer wrote: > On Mon, Apr 10, 2017 at 02:42:42PM +0200, Clément Bœsch wrote: > > On Mon, Apr 10, 2017 at 02:19:54PM +0200, Michael Niedermayer wrote: > > > On Mon, Apr 10, 2017 at 02:04:30PM +0200, Clément Bœsch wrote: > > > > On Mon, Apr 10,

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Clément Bœsch
On Mon, Apr 10, 2017 at 04:10:14PM +0200, Clément Bœsch wrote: > I reverted and I'm in the process of reporting to valgrind. Here you go: https://bugs.kde.org/show_bug.cgi?id=378622 -- Clément B. signature.asc Description: PGP signature ___ ffmpeg-de

Re: [FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

2017-04-10 Thread Michael Niedermayer
On Mon, Apr 10, 2017 at 04:14:44PM +0200, Clément Bœsch wrote: > On Mon, Apr 10, 2017 at 04:10:14PM +0200, Clément Bœsch wrote: > > I reverted and I'm in the process of reporting to valgrind. > > Here you go: > https://bugs.kde.org/show_bug.cgi?id=378622 Thanks! [...] -- Michael GnuPG fing

[FFmpeg-devel] [PATCH 2/2] lavf/tcp.c: Free allocated client URLContext in case of error.

2017-04-10 Thread Stephan Holljes
--- libavformat/tcp.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/tcp.c b/libavformat/tcp.c index 3055e48..85e500d 100644 --- a/libavformat/tcp.c +++ b/libavformat/tcp.c @@ -203,8 +203,10 @@ static int tcp_accept(URLContext *s, URLContext **c) return

[FFmpeg-devel] [PATCH 1/2] lavf/http.c: Free allocated client URLContext in case of error.

2017-04-10 Thread Stephan Holljes
--- libavformat/http.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index 293a8a7..f92549c 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -536,6 +536,9 @@ static int http_accept(URLContext *s, URLContext **c) cc->hd = cl; cc-

Re: [FFmpeg-devel] [PATCH v3] avutil/avstring: improve av_strreplace implement

2017-04-10 Thread Marton Balint
On Mon, 10 Apr 2017, Steven Liu wrote: 2017-04-10 7:09 GMT+08:00 Marton Balint : On Mon, 10 Apr 2017, Steven Liu wrote: 2017-04-10 4:39 GMT+08:00 Nicolas George : Le septidi 17 germinal, an CCXXV, Michael Niedermayer a écrit : > Its in no release, and there seems consensus that it shou

[FFmpeg-devel] [PATCHv2] ffprobe: only use custom logging callback if -show_log is set

2017-04-10 Thread Marton Balint
The custom callback can cause significant CPU usage on Windows for some large files with many index entries for some reason. v2: Move check after parsing options. Signed-off-by: Marton Balint --- ffprobe.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ffprobe.c b/ffpr

Re: [FFmpeg-devel] [PATCH] avutil/internal: Do not enable CHECKED with DEBUG

2017-04-10 Thread Michael Niedermayer
On Fri, Apr 07, 2017 at 12:58:50PM -0400, Ronald S. Bultje wrote: > Hi, > > On Fri, Apr 7, 2017 at 7:49 AM, Michael Niedermayer > wrote: > > > This avoids potential undefined behavior in debug mode while still allowing > > developers which want to check for potential additional overflows to do s

Re: [FFmpeg-devel] [PATCH v3] avutil/avstring: improve av_strreplace implement

2017-04-10 Thread Steven Liu
2017-04-11 4:16 GMT+08:00 Marton Balint : > > > On Mon, 10 Apr 2017, Steven Liu wrote: > > 2017-04-10 7:09 GMT+08:00 Marton Balint : >> >> >>> >>> On Mon, 10 Apr 2017, Steven Liu wrote: >>> >>> 2017-04-10 4:39 GMT+08:00 Nicolas George : >>> Le septidi 17 germinal, an CCXXV, Michael Niede

Re: [FFmpeg-devel] Refund request for travelling costs to Chemnitz

2017-04-10 Thread Michael Niedermayer
On Tue, Mar 28, 2017 at 02:08:30PM +0200, Thomas Volkert wrote: > > On 13.03.2017 11:04, Carl Eugen Hoyos wrote: > > Hi! > > > > Alexander Strasser, Thilo Borgmann, Thomas Volkert and myself > > represented FFmpeg at the Chemnitzer Linuxtage 2017. We showed > > different filters in action on video

[FFmpeg-devel] [PATCH] doc/APIChage: Add av_strireplace()

2017-04-10 Thread Steven Liu
Add av_strireplace() into APIChange Signed-off-by: Steven Liu --- doc/APIchanges | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index 1aced2f6a4..9fc59945ed 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -15,6 +15,9 @@ libavutil: 2015-08-28 API

Re: [FFmpeg-devel] [PATCH] doc/APIChage: Add av_strireplace()

2017-04-10 Thread James Almer
On 4/11/2017 12:00 AM, Steven Liu wrote: > Add av_strireplace() into APIChange > > Signed-off-by: Steven Liu > --- > doc/APIchanges | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/APIchanges b/doc/APIchanges > index 1aced2f6a4..9fc59945ed 100644 > --- a/doc/APIchanges > +++ b/do

Re: [FFmpeg-devel] [PATCH] doc/APIChage: Add av_strireplace()

2017-04-10 Thread Steven Liu
2017-04-11 11:09 GMT+08:00 James Almer : > On 4/11/2017 12:00 AM, Steven Liu wrote: > > Add av_strireplace() into APIChange > > > > Signed-off-by: Steven Liu > > --- > > doc/APIchanges | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/doc/APIchanges b/doc/APIchanges > > index 1ac

Re: [FFmpeg-devel] [PATCH] doc/APIChage: Add av_strireplace()

2017-04-10 Thread James Almer
On 4/11/2017 12:49 AM, Steven Liu wrote: > 2017-04-11 11:09 GMT+08:00 James Almer : > >> On 4/11/2017 12:00 AM, Steven Liu wrote: >>> Add av_strireplace() into APIChange >>> >>> Signed-off-by: Steven Liu >>> --- >>> doc/APIchanges | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a

[FFmpeg-devel] [PATCH] libavformat: add "capture:" protocol using AVIO

2017-04-10 Thread Timothy Lee
Capture is an input stream capture protocol that dumps the input stream to a file. The default name of the output file is "capture.dat", but it can be changed using the "capture_file" option. --- Changelog | 1 + doc/protocols.texi | 23 ++ libavformat/Makefile|

[FFmpeg-devel] [PATCH] doc/APIChages: Add av_strireplace()

2017-04-10 Thread Steven Liu
Add av_strireplace() into APIChanges Signed-off-by: Steven Liu --- doc/APIchanges | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index 1aced2f6a4..7359c62370 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -15,6 +15,9 @@ libavutil: 2015-08-28 AP

Re: [FFmpeg-devel] [PATCH] doc/APIChages: Add av_strireplace()

2017-04-10 Thread James Almer
On 4/11/2017 1:12 AM, Steven Liu wrote: > Add av_strireplace() into APIChanges > > Signed-off-by: Steven Liu > --- > doc/APIchanges | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/APIchanges b/doc/APIchanges > index 1aced2f6a4..7359c62370 100644 > --- a/doc/APIchanges > +++ b/do

[FFmpeg-devel] [PATCH] doc/APIChages: Add av_strireplace()

2017-04-10 Thread Steven Liu
Add av_strireplace() into APIChanges Signed-off-by: Steven Liu --- doc/APIchanges | 3 +++ libavutil/version.h | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 1aced2f6a4..7d93dd7baf 100644 --- a/doc/APIchanges +++ b/doc/APIchang

Re: [FFmpeg-devel] [PATCH] doc/APIChages: Add av_strireplace()

2017-04-10 Thread James Almer
On 4/11/2017 1:37 AM, Steven Liu wrote: > Add av_strireplace() into APIChanges > > Signed-off-by: Steven Liu > --- > doc/APIchanges | 3 +++ > libavutil/version.h | 4 ++-- > 2 files changed, 5 insertions(+), 2 deletions(-) > > diff --git a/doc/APIchanges b/doc/APIchanges > index 1aced2f6a

Re: [FFmpeg-devel] [PATCH] doc/APIChages: Add av_strireplace()

2017-04-10 Thread Steven Liu
2017-04-11 12:40 GMT+08:00 James Almer : > On 4/11/2017 1:37 AM, Steven Liu wrote: > > Add av_strireplace() into APIChanges > > > > Signed-off-by: Steven Liu > > --- > > doc/APIchanges | 3 +++ > > libavutil/version.h | 4 ++-- > > 2 files changed, 5 insertions(+), 2 deletions(-) > > > > di