On Sat, 15 Mar 2014 03:44:15 +0100, Vittorio Giovara 
<vittorio.giov...@gmail.com> wrote:
> From: Paul B Mahol <one...@gmail.com>
> 
> ---
>  libavcodec/xbmenc.c | 19 -------------------
>  1 file changed, 19 deletions(-)
> 
> diff --git a/libavcodec/xbmenc.c b/libavcodec/xbmenc.c
> index d6657dc..3650144 100644
> --- a/libavcodec/xbmenc.c
> +++ b/libavcodec/xbmenc.c
> @@ -24,16 +24,6 @@
>  #include "internal.h"
>  #include "mathops.h"
>  
> -static av_cold int xbm_encode_init(AVCodecContext *avctx)
> -{
> -    avctx->coded_frame = av_frame_alloc();
> -    if (!avctx->coded_frame)
> -        return AVERROR(ENOMEM);
> -    avctx->coded_frame->pict_type = AV_PICTURE_TYPE_I;
> -
> -    return 0;
> -}
> -
>  static int xbm_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
>                              const AVFrame *p, int *got_packet)
>  {
> @@ -67,21 +57,12 @@ static int xbm_encode_frame(AVCodecContext *avctx, 
> AVPacket *pkt,
>      return 0;
>  }
>  
> -static av_cold int xbm_encode_close(AVCodecContext *avctx)
> -{
> -    av_freep(&avctx->coded_frame);
> -
> -    return 0;
> -}
> -
>  AVCodec ff_xbm_encoder = {
>      .name         = "xbm",
>      .long_name    = NULL_IF_CONFIG_SMALL("XBM (X BitMap) image"),
>      .type         = AVMEDIA_TYPE_VIDEO,
>      .id           = AV_CODEC_ID_XBM,
> -    .init         = xbm_encode_init,
>      .encode2      = xbm_encode_frame,
> -    .close        = xbm_encode_close,
>      .pix_fmts     = (const enum AVPixelFormat[]) { AV_PIX_FMT_MONOWHITE,
>                                                   AV_PIX_FMT_NONE },
>  };
> -- 
> 1.8.3.4 (Apple Git-47)
> 

"unused"?
This sure does not look unused.

Though that av_freep() should be replaced with av_frame_free()

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to