Re: [FFmpeg-devel] [PATCH WIP 0/2] Update usage of SDL1 library to SDL2

2016-05-18 Thread Clément Bœsch
On Tue, May 17, 2016 at 06:00:21PM +0100, Josh de Kock wrote: [...] > Should I be replacing sdl.c instead of creating sdl2.c? This comes > under a more semantic question: should it be called SDL2? I'd say replace sdl.c and use .name = "sdl,sdl2"? It would be a shame to keep the SDL1 dependency an

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Paul B Mahol
On 5/18/16, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows to mux raw G.729 as supported by our G.729 demuxer. > > Please comment, Carl Eugen > Please attach actual patch with commit log message. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpe

[FFmpeg-devel] [PATCH] avformat: Add Pro-MPEG CoP #3-R2 FEC protocol

2016-05-18 Thread Vlad Tarca
Fixed a type issue in xor added after the last modification and cleaned up unnecessary NULL checks for av_dict_free Signed-off-by: Vlad Tarca --- Changelog | 1 + doc/general.texi| 1 + doc/protocols.texi | 35 libavformat/Makefile| 1 + libavformat/pr

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-18 Thread Michael Niedermayer
On Tue, May 17, 2016 at 06:49:44AM -0400, Ronald S. Bultje wrote: > Hi, > > On Tue, May 17, 2016 at 6:07 AM, Michael Niedermayer > wrote: > > > Most of the major public structures contain an AVClass. > > > What are you hoping to accomplish with this struct that can currently not > be done beca

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Derek Buitenhuis
On 5/18/2016 1:10 PM, Paul B Mahol wrote: > Please attach actual patch with commit log message. I would agree, but for some reason Carl feels he is exempt from the FFmpeg developer rules. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > Please attach actual patch with commit log message. As soon as you give a helpful comment for the Babyphone patch;-) Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Carl Eugen Hoyos
Derek Buitenhuis gmail.com> writes: > I would agree, but for some reason Carl feels he is exempt > from the FFmpeg developer rules. I should probably add that this is an interesting comment coming from somebody who breaks FFmpeg with nearly every commit and not only absolutely refuses to work

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Carl Eugen Hoyos
Derek Buitenhuis gmail.com> writes: [...] Since you indicated lately that you are so terribly interested in the tone here: Please stop making such comments! Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/l

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Derek Buitenhuis
On 5/18/2016 2:30 PM, Carl Eugen Hoyos wrote: > Since you indicated lately that you are so terribly interested > in the tone here: > Please stop making such comments! It's a legitimate criticism. You refuse to follow the FFmpeg developer rules of using git format-patch. - Derek _

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Derek Buitenhuis
On 5/18/2016 2:32 PM, Carl Eugen Hoyos wrote: > I should probably add that this is an interesting comment > coming from somebody who breaks FFmpeg with nearly every > commit and not only absolutely refuses to work on fixes but > even blocks fixes if others want to help him. Frankly, this is not

Re: [FFmpeg-devel] [PATCH] avformat: Add Pro-MPEG CoP #3-R2 FEC protocol

2016-05-18 Thread Kieran Kunhya
On Wed, 18 May 2016 at 13:52 Vlad Tarca wrote: > Fixed a type issue in xor added after the last modification and cleaned up > unnecessary NULL checks for av_dict_free > > Hi, Have you tested this on actual receivers? Are you aware this won't work effectively because of the bursty nature of FFmpe

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread compn
On Wed, 18 May 2016 14:41:46 +0100 Derek Buitenhuis wrote: > On 5/18/2016 2:30 PM, Carl Eugen Hoyos wrote: > > Since you indicated lately that you are so terribly interested > > in the tone here: > > Please stop making such comments! > > It's a legitimate criticism. You refuse to follow the FFm

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread compn
On Wed, 18 May 2016 14:10:57 +0200 Paul B Mahol wrote: > On 5/18/16, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch allows to mux raw G.729 as supported by our G.729 > > demuxer. > > > > Please comment, Carl Eugen > > > > Please attach actual patch with commit log message. commit messa

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Paul B Mahol
Dana 18. 5. 2016. 15:29 osoba "Carl Eugen Hoyos" napisala je: > > Paul B Mahol gmail.com> writes: > > > Please attach actual patch with commit log message. > > As soon as you give a helpful comment for the Babyphone patch;-) > Now it is official: I'm calling meeting to ban you from FFmpeg. We ca

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread wm4
On Wed, 18 May 2016 13:32:13 + (UTC) Carl Eugen Hoyos wrote: > Derek Buitenhuis gmail.com> writes: > > > I would agree, but for some reason Carl feels he is exempt > > from the FFmpeg developer rules. > > I should probably add that this is an interesting comment > coming from somebody w

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Paul B Mahol
Dana 18. 5. 2016. 16:05 osoba "compn" napisala je: > > On Wed, 18 May 2016 14:10:57 +0200 > Paul B Mahol wrote: > > > On 5/18/16, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch allows to mux raw G.729 as supported by our G.729 > > > demuxer. > > > > > > Please comment, Carl Eugen >

Re: [FFmpeg-devel] [PATCH] avformat: Add Pro-MPEG CoP #3-R2 FEC protocol

2016-05-18 Thread Vlad Tarca
> > Have you tested this on actual receivers? > Are you aware this won't work effectively because of the bursty nature of > FFmpeg? > > Kieran > Hello, I tested the FFmpeg version on a software receiver, not a hardware one. I initially implemented this for one of my applications based on VideoLAN

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Michael Niedermayer
On Wed, May 18, 2016 at 01:32:13PM +, Carl Eugen Hoyos wrote: > Derek Buitenhuis gmail.com> writes: > > > I would agree, but for some reason Carl feels he is exempt > > from the FFmpeg developer rules. > > I should probably add that this is an interesting comment > coming from somebody who

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread wm4
On Wed, 18 May 2016 10:05:41 -0400 compn wrote: > On Wed, 18 May 2016 14:10:57 +0200 > Paul B Mahol wrote: > > > On 5/18/16, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch allows to mux raw G.729 as supported by our G.729 > > > demuxer. > > > > > > Please comment, Carl Eugen >

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread compn
On Wed, 18 May 2016 16:12:40 +0200 Paul B Mahol wrote: > Dana 18. 5. 2016. 16:05 osoba "compn" napisala je: > > > > On Wed, 18 May 2016 14:10:57 +0200 > > Paul B Mahol wrote: > > > > > On 5/18/16, Carl Eugen Hoyos wrote: > > > > Hi! > > > > > > > > Attached patch allows to mux raw G.729 as sup

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Ronald S. Bultje
Hi, On Wed, May 18, 2016 at 10:25 AM, compn wrote: > On Wed, 18 May 2016 16:12:40 +0200 > Paul B Mahol wrote: > > > Dana 18. 5. 2016. 16:05 osoba "compn" napisala je: > > > > > > On Wed, 18 May 2016 14:10:57 +0200 > > > Paul B Mahol wrote: > > > > > > > On 5/18/16, Carl Eugen Hoyos wrote: >

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread compn
On Wed, 18 May 2016 16:24:53 +0200 wm4 wrote: > That's why it's important. But he keeps ignoring our requests to > change this, mostly because he disrespects us for working with his > sworn arch-enemy, Libav. comments like this are not helpful. -compn ___

Re: [FFmpeg-devel] [PATCH] avcodec: add properties for lossless to AVCodecParameters

2016-05-18 Thread Michael Niedermayer
On Mon, May 09, 2016 at 02:29:50PM +0200, Michael Niedermayer wrote: > On Sun, May 08, 2016 at 05:42:13PM +0200, Hendrik Leppkes wrote: > > On Sun, May 8, 2016 at 5:33 PM, Michael Niedermayer > > wrote: > > > On Sun, May 08, 2016 at 04:10:01PM +0200, wm4 wrote: > > >> On Sun, 8 May 2016 12:10:07

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Nicolas George
Le decadi 30 floréal, an CCXXIV, Derek Buitenhuis a écrit : > Frankly, this is nothing nothing but an ad hominem attack. > > I am tired of it. This happens constantly, and the community > is always silent[1], and far too accepting of such toxic behavior. > > It has given me far too much unneeded

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-18 Thread Paul B Mahol
Dana 16. 5. 2016. 10:26 osoba "Paul B Mahol" napisala je: > > On 5/15/16, Kyle Swanson wrote: > > On Sat, May 14, 2016 at 5:49 PM, Paul B Mahol wrote: > >> On 5/14/16, Kyle Swanson wrote: > >>> Hi, > >>> > >>> If nobody else has any changes, can this please be pushed? > >> > >> What about remov

Re: [FFmpeg-devel] Tee improvement - discussion

2016-05-18 Thread Jan Sebechlebsky
Hello Marton, sorry for a delayed reply. On 05/16/2016 01:20 AM, Marton Balint wrote: On Wed, 11 May 2016, Jan Sebechlebsky wrote: Hi, I'll be working on tee muxer improvement during GSoC 2016 and I thought maybe it is a good idea to ask about ideas which any of you might have regarding wha

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread James Almer
On 5/18/2016 10:29 AM, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> Please attach actual patch with commit log message. > > As soon as you give a helpful comment for the Babyphone patch;-) He asked you to follow the FFmpeg patch submit rules, which requires the usage of git fo

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Michael Niedermayer
On Wed, May 18, 2016 at 12:30:25PM -0300, James Almer wrote: > On 5/18/2016 10:29 AM, Carl Eugen Hoyos wrote: > > Paul B Mahol gmail.com> writes: > > > >> Please attach actual patch with commit log message. > > > > As soon as you give a helpful comment for the Babyphone patch;-) > > He asked yo

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Michael Niedermayer
On Wed, May 18, 2016 at 01:32:13PM +, Carl Eugen Hoyos wrote: > Derek Buitenhuis gmail.com> writes: > > > I would agree, but for some reason Carl feels he is exempt > > from the FFmpeg developer rules. > > I should probably add that this is an interesting comment > coming from somebody who

[FFmpeg-devel] [PATCH] doc/developer.texi: Add a code of conduct

2016-05-18 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- doc/developer.texi | 29 + 1 file changed, 29 insertions(+) diff --git a/doc/developer.texi b/doc/developer.texi index 6db93ce..4d3a7ae 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -403,6 +403,35 @@ finding a new

[FFmpeg-devel] [Vote] Code of Conduct

2016-05-18 Thread Michael Niedermayer
This is the version i had in my pending branch and should be the last version of the Code of Conduct from march, IIRC there where no further comments on the last version, so iam calling everyone to vote on this. Everyone because it should be binding to everyone. Kieran and Thilo asked for a form

Re: [FFmpeg-devel] [PATCH] doc/developer.texi: Add a code of conduct

2016-05-18 Thread Thilo Borgmann
Am 18.05.16 um 20:40 schrieb Michael Niedermayer: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 29 + > 1 file changed, 29 insertions(+) I agree. -Thilo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmp

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacenc_is: Assert that minthr is not 0.0, this would lead to division by 0 later

2016-05-18 Thread Claudio Freire
On Mon, May 16, 2016 at 4:10 PM, Michael Niedermayer wrote: > yes, > > with the patch fate fails: > > Test aac-pred-encode failed. Look at tests/data/fate/aac-pred-encode.err for > details. > make: *** [fate-aac-pred-encode] Error 134 > > >> >> A threshold of 0 would in theory cause a zeroed band

Re: [FFmpeg-devel] [PATCH] doc/developer.texi: Add a code of conduct

2016-05-18 Thread Jan Ekstrom
On Wed, May 18, 2016 at 9:40 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 29 + > 1 file changed, 29 insertions(+) > I agree to this addition. I think having this is a good initial stepping stone for any future im

[FFmpeg-devel] IRC meeting

2016-05-18 Thread Paul B Mahol
Hi, I want to propose to have an FFmpeg IRC meeting on the Saturday of the next week, Saturday May 28, UTC 17. Candidate topics of the day: - Code of Conduct and policy around it - technical development issues - misc topics Feel free to propose other topics. Best regards. ___

Re: [FFmpeg-devel] [PATCH] avformat/tee: Move to new BSF API

2016-05-18 Thread Jan Sebechlebsky
Hello Nicolas, I am sorry for a delayed reply, On 05/12/2016 12:17 PM, Nicolas George wrote: Le quartidi 24 floréal, an CCXXIV, Jan Sebechlebsky a écrit : I can change it to array, the advantage of using linked list was that number of bitstream filters used is not known before arguments are par

Re: [FFmpeg-devel] [PATCH] avfilter: add loudnorm

2016-05-18 Thread Paul B Mahol
On 5/18/16, Paul B Mahol wrote: > Dana 16. 5. 2016. 10:26 osoba "Paul B Mahol" napisala > je: >> >> On 5/15/16, Kyle Swanson wrote: >> > On Sat, May 14, 2016 at 5:49 PM, Paul B Mahol wrote: >> >> On 5/14/16, Kyle Swanson wrote: >> >>> Hi, >> >>> >> >>> If nobody else has any changes, can this

[FFmpeg-devel] [PATCH] ffmpeg: don't fail to explicitly set a decoder for an unknown stream type

2016-05-18 Thread Rodger Combs
--- ffmpeg_opt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index c217462..84b312d 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -618,7 +618,7 @@ static AVCodec *find_codec_or_die(const char *name, enum AVMediaType type, int e av_l

Re: [FFmpeg-devel] does the latest ffmpeg support h264 SVC

2016-05-18 Thread Moritz Barsnick
On Wed, May 18, 2016 at 14:50:32 +0800, qw wrote: > Does the latest ffmpeg support H.264 SVC? No. > Why ffmpeg doesn't support H.264 SVC? Because nobody has bothered to implement it yet. Are you volunteering? ;-) The next question: Why has nobody bothered? Because there seems to be little deman

Re: [FFmpeg-devel] [PATCH] ffmpeg: don't fail to explicitly set a decoder for an unknown stream type

2016-05-18 Thread Michael Niedermayer
On Wed, May 18, 2016 at 04:37:57PM -0500, Rodger Combs wrote: > --- > ffmpeg_opt.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > index c217462..84b312d 100644 > --- a/ffmpeg_opt.c > +++ b/ffmpeg_opt.c > @@ -618,7 +618,7 @@ static AVCodec *

[FFmpeg-devel] [PATCH] web: remove myself from consulting

2016-05-18 Thread Clément Bœsch
I get too much mails, on a mailbox I forget to check, about issues I'm most of the time not willing to address. And I feel bad about not answering anymore lately. --- src/consulting | 39 --- 1 file changed, 12 insertions(+), 27 deletions(-) diff --git a/src/co

Re: [FFmpeg-devel] [PATCH] web: remove myself from consulting

2016-05-18 Thread Thilo Borgmann
Hi, Am 19.05.16 um 08:37 schrieb Clément Bœsch: > I get too much mails, on a mailbox I forget to check, about issues I'm > most of the time not willing to address. And I feel bad about not > answering anymore lately. > --- > src/consulting | 39 --- > 1 file ch

Re: [FFmpeg-devel] [PATCH] web: remove myself from consulting

2016-05-18 Thread Clément Bœsch
On Thu, May 19, 2016 at 08:51:24AM +0200, Thilo Borgmann wrote: > Hi, > > Am 19.05.16 um 08:37 schrieb Clément Bœsch: > > I get too much mails, on a mailbox I forget to check, about issues I'm > > most of the time not willing to address. And I feel bad about not > > answering anymore lately. > > -

Re: [FFmpeg-devel] [PATCH] doc/developer.texi: Add a code of conduct

2016-05-18 Thread Tobias Rapp
On 18.05.2016 20:40, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- doc/developer.texi | 29 + 1 file changed, 29 insertions(+) diff --git a/doc/developer.texi b/doc/developer.texi index 6db93ce..4d3a7ae 100644 --- a/doc/developer.texi +++ b/doc