[FFmpeg-devel] [PATCH] avcodec/decode: remove unused AVCodecInternal compat_decode field

[James Almer]

On 3/9/2020 6:09 AM, Hendrik Leppkes wrote:
> On Mon, Mar 9, 2020 at 3:20 AM James Almer <jamrial at gmail.com> wrote:
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/decode.c   | 1 -
>>  libavcodec/internal.h | 1 -
>>  2 files changed, 2 deletions(-)
>>
>> diff --git a/libavcodec/decode.c b/libavcodec/decode.c
>> index 03b9da25f9..0e3d7e17e4 100644
>> --- a/libavcodec/decode.c
>> +++ b/libavcodec/decode.c
>> @@ -826,7 +826,6 @@ static int compat_decode(AVCodecContext *avctx, AVFrame *frame,
>>      }
>>
>>      *got_frame = 0;
>> -    avci->compat_decode = 1;
>>
>>      if (avci->compat_decode_partial_size > 0 &&
>>          avci->compat_decode_partial_size != pkt->size) {
>> diff --git a/libavcodec/internal.h b/libavcodec/internal.h
>> index bccd9222d4..972e93bcfa 100644
>> --- a/libavcodec/internal.h
>> +++ b/libavcodec/internal.h
>> @@ -208,7 +208,6 @@ typedef struct AVCodecInternal {
>>      AVFrame *buffer_frame;
>>      int draining_done;
>>      /* set to 1 when the caller is using the old decoding API */
>> -    int compat_decode;
>>      int compat_decode_warned;
>>      /* this variable is set by the decoder internals to signal to the old
>>       * API compat wrappers the amount of data consumed from the last packet */
> 
> The comment above it seems to want to reference the removed field?

Good catch. Removed locally.

> 
> - Hendrik
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>