Re: [FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-27 Thread Carlos Fernandez Sanz
On Mon, Sep 26, 2016 at 3:25 PM, Michael Niedermayer wrote: > > this breaks fate: > make: *** [fate-h264-skip-nokey] Error 1 Thanks - solved in V11. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-27 Thread Carlos Fernandez Sanz
On Tue, Sep 27, 2016 at 12:30 AM, Marton Balint wrote: >> +} else if (ts->pids[i] && ts->pids[i]->type == >> MPEGTS_SECTION) { >> +return ret; > > > Why do you need this hunk? I think you can delete it, and everything will > remain working. This loop flushes existing h

Re: [FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-27 Thread Marton Balint
On Mon, 26 Sep 2016, Carlos Fernandez Sanz wrote: From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 64 ++-- 1 file changed, 62 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts

Re: [FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-26 Thread Michael Niedermayer
On Mon, Sep 26, 2016 at 10:59:45AM -0700, Carlos Fernandez Sanz wrote: > From: Carlos Fernandez > > Signed-off-by: Carlos Fernandez > --- > libavformat/mpegts.c | 64 > ++-- > 1 file changed, 62 insertions(+), 2 deletions(-) > this breaks fate:

[FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-26 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 64 ++-- 1 file changed, 62 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..ad6f141 100644 --- a/libavformat/m