Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Benoit Fouet
Hi, On 12/06/2016 22:58, Paul B Mahol wrote: Hi, As requested in the IRC meeting I hereby request for the voting committee to begin voting on whatever to ban Carl Eugen Hoyos from mailing list, trac and IRC for 4 months, starting after the voting has finished. Voting will last 7 days from now

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

2016-06-13 Thread Clément Bœsch
On Mon, Jun 13, 2016 at 05:50:18AM +0200, Matt Oliver wrote: > ffmpeg | branch: master | Matt Oliver | Mon Jun 6 > 17:04:39 2016 +1000| [37787f261639c53998487400e874741c17e85fc6] | committer: > Matt Oliver > > lavf/os_support.h: Fix for unicode filenames on windows. > > Fixes #819 #5256 #5281

Re: [FFmpeg-devel] [PATCH] tools: update normalize script example to use loudnorm

2016-06-13 Thread Clément Bœsch
On Sun, Jun 12, 2016 at 10:44:15PM -0500, Kyle Swanson wrote: > Signed-off-by: Kyle Swanson > --- > tools/normalize.py | 33 -- > tools/normalize.rb | 60 > ++ > 2 files changed, 60 insertions(+), 33 deletions(-) >

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Paul B Mahol
On 6/13/16, Ivan Kalvachev wrote: > On 6/13/16, Paul B Mahol wrote: >> On 6/13/16, Ivan Kalvachev wrote: >>> On 6/12/16, Paul B Mahol wrote: Hi, As requested in the IRC meeting I hereby request for the voting committee to begin voting on whatever to ban Carl Eugen Hoyos

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Thilo Borgmann
Hi, > As requested in the IRC meeting I hereby request for the > voting committee to begin voting on whatever to ban Carl > Eugen Hoyos from mailing list, trac and IRC for 4 months, > starting after the voting has finished. I don't remember and don't have time to investigate if there has been suc

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

2016-06-13 Thread Benoit Fouet
Hi, On 13/06/2016 10:21, Clément Bœsch wrote: On Mon, Jun 13, 2016 at 05:50:18AM +0200, Matt Oliver wrote: ffmpeg | branch: master | Matt Oliver | Mon Jun 6 17:04:39 2016 +1000| [37787f261639c53998487400e874741c17e85fc6] | committer: Matt Oliver lavf/os_support.h: Fix for unicode filename

Re: [FFmpeg-devel] [PATCH] FFmpeg Decklink input connection options

2016-06-13 Thread Amnon
Is this really applied? > On 11 May 2016, at 2:14 AM, Michael Niedermayer > wrote: > > On Tue, May 10, 2016 at 11:44:00PM +0200, Deti Fliegl wrote: >> Hi, >> >> thank you. Patch approved. > > applied > > thx > > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78704

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Thilo Borgmann
Am 13.06.16 um 10:23 schrieb Paul B Mahol: > On 6/13/16, Ivan Kalvachev wrote: >> On 6/13/16, Paul B Mahol wrote: >>> On 6/13/16, Ivan Kalvachev wrote: On 6/12/16, Paul B Mahol wrote: > Hi, > > As requested in the IRC meeting I hereby request for the > voting committee to b

Re: [FFmpeg-devel] [PATCH] avfilter/graphparser: remove '\n' from parse_filter

2016-06-13 Thread Muhammad Faiz
On Sun, Jun 12, 2016 at 3:09 PM, Muhammad Faiz wrote: > On Mon, Jun 6, 2016 at 12:19 AM, Muhammad Faiz wrote: >> On Sun, Jun 5, 2016 at 12:33 AM, Michael Niedermayer >> wrote: >>> On Thu, Jun 02, 2016 at 03:32:49PM +0700, Muhammad Faiz wrote: On Thu, May 5, 2016 at 2:21 PM, Muhammad Faiz w

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-06-13 Thread Nicolas George
Le primidi 11 prairial, an CCXXIV, Michael Niedermayer a écrit : > Please state clearly if you agree to add AVClass&AVOption to > AVCodecParameters or if you disagree about adding it or if you dont > care either way Since it is necessary to break the tie, I express clearly that I am against, becau

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Nicolas George
Le quintidi 25 prairial, an CCXXIV, Paul B Mahol a écrit : > As requested in the IRC meeting I hereby request for the > voting committee to begin voting on whatever to ban Carl > Eugen Hoyos from mailing list, trac and IRC for 4 months, > starting after the voting has finished. > > Voting will las

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: use ff_h264_decode_extradata to extract PPS/SPS

2016-06-13 Thread Matthieu Bouron
On Fri, Jun 10, 2016 at 03:08:48PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > Fixes playback of HLS streams on MediaTek devices which requires PPS/SPS > to be set in their respective csd-{0,1} buffers. > --- > > Hello, > > The attached patch fixes playback of HLS streams on Media

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Clément Bœsch
On Sun, Jun 12, 2016 at 10:58:37PM +0200, Paul B Mahol wrote: > Hi, > > As requested in the IRC meeting I hereby request for the > voting committee to begin voting on whatever to ban Carl > Eugen Hoyos from mailing list, trac and IRC for 4 months, > starting after the voting has finished. > > Vot

Re: [FFmpeg-devel] [PATCH] FTP graceful close data connection to avoid server abort

2016-06-13 Thread Camille Gonnet
2016-06-09 1:19 GMT+02:00 Lukasz Marek : > On 09.06.2016 00:25, Michael Niedermayer wrote: > > On Thu, Jun 02, 2016 at 02:29:47PM +0200, Camille Gonnet wrote: > >> When writing files to FTP, if the data connection is closed before the > >> control connection, the server may handle it as an aborted

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: use ff_h264_decode_extradata to extract PPS/SPS

2016-06-13 Thread Hendrik Leppkes
On Mon, Jun 13, 2016 at 11:51 AM, Matthieu Bouron wrote: > On Fri, Jun 10, 2016 at 03:08:48PM +0200, Matthieu Bouron wrote: >> From: Matthieu Bouron >> >> Fixes playback of HLS streams on MediaTek devices which requires PPS/SPS >> to be set in their respective csd-{0,1} buffers. >> --- >> >> Hell

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Reto Kromer
Thilo Borgmann wrote: >Am 13.06.16 um 10:23 schrieb Paul B Mahol: >> On 6/13/16, Ivan Kalvachev wrote: [...] >>> Feel free to prove me wrong, with quotes. >> >> If you want quotes, ask Carl to give you all his private >> emails. > >Do you even realize what you are saying? The accused person >h

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Reto Kromer
Paul B Mahol wrote: >As requested in the IRC meeting I hereby request for the >voting committee to begin voting on whatever to ban Carl >Eugen Hoyos from mailing list, trac and IRC for 4 months, >starting after the voting has finished. If I have the right to vote, then I vote no. Best regards, R

Re: [FFmpeg-devel] [PATCH 1/2] avformat/utils: Export coded dimensions unconditionally

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 06, 2016 at 09:02:34AM +0200, Hendrik Leppkes wrote: > On Mon, Jun 6, 2016 at 9:01 AM, Hendrik Leppkes wrote: > > On Sun, Jun 5, 2016 at 5:13 AM, Michael Niedermayer > > wrote: > >> This fixes a API regression > >> Probably fixes Ticket5451 > >> > >> Signed-off-by: Michael Niedermayer

[FFmpeg-devel] [PATCH] avformat/version: Change the version bumping comment

2016-06-13 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/version.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/version.h b/libavformat/version.h index 5679cce..c5635b7 100644 --- a/libavformat/version.h +++ b/libavformat/version.h @@ -29,8 +29,8 @@ #include

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Paul B Mahol
On 6/13/16, Thilo Borgmann wrote: > Am 13.06.16 um 10:23 schrieb Paul B Mahol: >> On 6/13/16, Ivan Kalvachev wrote: >>> On 6/13/16, Paul B Mahol wrote: On 6/13/16, Ivan Kalvachev wrote: > On 6/12/16, Paul B Mahol wrote: >> Hi, >> >> As requested in the IRC meeting I hereby

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Reto Kromer
Paul B Mahol wrote: >Looks like you prefer to loose valuable developers instead >of punishing bad behaviour, so be it. We like to loose nobody, neither you nor Carl. Best regards, Reto ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpe

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Ronald S. Bultje
Hi, On Mon, Jun 13, 2016 at 4:31 AM, Thilo Borgmann wrote: > Hi, > > > As requested in the IRC meeting I hereby request for the > > voting committee to begin voting on whatever to ban Carl > > Eugen Hoyos from mailing list, trac and IRC for 4 months, > > starting after the voting has finished. >

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: use ff_h264_decode_extradata to extract PPS/SPS

2016-06-13 Thread Matthieu Bouron
On Mon, Jun 13, 2016 at 12:23:07PM +0200, Hendrik Leppkes wrote: > On Mon, Jun 13, 2016 at 11:51 AM, Matthieu Bouron > wrote: > > On Fri, Jun 10, 2016 at 03:08:48PM +0200, Matthieu Bouron wrote: > >> From: Matthieu Bouron > >> > >> Fixes playback of HLS streams on MediaTek devices which requires

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: refactor ff_AMediaCodecList_getCodecByType

2016-06-13 Thread Matthieu Bouron
On Wed, Jun 08, 2016 at 11:19:51PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > Allows to select a codec (encoder or decoder) only if it supports a > specific profile. > > Adds ff_AMediaCodecProfile_getProfileFromAVCodecContext to convert an > AVCodecContext profile to a MediaCodec

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-06-13 Thread Michael Niedermayer
On Sun, May 29, 2016 at 01:32:54AM +0200, Michael Niedermayer wrote: > Hi > > It was suggested in the IRC meeting today that i start a vote to > resolve if AVClass & AVOption should be added to AVCodecParameters > This question needs to be awnsered before the next release because > the ABI would b

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Nicolas George
Le sextidi 26 prairial, an CCXXIV, Paul B Mahol a écrit : > Looks like you prefer to loose valuable developers instead of punishing > bad behaviour, so be it. Be very careful: this message could be parsed as a threat, "satisfy my personal vendetta or I will leave". I am sure this was not what you

Re: [FFmpeg-devel] [VOTE] Ban Carl Eugen Hoyos

2016-06-13 Thread Michael Niedermayer
On Sun, Jun 12, 2016 at 10:58:37PM +0200, Paul B Mahol wrote: > Hi, > > As requested in the IRC meeting I hereby request for the > voting committee to begin voting on whatever to ban Carl > Eugen Hoyos from mailing list, trac and IRC for 4 months, > starting after the voting has finished. > > Vot

[FFmpeg-devel] [PATCH] lavc/h264_sei: fix green metadata typo

2016-06-13 Thread Clément Bœsch
From: Clément Bœsch --- I'm guessing this is what was wanted. --- libavcodec/h264_sei.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c index bdc5c9f..3c6d988 100644 --- a/libavcodec/h264_sei.c +++ b/libavcodec/h264_sei.c @@ -381,

Re: [FFmpeg-devel] [PATCH] rtpenc: packetizer for VP9 RTP payload format (draft v2)

2016-06-13 Thread Thomas Volkert
On 30.05.2016 16:41, Thomas Volkert wrote: From: Thomas Volkert --- libavformat/Makefile | 1 + libavformat/rtpenc.c | 14 + libavformat/rtpenc.h | 1 + libavformat/rtpenc_vp9.c | 54 libavformat/sdp.c| 4

[FFmpeg-devel] [PATCH] avcodec/ffv1dec: fix some unsupported pix_fmt

2016-06-13 Thread Jerome Martinez
FFV1 decoder: When checking pix_fmt mapping, some bitstreams are mapped to an incorrect pix_fmt instead of being rejected (ENOSYS). Actually, such bitstreams are not supported (FFmpeg encoder does not produce such bitstream, such bitstream may come only from another encoder for the moment).

Re: [FFmpeg-devel] [PATCH] FFmpeg Decklink input connection options

2016-06-13 Thread Marton Balint
On Mon, 13 Jun 2016, Amnon wrote: Is this really applied? I think this patch never reached the ML, and Michael actually applied a different decklink patch (Convert decklink input module to use codecpar), but replied to the wrong thread. Regards, Marton __

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1dec: fix some unsupported pix_fmt

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 13, 2016 at 07:31:15PM +0200, Jerome Martinez wrote: > FFV1 decoder: > > When checking pix_fmt mapping, some bitstreams are mapped to an > incorrect pix_fmt instead of being rejected (ENOSYS). > Actually, such bitstreams are not supported (FFmpeg encoder does not > produce such bitstre

Re: [FFmpeg-devel] [PATCH] lavc/h264_sei: fix green metadata typo

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 13, 2016 at 05:42:00PM +0200, Clément Bœsch wrote: > From: Clément Bœsch > > --- > I'm guessing this is what was wanted. > --- > libavcodec/h264_sei.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) should be ok, i found a draft that matches this but not the number of bits o

Re: [FFmpeg-devel] [PATCH 2/2] avformat/dump: Show coded dimensions again

2016-06-13 Thread Michael Niedermayer
On Sun, Jun 05, 2016 at 05:13:28AM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/dump.c |2 ++ > 1 file changed, 2 insertions(+) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster algo

Re: [FFmpeg-devel] [PATCH 04/13] lavf/mux: run AVCodec::deinit if write_header fails

2016-06-13 Thread Michael Niedermayer
On Sun, Jun 12, 2016 at 04:30:52PM -0500, Rodger Combs wrote: > --- > libavformat/mux.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) should be ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst form of inequality is to try to make

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

2016-06-13 Thread Vignesh Venkatasubramanian
On Sat, Jun 11, 2016 at 2:22 PM, Ronald S. Bultje wrote: > > 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-form

Re: [FFmpeg-devel] [PATCH] hls muxer doc: clarify segment splitting option

2016-06-13 Thread Benoit Fouet
Hi, Le 08/06/2016 11:46, Benoit Fouet a écrit : Hi, find attached a patch to $subj This would have been useful at least to me :-) Anyone against this patch? If not, can someone please apply it? Thanks, -- Ben ___ ffmpeg-devel mailing list ffmpeg

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

2016-06-13 Thread Yang Kongqun
I am thinking of naming it as vpc instead, which aligns with avc.c and hevc.c. What do you think? On Mon, Jun 13, 2016 at 11:39 AM, Vignesh Venkatasubramanian < vigneshv-at-google@ffmpeg.org> wrote: > On Sat, Jun 11, 2016 at 2:22 PM, Ronald S. Bultje > wrote: > > > > Hi, > > > > On Fri, Jun

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1dec: fix some unsupported pix_fmt

2016-06-13 Thread Jerome Martinez
Le 13/06/2016 à 19:55, Michael Niedermayer a écrit : On Mon, Jun 13, 2016 at 07:31:15PM +0200, Jerome Martinez wrote: FFV1 decoder: When checking pix_fmt mapping, some bitstreams are mapped to an incorrect pix_fmt instead of being rejected (ENOSYS). Actually, such bitstreams are not supported (

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

2016-06-13 Thread Ronald S. Bultje
Hi guys, On Mon, Jun 13, 2016 at 2:42 PM, Yang Kongqun wrote: > I am thinking of naming it as vpc instead, which aligns with avc.c and > hevc.c. What do you think? > > On Mon, Jun 13, 2016 at 11:39 AM, Vignesh Venkatasubramanian < > vigneshv-at-google@ffmpeg.org> wrote: > > > On Sat, Jun 11,

[FFmpeg-devel] [PATCH 1/5] rtpdec_vp9: Make sure to free the temp buffer on close

2016-06-13 Thread Thomas Volkert
From: Martin Storsjö Signed-off-by: Martin Storsjö --- libavformat/rtpdec_vp9.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/rtpdec_vp9.c b/libavformat/rtpdec_vp9.c index e50bede..7b1c38f 100644 --- a/libavformat/rtpdec_vp9.c +++ b/libavformat/rtpdec_vp9.c @@ -279,11 +2

[FFmpeg-devel] [PATCH 5/5] rtpdec_vp9: Support parsing the scalability structure

2016-06-13 Thread Thomas Volkert
From: Martin Storsjö We still only support one single layer though, but this allows receiving streams that have this structure present even for single layer streams. Signed-off-by: Martin Storsjö --- libavformat/rtpdec_vp9.c | 60 ++-- 1 file changed

[FFmpeg-devel] [PATCH 2/5] rtpdec_vp9: Update header parsing to spec draft 02

2016-06-13 Thread Thomas Volkert
From: Martin Storsjö Signed-off-by: Martin Storsjö --- libavformat/rtpdec_vp9.c | 34 -- 1 file changed, 12 insertions(+), 22 deletions(-) diff --git a/libavformat/rtpdec_vp9.c b/libavformat/rtpdec_vp9.c index 7b1c38f..506a5cd 100644 --- a/libavformat/rtpdec_vp9

[FFmpeg-devel] [PATCH 3/5] rtpdec: Use AVERROR_PATCHWELCOME instead of AVERROR(ENOSYS) for unimplemented features

2016-06-13 Thread Thomas Volkert
From: Martin Storsjö Signed-off-by: Martin Storsjö --- libavformat/rtpdec_h264.c | 2 +- libavformat/rtpdec_vp9.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/rtpdec_h264.c b/libavformat/rtpdec_h264.c index c5eb341..7216dd8 100644 --- a/libavformat/rtpdec_

[FFmpeg-devel] [PATCH 4/5] rtpdec_h264: Use avpriv_report_missing_feature instead of a manual av_log

2016-06-13 Thread Thomas Volkert
From: Martin Storsjö Signed-off-by: Martin Storsjö --- libavformat/rtpdec_h264.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/rtpdec_h264.c b/libavformat/rtpdec_h264.c index 7216dd8..4ab0eee 100644 --- a/libavformat/rtpdec_h264.c +++ b/libavformat/rtpdec_h2

Re: [FFmpeg-devel] MetaData in Sun AU format

2016-06-13 Thread Michael Niedermayer
On Wed, Jun 08, 2016 at 04:16:31PM +0200, miniupnp wrote: > Le 08/06/2016 15:37, Michael Niedermayer a écrit : > > On Wed, Jun 08, 2016 at 12:03:10AM +0200, miniupnp wrote: > On 6/1/16, miniupnp wrote: > > Hello, > > > > I'm using the .AU audio file format and noticed that FFmpeg

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1dec: fix some unsupported pix_fmt

2016-06-13 Thread Jerome Martinez
Le 13/06/2016 à 19:31, Jerome Martinez a écrit : [...] - JPEG 2000 RCT 9/10/12/14 bit depths with alpha are mapped to a FFmpeg pix_fmt without alpha (e.g. AV_PIX_FMT_GBRP9 for 9-bit with alpha), which is not expected. Sorry, I forgot a remark about this part of the patch: the decoder can dec

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1dec: fix some unsupported pix_fmt

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 13, 2016 at 08:44:36PM +0200, Jerome Martinez wrote: > Le 13/06/2016 à 19:55, Michael Niedermayer a écrit : > >On Mon, Jun 13, 2016 at 07:31:15PM +0200, Jerome Martinez wrote: > >>FFV1 decoder: > >> > >>When checking pix_fmt mapping, some bitstreams are mapped to an > >>incorrect pix_fm

Re: [FFmpeg-devel] [PATCH] lavc/h264_sei: fix green metadata typo

2016-06-13 Thread Clément Bœsch
On Mon, Jun 13, 2016 at 07:51:39PM +0200, Michael Niedermayer wrote: > On Mon, Jun 13, 2016 at 05:42:00PM +0200, Clément Bœsch wrote: > > From: Clément Bœsch > > > > --- > > I'm guessing this is what was wanted. > > --- > > libavcodec/h264_sei.c | 2 +- > > 1 file changed, 1 insertion(+), 1 dele

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

2016-06-13 Thread Marton Balint
On Sun, 12 Jun 2016, Tomas Härdin wrote: 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 f8

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

2016-06-13 Thread Marton Balint
On Sat, 11 Jun 2016, Michael Niedermayer wrote: 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 Thanks, applied

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

2016-06-13 Thread Marton Balint
On Sat, 11 Jun 2016, Marton Balint wrote: 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(+)

[FFmpeg-devel] [PATCH 2/4] lavc/h264_sei: reduce scope of parameters for green meta decode

2016-06-13 Thread Clément Bœsch
This is again will help the merge as ff_h264_decode_sei will not have access to H264Context anymore. --- libavcodec/h264_sei.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c index b23878c..46d1814

[FFmpeg-devel] [PATCH 3/4] lavc/h264: rename GreenMetaData to H264SEIGreenMetaData

2016-06-13 Thread Clément Bœsch
Reduces diff for the next merge with Libav. --- libavcodec/h264.c | 2 +- libavcodec/h264.h | 6 +++--- libavcodec/h264_sei.c | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 46eaec2..795e599 100644 --- a/libavcodec/h264.c +

[FFmpeg-devel] [PATCH 1/4] lavc/h264: move green meta logging out of the sei decoding

2016-06-13 Thread Clément Bœsch
This will simplify the next Libav merge where SEI decoding doesn't have access to the debug level anymore. --- libavcodec/h264.c | 30 ++ libavcodec/h264_sei.c | 33 - 2 files changed, 30 insertions(+), 33 deletions(-) diff --git a/l

[FFmpeg-devel] [PATCH 4/4] lavc/h264_sei: fix broken style around green metadata code

2016-06-13 Thread Clément Bœsch
--- libavcodec/h264.h | 10 +- libavcodec/h264_sei.c | 30 ++ 2 files changed, 19 insertions(+), 21 deletions(-) diff --git a/libavcodec/h264.h b/libavcodec/h264.h index 8c7a537..b156805 100644 --- a/libavcodec/h264.h +++ b/libavcodec/h264.h @@ -293,13 +293

Re: [FFmpeg-devel] [PATCH 3/5] rtpdec: Use AVERROR_PATCHWELCOME instead of AVERROR(ENOSYS) for unimplemented features

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 13, 2016 at 09:03:20PM +0200, Thomas Volkert wrote: > From: Martin Storsjö > > Signed-off-by: Martin Storsjö > --- > libavformat/rtpdec_h264.c | 2 +- LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who would give up essential L

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

2016-06-13 Thread Matt Oliver
On 13 June 2016 at 18:29, Benoit Fouet wrote: > Hi, > > > > On 13/06/2016 10:21, Clément Bœsch wrote: > >> On Mon, Jun 13, 2016 at 05:50:18AM +0200, Matt Oliver wrote: >> >>> ffmpeg | branch: master | Matt Oliver | Mon Jun >>> 6 17:04:39 2016 +1000| [37787f261639c53998487400e874741c17e85fc6] | >

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

2016-06-13 Thread Clément Bœsch
On Tue, Jun 14, 2016 at 07:08:58AM +1000, Matt Oliver wrote: > On 13 June 2016 at 18:29, Benoit Fouet wrote: > > > Hi, > > > > > > > > On 13/06/2016 10:21, Clément Bœsch wrote: > > > >> On Mon, Jun 13, 2016 at 05:50:18AM +0200, Matt Oliver wrote: > >> > >>> ffmpeg | branch: master | Matt Oliver

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

2016-06-13 Thread Kongqun Yang
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/Makefile | 2 +- libavformat/isom.c | 3 + libavformat

Re: [FFmpeg-devel] [PATCH] FTP graceful close data connection to avoid server abort

2016-06-13 Thread Lukasz Marek
On 02.06.2016 14:29, Camille Gonnet wrote: When writing files to FTP, if the data connection is closed before the control connection, the server may handle it as an aborted file transfer and create and leave the file empty. --- libavformat/ftp.c | 14 ++ 1 file changed, 10 inserti

Re: [FFmpeg-devel] [PATCH 4/5] rtpdec_h264: Use avpriv_report_missing_feature instead of a manual av_log

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 13, 2016 at 09:03:21PM +0200, Thomas Volkert wrote: > From: Martin Storsjö > > Signed-off-by: Martin Storsjö > --- > libavformat/rtpdec_h264.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611E

Re: [FFmpeg-devel] [PATCH 1/4] lavc/h264: move green meta logging out of the sei decoding

2016-06-13 Thread Hendrik Leppkes
On Mon, Jun 13, 2016 at 11:02 PM, Clément Bœsch wrote: > This will simplify the next Libav merge where SEI decoding doesn't have > access to the debug level anymore. This whole business looks rather fragile and wtf'ish, but its better then before and should be fine. So the entire set LGTM. __

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

2016-06-13 Thread Michael Niedermayer
On Mon, Jun 13, 2016 at 01:34:39PM +0700, Muhammad Faiz wrote: > On Sun, Jun 12, 2016 at 8:14 PM, Michael Niedermayer > wrote: > > On Sun, Jun 12, 2016 at 09:37:28AM +, Petru Rares Sincraian wrote: > >> Hi there, > >> > >> I'm sorry, I hadn't considered mingw. Here is the patch without the >

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/os_support.h: Fix for unicode filenames on windows.

2016-06-13 Thread Ricardo Constantino
This patch seems to break build with decklink in MinGW: In file included from C:/builds/ab-full/build/ffmpeg-git/libavformat/internal.h:28:0, from C:/builds/ab-full/build/ffmpeg-git/libavdevice/decklink_common.cpp:34: C:/builds/ab-full/build/ffmpeg-git/libavformat/os_support.h: In

Re: [FFmpeg-devel] [PATCH 1/5] rtpdec_vp9: Make sure to free the temp buffer on close

2016-06-13 Thread Thomas Volkert
On 13.06.2016 21:03, Thomas Volkert wrote: From: Martin Storsjö Signed-off-by: Martin Storsjö --- libavformat/rtpdec_vp9.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/rtpdec_vp9.c b/libavformat/rtpdec_vp9.c index e50bede..7b1c38f 100644 --- a/libavformat/rtpdec_vp9

Re: [FFmpeg-devel] [PATCH 1/4] lavc/h264: move green meta logging out of the sei decoding

2016-06-13 Thread Clément Bœsch
On Tue, Jun 14, 2016 at 12:01:25AM +0200, Hendrik Leppkes wrote: > On Mon, Jun 13, 2016 at 11:02 PM, Clément Bœsch wrote: > > This will simplify the next Libav merge where SEI decoding doesn't have > > access to the debug level anymore. > > This whole business looks rather fragile and wtf'ish, bu