Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-11 Thread Michael Niedermayer
On Tue, Nov 11, 2014 at 08:47:01AM +0100, Marvin Scholz wrote: > > > On 11 Nov 2014, at 0:56, Timothy Gu wrote: > > > Real name please. > > Changed the patches to used my realname now. > > > These are two unrelated changes and should therefore be split into two > > patches. > > Splitted them

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Marvin Scholz
On 11 Nov 2014, at 0:56, Timothy Gu wrote: > Real name please. Changed the patches to used my realname now. > These are two unrelated changes and should therefore be split into two > patches. Splitted them up into two different patches >> -if (NOT_EMPTY(s->content_type)) >> -av_d

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Lukasz Marek
W dniu wtorek, 11 listopada 2014 Marvin Scholz napisał(a): > Please ignore the patches I sent before ([PATCH 1/2] ffserver_config: drop > requirement videosize being multiple of 16), git send-email did > something unexpected and somehow picked the wrong ones… This mailing list > stuff drives

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Marvin Scholz
Please ignore the patches I sent before ([PATCH 1/2] ffserver_config: drop requirement video size being multiple of 16), git send-email did something unexpected and somehow picked the wrong ones… This mailing list stuff drives me crazy. ___ ffmpeg-dev

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > > From: ePirat gmail.com > > > Real name please. Could you elaborate? There are both active developers who don't use real names and copyright holders for different source files who chose to use an alias. Why shouldn't this be ok in this case? Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Timothy Gu
Hi, On Monday, November 10, 2014, wrote: > From: ePirat > Real name please. > > Send 100-continue header so we detect errors before sending data > Use a default content-type (content-type is required since Icecast 2.4.1) These are two unrelated changes and should therefore be split into tw

[FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread epirat07
Fixed space vs. tab messup ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread epirat07
From: ePirat Send 100-continue header so we detect errors before sending data Use a default content-type (content-type is required since Icecast 2.4.1) --- libavformat/icecast.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/icecast.c b/libavformat/icecast.