[FFmpeg-devel] [PATCH] avformat/webvttdec: Remove write-only variable

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Tue Apr 21 03:24:08 EEST 2020


Andreas Rheinhardt:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavformat/webvttdec.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c
> index 52579c5ed2..6c4d5f6736 100644
> --- a/libavformat/webvttdec.c
> +++ b/libavformat/webvttdec.c
> @@ -60,7 +60,7 @@ static int64_t read_ts(const char *s)
>  static int webvtt_read_header(AVFormatContext *s)
>  {
>      WebVTTContext *webvtt = s->priv_data;
> -    AVBPrint header, cue;
> +    AVBPrint cue;
>      int res = 0;
>      AVStream *st = avformat_new_stream(s, NULL);
>  
> @@ -71,7 +71,6 @@ static int webvtt_read_header(AVFormatContext *s)
>      st->codecpar->codec_id   = AV_CODEC_ID_WEBVTT;
>      st->disposition |= webvtt->kind;
>  
> -    av_bprint_init(&header, 0, AV_BPRINT_SIZE_UNLIMITED);
>      av_bprint_init(&cue,    0, AV_BPRINT_SIZE_UNLIMITED);
>  
>      for (;;) {
> @@ -166,7 +165,6 @@ static int webvtt_read_header(AVFormatContext *s)
>  
>  end:
>      av_bprint_finalize(&cue,    NULL);
> -    av_bprint_finalize(&header, NULL);
>      return res;
>  }
>  
> 
Will apply tomorrow unless there are objections.

- Andreas


More information about the ffmpeg-devel mailing list