diff mbox series

[FFmpeg-devel,20/60] fftools/ffmpeg_mux_init: remove unused variable

Message ID D41CE2FHWKB4.2DN9FO5J9JNP5@gmail.com
State New
Headers show
Series [FFmpeg-devel,01/60] fftools/ffmpeg_opt: fix variable shadowing | expand

Commit Message

Marvin Scholz Sept. 8, 2024, 8:20 p.m. UTC 
This dict is declared and freed but nothing is ever written to it.
---
 fftools/ffmpeg_mux_init.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Anton Khirnov Sept. 11, 2024, 12:09 p.m. UTC | #1 
Quoting Marvin Scholz (2024-09-08 22:20:42)
> This dict is declared and freed but nothing is ever written to it.
> ---
>  fftools/ffmpeg_mux_init.c | 2 --
>  1 file changed, 2 deletions(-)

Very ok
diff mbox series

Patch

diff --git a/fftools/ffmpeg_mux_init.c b/fftools/ffmpeg_mux_init.c
index 37d626add6..8d475f5b45 100644
--- a/fftools/ffmpeg_mux_init.c
+++ b/fftools/ffmpeg_mux_init.c
@@ -915,7 +915,6 @@  static int streamcopy_init(const Muxer *mux, OutputStream *ost, AVDictionary **e
     uint32_t             codec_tag  = par->codec_tag;
 
     AVCodecContext      *codec_ctx  = NULL;
-    AVDictionary        *codec_opts = NULL;
 
     AVRational           fr         = ost->frame_rate;
 
@@ -1019,7 +1018,6 @@  static int streamcopy_init(const Muxer *mux, OutputStream *ost, AVDictionary **e
 
 fail:
     avcodec_free_context(&codec_ctx);
-    av_dict_free(&codec_opts);
     return ret;
 }