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

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Thu Apr 23 21:31:20 EEST 2020


Andreas Rheinhardt:
> 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
> 
Applied.

- Andreas


More information about the ffmpeg-devel mailing list