On 2013-01-08 15:37:20 +0100, Anton Khirnov wrote:
> ---
>  libavcodec/vcr1.c |   39 +++++----------------------------------
>  1 file changed, 5 insertions(+), 34 deletions(-)
> 
> diff --git a/libavcodec/vcr1.c b/libavcodec/vcr1.c
> index d951469..62c21f4 100644
> --- a/libavcodec/vcr1.c
> +++ b/libavcodec/vcr1.c
> @@ -30,55 +30,28 @@
>  #include "libavutil/internal.h"
>  
>  typedef struct VCR1Context {
> -    AVFrame picture;
>      int delta[16];
>      int offset[4];
>  } VCR1Context;
>  
> -static av_cold int vcr1_common_init(AVCodecContext *avctx)
> -{
> -    VCR1Context *const a = avctx->priv_data;
> -
> -    avctx->coded_frame = &a->picture;
> -
> -    return 0;
> -}
> -
>  static av_cold int vcr1_decode_init(AVCodecContext *avctx)
>  {
> -    vcr1_common_init(avctx);
> -
>      avctx->pix_fmt = AV_PIX_FMT_YUV410P;
>  
>      return 0;
>  }
>  
> -static av_cold int vcr1_decode_end(AVCodecContext *avctx)
> -{
> -    VCR1Context *s = avctx->priv_data;
> -
> -    if (s->picture.data[0])
> -        avctx->release_buffer(avctx, &s->picture);
> -
> -    return 0;
> -}
> -
>  static int vcr1_decode_frame(AVCodecContext *avctx, void *data,
>                               int *got_frame, AVPacket *avpkt)
>  {
>      const uint8_t *buf        = avpkt->data;
>      int buf_size              = avpkt->size;
>      VCR1Context *const a      = avctx->priv_data;
> -    AVFrame *picture          = data;
> -    AVFrame *const p          = &a->picture;
> +    AVFrame *const p          = data;
>      const uint8_t *bytestream = buf;
>      int i, x, y, ret;
>  
> -    if (p->data[0])
> -        avctx->release_buffer(avctx, p);
> -
> -    p->reference = 0;
> -    if ((ret = ff_get_buffer(avctx, p)) < 0) {
> +    if ((ret = ff_get_buffer(avctx, p, 0)) < 0) {
>          av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
>          return ret;
>      }
> @@ -92,11 +65,11 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void 
> *data,
>  
>      for (y = 0; y < avctx->height; y++) {
>          int offset;
> -        uint8_t *luma = &a->picture.data[0][y * a->picture.linesize[0]];
> +        uint8_t *luma = &p->data[0][y * p->linesize[0]];
>  
>          if ((y & 3) == 0) {
> -            uint8_t *cb = &a->picture.data[1][(y >> 2) * 
> a->picture.linesize[1]];
> -            uint8_t *cr = &a->picture.data[2][(y >> 2) * 
> a->picture.linesize[2]];
> +            uint8_t *cb = &p->data[1][(y >> 2) * p->linesize[1]];
> +            uint8_t *cr = &p->data[2][(y >> 2) * p->linesize[2]];
>  
>              for (i = 0; i < 4; i++)
>                  a->offset[i] = *bytestream++;
> @@ -132,7 +105,6 @@ static int vcr1_decode_frame(AVCodecContext *avctx, void 
> *data,
>          }
>      }
>  
> -    *picture   = a->picture;
>      *got_frame = 1;
>  
>      return buf_size;
> @@ -144,7 +116,6 @@ AVCodec ff_vcr1_decoder = {
>      .id             = AV_CODEC_ID_VCR1,
>      .priv_data_size = sizeof(VCR1Context),
>      .init           = vcr1_decode_init,
> -    .close          = vcr1_decode_end,
>      .decode         = vcr1_decode_frame,
>      .capabilities   = CODEC_CAP_DR1,
>      .long_name      = NULL_IF_CONFIG_SMALL("ATI VCR1"),

ok

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to