Re: [FFmpeg-devel] [PATCH 17/20] avcodec/snow: Remove outdated assert

[Andreas Rheinhardt <andreas.rheinhardt@outlook.com>]

Andreas Rheinhardt:
> It comes from a time before frames were refcounted; it has indeed been
> added in a follow-up commit to c13e490dce1a66d79e1f053d8a38fb9b2eb53267:
> "codec_release_buffer: fix handling of non user buffers". This type
> of check is obsolete now.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libavcodec/snow.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavcodec/snow.c b/libavcodec/snow.c
> index 7e93005492..dcd6d67ad5 100644
> --- a/libavcodec/snow.c
> +++ b/libavcodec/snow.c
> @@ -607,9 +607,6 @@ av_cold void ff_snow_common_end(SnowContext *s)
>      av_freep(&s->scratchbuf);
>  
>      for(i=0; i<MAX_REF_FRAMES; i++){
> -        if(s->last_picture[i] && s->last_picture[i]->data[0]) {
> -            av_assert0(s->last_picture[i]->data[0] != s->current_picture->data[0]);
> -        }
>          av_frame_free(&s->last_picture[i]);
>      }
>  

Will apply this patch tomorrow unless there are objections.

- Andreas

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".