Re: [FFmpeg-devel] [PATCH 2/2] lavf/utils: bail early if we don't see any packets in an MPEGTS stream

2017-05-02 Thread Michael Niedermayer
On Mon, May 01, 2017 at 08:42:18PM -0500, Rodger Combs wrote: > --- > libavformat/utils.c | 18 +- > 1 file changed, 17 insertions(+), 1 deletion(-) this breaks ./ffmpeg -i ~/tickets/3673/no-video-stream_cut.flv -an -vcodec copy -f framecrc -

Re: [FFmpeg-devel] [PATCH 2/2] lavf/utils: bail early if we don't see any packets in an MPEGTS stream

2017-05-02 Thread Matthias Hunstock
Am 02.05.2017 um 11:07 schrieb Rodger Combs: > This doesn't apply to subtitle streams, and audio or video streams generally > have at least several packets per second, so streams that are actually active > _shouldn't_ be affected even with the smaller value. I think this is a dangerous

Re: [FFmpeg-devel] [PATCH 2/2] lavf/utils: bail early if we don't see any packets in an MPEGTS stream

2017-05-02 Thread Rodger Combs
> On May 2, 2017, at 03:59, Matthias Hunstock wrote: > > Am 02.05.2017 um 03:42 schrieb Rodger Combs: >> +max_empty_analyze_duration = max_analyze_duration; >> if (!max_analyze_duration) { >> +max_empty_analyze_duration = >>

Re: [FFmpeg-devel] [PATCH 2/2] lavf/utils: bail early if we don't see any packets in an MPEGTS stream

2017-05-02 Thread Matthias Hunstock
Am 02.05.2017 um 03:42 schrieb Rodger Combs: > +max_empty_analyze_duration = max_analyze_duration; > if (!max_analyze_duration) { > +max_empty_analyze_duration = > max_stream_analyze_duration = > max_analyze_duration= 5*AV_TIME_BASE; >

[FFmpeg-devel] [PATCH 2/2] lavf/utils: bail early if we don't see any packets in an MPEGTS stream

2017-05-01 Thread Rodger Combs
--- libavformat/utils.c | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index ba82a766dc..4028d8dbcb 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -3505,6 +3505,8 @@ int