On date Friday 2024-03-29 11:00:28 +0100, Stefano Sabatini wrote: > On date Friday 2024-03-29 10:26:19 +0100, Nicolas Gaullier wrote: > > Yet another probesize used to get the durations when > > estimate_timings_from_pts is required. It is aimed at users interested > > in better durations probing for itself, or because using > > avformat_find_stream_info indirectly and requiring exact values: for > > concatdec for example, especially if streamcopying above it. > > The current code is a performance trade-off that can fail to get video > > stream durations in a scenario with high bitrates and buffering for > > files ending cleanly (as opposed to live captures): the physical gap > > between the last video packet and the last audio packet is very high in > > such a case. > > > > Default behaviour is unchanged: 250k up to 250k << 6 (step by step). > > Setting this new option has two effects: > > - override the maximum probesize (currently 250k << 6) > > - reduce the number of steps to 1 instead of 6, this is to avoid > > detecting the audio "too early" and failing to reach a video packet. > > Even if a single audio stream duration is found but not the other > > audio/video stream durations, there will be a retry, so at the end the > > full user-overriden probesize will be used as expected by the user. > > > > Signed-off-by: Nicolas Gaullier <nicolas.gaullier@cji.paris> > > --- > > doc/APIchanges | 3 +++ > > doc/formats.texi | 19 ++++++++++++++++++- > > libavformat/avformat.h | 16 ++++++++++++++-- > > libavformat/demux.c | 13 ++++++++----- > > libavformat/options_table.h | 1 + > > libavformat/version.h | 2 +- > > 6 files changed, 45 insertions(+), 9 deletions(-) > > Looks good to me, will apply in a few days if there are no further > comments.
Will apply soon, thanks. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
