diff mbox series

[FFmpeg-devel,05/23] fftools/ffmpeg_demux: do not set AVCodecContext.framerate

Message ID 20230531145453.20994-5-anton@khirnov.net
State Accepted
Commit c7a05ac117bc4152b27aa6127d67cb5bc0f09271
Headers show
Series [FFmpeg-devel,01/23] fftools/ffmpeg_enc: move nb_frames{dup, drop} globals into OutputStream | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Anton Khirnov May 31, 2023, 2:54 p.m. UTC 
For decoding, this field is used by the decoder to export information
to the caller; it does not make sense for the caller to set it.
---
 fftools/ffmpeg_demux.c | 3 ---
 1 file changed, 3 deletions(-)
diff mbox series

Patch

diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c
index 5c15b8bad3..5e5f106368 100644
--- a/fftools/ffmpeg_demux.c
+++ b/fftools/ffmpeg_demux.c
@@ -1194,9 +1194,6 @@  static void add_input_streams(const OptionsContext *o, Demuxer *d)
 
         switch (par->codec_type) {
         case AVMEDIA_TYPE_VIDEO:
-            // avformat_find_stream_info() doesn't set this for us anymore.
-            ist->dec_ctx->framerate = st->avg_frame_rate;
-
             MATCH_PER_STREAM_OPT(frame_rates, str, framerate, ic, st);
             if (framerate && av_parse_video_rate(&ist->framerate,
                                                  framerate) < 0) {