Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Mats Peterson
"Ronald S. Bultje" skrev: (6 mars 2016 14:37:25 CET) >Hi, > >On Sun, Mar 6, 2016 at 7:37 AM, Mats Peterson < >matsp888-at-yahoo@ffmpeg.org> wrote: > >> On 03/06/2016 01:29 PM, Ronald S. Bultje wrote: >> >>> Hi Mats, >>> >>> On Sun, Mar 6, 2016 at 6:51 AM, Mats Peterson < >>> matsp888-at-yahoo.

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Ronald S. Bultje
Hi, On Sun, Mar 6, 2016 at 7:37 AM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: > On 03/06/2016 01:29 PM, Ronald S. Bultje wrote: > >> Hi Mats, >> >> On Sun, Mar 6, 2016 at 6:51 AM, Mats Peterson < >> matsp888-at-yahoo@ffmpeg.org> wrote: >> >> On 03/06/2016 12:09 PM, Michael Nied

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Mats Peterson
On 03/06/2016 01:29 PM, Ronald S. Bultje wrote: Hi Mats, On Sun, Mar 6, 2016 at 6:51 AM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: On 03/06/2016 12:09 PM, Michael Niedermayer wrote: Once again, why are these tests unneeded? You shouldn't send the same message 5x within 2 sec

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Reimar Döffinger
On Sun, Mar 06, 2016 at 01:15:33PM +0100, Mats Peterson wrote: > On 03/06/2016 12:55 PM, Mats Peterson wrote: > >On 03/06/2016 12:51 PM, Mats Peterson wrote: > >>On 03/06/2016 12:09 PM, Michael Niedermayer wrote: > >> > >>Once again, why are these tests unneeded? > >> > >>Mats > >> > > > >avio_tell

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Ronald S. Bultje
Hi Mats, On Sun, Mar 6, 2016 at 6:51 AM, Mats Peterson < matsp888-at-yahoo@ffmpeg.org> wrote: > On 03/06/2016 12:09 PM, Michael Niedermayer wrote: > > Once again, why are these tests unneeded? You shouldn't send the same message 5x within 2 seconds. Please be patient. Why are they unneeded

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Mats Peterson
On 03/06/2016 12:55 PM, Mats Peterson wrote: On 03/06/2016 12:51 PM, Mats Peterson wrote: On 03/06/2016 12:09 PM, Michael Niedermayer wrote: Once again, why are these tests unneeded? Mats avio_tell() won't work on an unseekable stream, since it uses avio_seek(), as far as I know. Mats I

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Mats Peterson
On 03/06/2016 12:51 PM, Mats Peterson wrote: On 03/06/2016 12:09 PM, Michael Niedermayer wrote: Once again, why are these tests unneeded? Mats avio_tell() won't work on an unseekable stream, since it uses avio_seek(), as far as I know. Mats -- Mats Peterson http://matsp888.no-ip.org/~mat

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Mats Peterson
On 03/06/2016 12:09 PM, Michael Niedermayer wrote: Once again, why are these tests unneeded? Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Mats Peterson
On 03/06/2016 11:53 AM, Reimar Döffinger wrote: On Sat, Mar 05, 2016 at 06:38:40PM +0100, Michael Niedermayer wrote: diff --git a/libavformat/avienc.c b/libavformat/avienc.c index 0cfffb7..357dd34 100644 --- a/libavformat/avienc.c +++ b/libavformat/avienc.c @@ -306,8 +306,7 @@ static int avi_wri

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Michael Niedermayer
On Sun, Mar 06, 2016 at 11:53:27AM +0100, Reimar Döffinger wrote: > On Sat, Mar 05, 2016 at 06:38:40PM +0100, Michael Niedermayer wrote: > > diff --git a/libavformat/avienc.c b/libavformat/avienc.c > > index 0cfffb7..357dd34 100644 > > --- a/libavformat/avienc.c > > +++ b/libavformat/avienc.c > > @

Re: [FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-06 Thread Reimar Döffinger
On Sat, Mar 05, 2016 at 06:38:40PM +0100, Michael Niedermayer wrote: > diff --git a/libavformat/avienc.c b/libavformat/avienc.c > index 0cfffb7..357dd34 100644 > --- a/libavformat/avienc.c > +++ b/libavformat/avienc.c > @@ -306,8 +306,7 @@ static int avi_write_header(AVFormatContext *s) >

[FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

2016-03-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/avienc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavformat/avienc.c b/libavformat/avienc.c index 0cfffb7..357dd34 100644 --- a/libavformat/avienc.c +++ b/libavformat/avienc.c @@ -306,8 +306,7 @@ static int avi_