Re: [FFmpeg-devel] [PATCH 1/2] libavformat/flvenc: refactoring: extracted method for writing codec headers

2016-06-11 Thread Michael Niedermayer
On Thu, Jun 09, 2016 at 10:12:09PM -0400, Ivan wrote: > --- > libavformat/flvenc.c | 116 > --- > 1 file changed, 64 insertions(+), 52 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH 2/2] libavformat/flvenc: support for codec configuration change mid stream

2016-06-11 Thread Michael Niedermayer
On Thu, Jun 09, 2016 at 10:12:10PM -0400, Ivan wrote: > --- > libavformat/flvenc.c | 13 + > 1 file changed, 13 insertions(+) applied This seems not tested by fate, so a fate test would be useful too thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78704

Re: [FFmpeg-devel] [PATCH] lavu/intmath.h: fix compilation with msvc10.

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 12:00:35AM +1000, Matt Oliver wrote: > On 10 June 2016 at 05:30, Michael Niedermayer > wrote: > > > On Mon, Jun 06, 2016 at 05:11:34PM +1000, Matt Oliver wrote: > > > --- > > > libavutil/x86/intmath.h | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/

Re: [FFmpeg-devel] [PATCH] FFPlay on a HLS stream skips audio from time to time.

2016-06-11 Thread Marton Balint
On Wed, 8 Jun 2016, Adrian Cruceru wrote: On Tue Jun 7 Marton Balint wrote: On Wed, 1 Jun 2016, Adrian Cruceru wrote: >/Hi, />//>/Patch is attached, />//>/Quick notes: />/- Playing HLS streams - FFPlay stalls a bit before each segment download />/- Stalls accumulate and FFPlay falls behin

[FFmpeg-devel] [PATCH 2/2] ffplay: ensure that we buffer at least 1 second of content

2016-06-11 Thread Marton Balint
In order to do that, we keep track of the total duration of packets in a packet queue. Signed-off-by: Marton Balint --- ffplay.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index 9f5b2a4..d32b023 100644 --- a/ffplay.c +++ b/ffplay.c @@ -117,6 +117

[FFmpeg-devel] [PATCH 1/2] ffplay: factorize checking if a stream needs additional packets

2016-06-11 Thread Marton Balint
Signed-off-by: Marton Balint --- ffplay.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/ffplay.c b/ffplay.c index 11c5091..9f5b2a4 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2796,6 +2796,13 @@ static int decode_interrupt_cb(void *ctx) return is->abort_requ

[FFmpeg-devel] [RFC] MAINTAINERS cleanup

2016-06-11 Thread Michael Niedermayer
Hi the MAINTAINERs file contains a bunch of inaccurate and outdated entries. What should be done about this ? should we remove everyone who was inactive in FFmpeg (aka no commit/author since 2 years) as in git log --first-parent ... ? should we mark everyone above as inactive instead like "(inact

Re: [FFmpeg-devel] [PATCH] Add support for vp9 in iso-bmff

2016-06-11 Thread Carl Eugen Hoyos
Kongqun Yang gmail.com> writes: > Implemented according to the draft specification > "VP Codec ISO Media File Format Binding": How likely is it that the draft will be changed? You may require -strict -2 in your patch when writing files to make it more acceptable. Carl Eugen _

Re: [FFmpeg-devel] [RFC] MAINTAINERS cleanup

2016-06-11 Thread Clément Bœsch
On Sat, Jun 11, 2016 at 12:57:13PM +0200, Michael Niedermayer wrote: > Hi > > the MAINTAINERs file contains a bunch of inaccurate and outdated > entries. > > What should be done about this ? > should we remove everyone who was inactive in FFmpeg > (aka no commit/author since 2 years) as in git lo

[FFmpeg-devel] [PATCH] avdevice/decklink: add support for setting duplex mode

2016-06-11 Thread Marton Balint
Signed-off-by: Marton Balint --- configure | 3 ++- doc/indevs.texi | 4 libavdevice/decklink_common.cpp | 37 + libavdevice/decklink_common.h | 4 libavdevice/decklink_common_c.h | 1 + libavdevice/decklink

Re: [FFmpeg-devel] [RFC] Bugs and CC

2016-06-11 Thread Nicolas George
Le tridi 23 prairial, an CCXXIV, Clement Boesch a écrit : > Same here. Me too. Something useful too would be to have the whole commit header in the comment when flagging a regression, instead of just "regression since ". That way, it is easier to notice we are responsible. Of course, this would m

Re: [FFmpeg-devel] [RFC] BSF list API

2016-06-11 Thread Nicolas George
Le primidi 21 prairial, an CCXXIV, Marton Balint a écrit : > My GSOC student Jan is working on improving the tee muxer, and as a side > project he is also working on converting the tee muxer to the new BSF API. > > I have checked his WIP patches, and it seems to me that developing some API > which

Re: [FFmpeg-devel] [PATCH] Add support for vp9 in iso-bmff

2016-06-11 Thread James Almer
On 6/10/2016 7:25 PM, Kongqun Yang wrote: > Implemented according to the draft specification > "VP Codec ISO Media File Format Binding": > http://www.webmproject.org/vp9/#draft-vp-codec-iso-media-file-format-binding > > Change-Id: Iaa7ddf5524b17e8d79cd1923b26f096d6e91 > --- > libavformat/Make

Re: [FFmpeg-devel] [RFC] MAINTAINERS cleanup

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 01:55:01PM +0200, Clément Bœsch wrote: > On Sat, Jun 11, 2016 at 12:57:13PM +0200, Michael Niedermayer wrote: > > Hi > > > > the MAINTAINERs file contains a bunch of inaccurate and outdated > > entries. > > > > What should be done about this ? > > should we remove everyone

Re: [FFmpeg-devel] [RFC] MAINTAINERS cleanup

2016-06-11 Thread Paul B Mahol
On 6/11/16, Michael Niedermayer wrote: > On Sat, Jun 11, 2016 at 01:55:01PM +0200, Clement Boesch wrote: >> On Sat, Jun 11, 2016 at 12:57:13PM +0200, Michael Niedermayer wrote: >> > Hi >> > >> > the MAINTAINERs file contains a bunch of inaccurate and outdated >> > entries. >> > >> > What should be

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/vf_histogram: shortcuts for histogram options

2016-06-11 Thread Paul B Mahol
On 6/11/16, Dave Rice wrote: > This patch adds some shortcuts for the histogram filter's options in a way > that is consistent with the waveform and vectorscope filter. > Dave Rice > > applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http:/

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/vf_histogram: indent histogram options

2016-06-11 Thread Paul B Mahol
On 6/11/16, Dave Rice wrote: > Some indentation after the previous patch. > Dave Rice > applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] fate: add afade tes

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 12:27:03PM +, Petru Rares Sincraian wrote: > > > Hi there, > > > Today I add a test for afade effect and acrossfade effect. acrossfade uses > afade, so I add in the same patch. I tested the test in x64 and x32 > architectures. > > > Regards, > Petru Rares. > fa

Re: [FFmpeg-devel] Patch that adds timeline function for eq filter

2016-06-11 Thread Michael Niedermayer
On Wed, Jun 01, 2016 at 04:55:45AM +, Maxim Petrov wrote: > > vf_eq.c |1 + > 1 file changed, 1 insertion(+) > 30ab72e9b52283cc69755e0205bd8c2f0c133954 > 0001-Timeline-function-for-the-eq-filter.patch > From 570b215c110b027badc41d1206b0ff6d4806a98f Mon Sep 17 00:00:00 2001 > From: Ily

Re: [FFmpeg-devel] [PATCH] fate: add afade tes

2016-06-11 Thread James Almer
On 6/11/2016 2:30 PM, Michael Niedermayer wrote: > On Sat, Jun 11, 2016 at 12:27:03PM +, Petru Rares Sincraian wrote: >> >> >> Hi there, >> >> >> Today I add a test for afade effect and acrossfade effect. acrossfade uses >> afade, so I add in the same patch. I tested the test in x64 and x32 >

[FFmpeg-devel] [PATCH 4/4] avformat/mux: do not call write_packet with a flush packet if header is not written

2016-06-11 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mux.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/mux.c b/libavformat/mux.c index b1b65a1..f21b4dd 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -823,6 +823,11 @@ int av_write_frame(AVFormatContext *s, AVPacket *pkt

[FFmpeg-devel] [PATCH 1/4] avformat/mux: call deinit if write_header fails

2016-06-11 Thread Marton Balint
Docs clearly states that av_write_trailer should only be called if avformat_write_header was successful, therefore we have to deinit if we return failure. Signed-off-by: Marton Balint --- libavformat/mux.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavformat/m

[FFmpeg-devel] [PATCH 2/4] avformat/mux: factorize header writing code

2016-06-11 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mux.c | 44 ++-- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index bef230f..08ed940 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -473,6 +473,

[FFmpeg-devel] [PATCH 3/4] avformat/mux: do not call write_header multiple times if it fails the first time

2016-06-11 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/internal.h | 1 + libavformat/mux.c | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index 40ba089..b6c2020 100644 --- a/libavformat/internal.h +++ b/libavformat/internal.h @

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: relax opt_order limit

2016-06-11 Thread Thilo Borgmann
Hi, > Am 09.06.2016 um 03:33 schrieb Michael Niedermayer : > > Fixes: Ticket5297 > > Needs review by maintainer / author to check that this is ok and sufficient > --- > libavcodec/alsdec.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) I'll have a look but I need some more

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: check if source_package is NULL

2016-06-11 Thread Marton Balint
On Tue, 31 May 2016, Marton Balint wrote: Fixes ticket #5554. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index f8cf922..9bf676c 100644 --- a/libavformat/mxfdec.c +++ b/li

Re: [FFmpeg-devel] [PATCH] fate: add afade tes

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 03:32:40PM -0300, James Almer wrote: > On 6/11/2016 2:30 PM, Michael Niedermayer wrote: > > On Sat, Jun 11, 2016 at 12:27:03PM +, Petru Rares Sincraian wrote: > >> > >> > >> Hi there, > >> > >> > >> Today I add a test for afade effect and acrossfade effect. acrossfade us

[FFmpeg-devel] [PATCH]Handle tmp on Android as on Win32

2016-06-11 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #5620 for me (the OP did not provide a testcase). Carl Eugen From 4c9d1c6f9a8f166703dc0333b53f5dba8dc5b414 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sat, 11 Jun 2016 22:41:18 +0200 Subject: [PATCH] lavu/file_open: Use current directory for temporary fi

Re: [FFmpeg-devel] [PATCH]Handle tmp on Android as on Win32

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 10:44:18PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5620 for me (the OP did not provide a testcase). > > Carl Eugen > file_open.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > b1aa303b23de1be50c57a8813ea8a16095fc93b1 > 0001-l

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mux: do not call write_header multiple times if it fails the first time

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 08:33:42PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavformat/internal.h | 1 + > libavformat/mux.c | 5 +++-- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/libavformat/internal.h b/libavformat/internal.h > index 40ba

Re: [FFmpeg-devel] [PATCH 2/4] avformat/mux: factorize header writing code

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 08:33:41PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavformat/mux.c | 44 ++-- > 1 file changed, 22 insertions(+), 22 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF1

Re: [FFmpeg-devel] [PATCH]Handle tmp on Android as on Win32

2016-06-11 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > Fixes ticket #5620. > > --- > > should be ok Patch applied. > unless someone has a better solution I'll be happy to revert in that case... Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] [PATCH] Add support for vp9 in iso-bmff

2016-06-11 Thread Ronald S. Bultje
Hi, On Sat, Jun 11, 2016 at 11:48 AM, James Almer wrote: > > +enum VpxColorSpace > > +{ > > +VPX_COLOR_SPACE_UNSPECIFIED = 0, > > +VPX_COLOR_SPACE_BT601 = 1, > > +VPX_COLOR_SPACE_BT709 = 2, > > +VPX_COLOR_SPACE_SMPTE_170 = 3, > > +VPX_COLOR_SPACE_SMPTE_240 = 4, > > +VPX_C

Re: [FFmpeg-devel] [PATCH] Add support for vp9 in iso-bmff

2016-06-11 Thread Ronald S. Bultje
Hi, On Fri, Jun 10, 2016 at 6:25 PM, Kongqun Yang wrote: > Implemented according to the draft specification > "VP Codec ISO Media File Format Binding": > > http://www.webmproject.org/vp9/#draft-vp-codec-iso-media-file-format-binding > > Change-Id: Iaa7ddf5524b17e8d79cd1923b26f096d6e91 > ---

Re: [FFmpeg-devel] [PATCH]Handle tmp on Android as on Win32

2016-06-11 Thread Hendrik Leppkes
On Sat, Jun 11, 2016 at 11:21 PM, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > >> > Fixes ticket #5620. >> > --- >> >> should be ok > > Patch applied. > >> unless someone has a better solution > > I'll be happy to revert in that case... > Maybe you should leave the pat

Re: [FFmpeg-devel] [RFC] BSF list API

2016-06-11 Thread Marton Balint
On Sat, 11 Jun 2016, Nicolas George wrote: Instead of adding a totally new API, I suggest to use a "container" bitstream filter called 'list', which will instantiate the "child" bitstream filters and pump the packets through them. Thanks to the N:M nature of the new BSF api, this should be ach

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mux: do not call write_header multiple times if it fails the first time

2016-06-11 Thread Marton Balint
On Sat, 11 Jun 2016, Michael Niedermayer wrote: On Sat, Jun 11, 2016 at 08:33:42PM +0200, Marton Balint wrote: Signed-off-by: Marton Balint --- libavformat/internal.h | 1 + libavformat/mux.c | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal

Re: [FFmpeg-devel] [PATCH]Handle tmp on Android as on Win32

2016-06-11 Thread Carl Eugen Hoyos
Hendrik Leppkes gmail.com> writes: [...] Sorry, I am not sure I understood: Do you want me to revert? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]Handle tmp on Android as on Win32

2016-06-11 Thread Hendrik Leppkes
On Sat, Jun 11, 2016 at 11:54 PM, Carl Eugen Hoyos wrote: > Hendrik Leppkes gmail.com> writes: > > [...] > > Sorry, I am not sure I understood: > Do you want me to revert? No, I want you to wait more then 30 minutes before pushing a patch in the future, especially if a comment asks for further t

Re: [FFmpeg-devel] [PATCH] fate: add afade tes

2016-06-11 Thread James Almer
On 6/11/2016 5:40 PM, Michael Niedermayer wrote: > On Sat, Jun 11, 2016 at 03:32:40PM -0300, James Almer wrote: >> On 6/11/2016 2:30 PM, Michael Niedermayer wrote: >>> On Sat, Jun 11, 2016 at 12:27:03PM +, Petru Rares Sincraian wrote: Hi there, Today I add a test f

Re: [FFmpeg-devel] [PATCH] fate: add afade tes

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 07:28:56PM -0300, James Almer wrote: > On 6/11/2016 5:40 PM, Michael Niedermayer wrote: > > On Sat, Jun 11, 2016 at 03:32:40PM -0300, James Almer wrote: > >> On 6/11/2016 2:30 PM, Michael Niedermayer wrote: > >>> On Sat, Jun 11, 2016 at 12:27:03PM +, Petru Rares Sincraia

[FFmpeg-devel] [PATCH] [RFC] libavformat/mpegts: Prevent wrapping of PTS & DTS

2016-06-11 Thread DeHackEd
Presently the mpegts demuxer passes the timestamps from received packets directly to the output AVPackets. 2^33 / 9 seconds is about 26.5 hours at which point applications start having a fit, and that's assuming timestamps begin at time 0. So here's a first revision of a patch to fix that is

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-06-11 Thread Paul B Mahol
On 2/14/16, James Almer wrote: > On 2/14/2016 12:20 PM, Carl Eugen Hoyos wrote: >> James Almer gmail.com> writes: >> >>> How do you plan to get around the fact the only working >>> version of this library is an obscure outdated fork that >>> doesn't even compile on most OSes >> >> I am sorry, I d

Re: [FFmpeg-devel] [PATCH] [RFC] libavformat/mpegts: Prevent wrapping of PTS & DTS

2016-06-11 Thread Marton Balint
On Sat, 11 Jun 2016, DeHackEd wrote: Presently the mpegts demuxer passes the timestamps from received packets directly to the output AVPackets. 2^33 / 9 seconds is about 26.5 hours at which point applications start having a fit, and that's assuming timestamps begin at time 0. So here's a

Re: [FFmpeg-devel] [PATCH] [RFC] libavformat/mpegts: Prevent wrapping of PTS & DTS

2016-06-11 Thread DeHackEd
On 06/11/2016 08:19 PM, Marton Balint wrote: > > On Sat, 11 Jun 2016, DeHackEd wrote: > >> Presently the mpegts demuxer passes the timestamps from received packets >> directly to the output AVPackets. 2^33 / 9 >> seconds is about 26.5 hours at which point applications start having a fit, >>

[FFmpeg-devel] [PATCH] swresample: add exact_rational option

2016-06-11 Thread Muhammad Faiz
give high quality resampling as good as with linear_interp=on as fast as without linear_interp=on tested visually with ffplay ffplay -f lavfi "aevalsrc='sin(1*t*t)', aresample=osr=48000, showcqt=gamma=5" ffplay -f lavfi "aevalsrc='sin(1*t*t)', aresample=osr=48000:linear_interp=on, showcqt=g

[FFmpeg-devel] [PATCH] lavf: add libopenmpt demuxer

2016-06-11 Thread Josh de Kock
--- configure| 4 + libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/libopenmpt.c | 185 +++ 4 files changed, 191 insertions(+) create mode 100644 libavformat/libopenmpt.c diff --git a/configure b/config

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mux: do not call write_header multiple times if it fails the first time

2016-06-11 Thread Michael Niedermayer
On Sat, Jun 11, 2016 at 11:54:31PM +0200, Marton Balint wrote: > > > On Sat, 11 Jun 2016, Michael Niedermayer wrote: > > >On Sat, Jun 11, 2016 at 08:33:42PM +0200, Marton Balint wrote: > >>Signed-off-by: Marton Balint > >>--- > >> libavformat/internal.h | 1 + > >> libavformat/mux.c | 5 +++

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: check if source_package is NULL

2016-06-11 Thread Tomas Härdin
On Tue, 2016-05-31 at 23:02 +0200, Marton Balint wrote: > Fixes ticket #5554. > > Signed-off-by: Marton Balint > --- >  libavformat/mxfdec.c | 2 +- >  1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index f8cf922..9bf676c 100644 > ---