From 8952254ffe7f43a967a3be5fdd40e48d899c2e01 Mon Sep 17 00:00:00 2001 From: James Almer Date: Sat, 23 May 2015 14:24:34 -0300 Subject: [PATCH] libwebp: simplify AVCodec.close functions Reviewed-by: Michael Niedermayer Signed-off-by: James Almer --- libavcodec/libwebpenc.c | 5 ++++- libavcodec/libwebpenc_animencoder.c | 11 +++++------ libavcodec/libwebpenc_common.c | 7 ------- libavcodec/libwebpenc_common.h | 2 -- 4 files changed, 9 insertions(+), 16 deletions(-) diff --git a/libavcodec/libwebpenc.c b/libavcodec/libwebpenc.c index 4c22a9b04a..db96e1637b 100644 --- a/libavcodec/libwebpenc.c +++ b/libavcodec/libwebpenc.c @@ -80,7 +80,10 @@ end: static int libwebp_encode_close(AVCodecContext *avctx) { - return ff_libwebp_encode_close_common(avctx); + LibWebPContextCommon *s = avctx->priv_data; + av_frame_free(&s->ref); + + return 0; } static const AVClass class = { diff --git a/libavcodec/libwebpenc_animencoder.c b/libavcodec/libwebpenc_animencoder.c index c1e2a1cb9a..e9582016d3 100644 --- a/libavcodec/libwebpenc_animencoder.c +++ b/libavcodec/libwebpenc_animencoder.c @@ -116,12 +116,11 @@ end: static int libwebp_anim_encode_close(AVCodecContext *avctx) { - int ret = ff_libwebp_encode_close_common(avctx); - if (!ret) { - LibWebPAnimContext *s = avctx->priv_data; - WebPAnimEncoderDelete(s->enc); - } - return ret; + LibWebPAnimContext *s = avctx->priv_data; + av_frame_free(&s->cc.ref); + WebPAnimEncoderDelete(s->enc); + + return 0; } static const AVClass class = { diff --git a/libavcodec/libwebpenc_common.c b/libavcodec/libwebpenc_common.c index 373adca756..a76b6da5d6 100644 --- a/libavcodec/libwebpenc_common.c +++ b/libavcodec/libwebpenc_common.c @@ -252,10 +252,3 @@ int ff_libwebp_get_frame(AVCodecContext *avctx, LibWebPContextCommon *s, end: return ret; } - -int ff_libwebp_encode_close_common(AVCodecContext *avctx) -{ - LibWebPContextCommon *s = avctx->priv_data; - av_frame_free(&s->ref); - return 0; -} diff --git a/libavcodec/libwebpenc_common.h b/libavcodec/libwebpenc_common.h index 7e653dab2b..e74e57939e 100644 --- a/libavcodec/libwebpenc_common.h +++ b/libavcodec/libwebpenc_common.h @@ -57,8 +57,6 @@ int ff_libwebp_get_frame(AVCodecContext *avctx, LibWebPContextCommon *s, const AVFrame *frame, AVFrame **alt_frame_ptr, WebPPicture **pic_ptr); -int ff_libwebp_encode_close_common(AVCodecContext *avctx); - #define OFFSET(x) offsetof(LibWebPContextCommon, x) #define VE AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_ENCODING_PARAM static const AVOption options[] = {