On Wed,  9 Jan 2013 12:11:53 -0500, Justin Ruggles <justin.rugg...@gmail.com> 
wrote:
> ---
>  libavcodec/mlpdec.c |   17 ++++++-----------
>  1 files changed, 6 insertions(+), 11 deletions(-)
> 
> diff --git a/libavcodec/mlpdec.c b/libavcodec/mlpdec.c
> index 3852f6e..63aa477 100644
> --- a/libavcodec/mlpdec.c
> +++ b/libavcodec/mlpdec.c
> @@ -115,7 +115,6 @@ typedef struct SubStream {
>  
>  typedef struct MLPDecodeContext {
>      AVCodecContext *avctx;
> -    AVFrame     frame;
>  
>      /// Current access unit being read has a major sync.
>      int         is_major_sync_unit;
> @@ -235,9 +234,6 @@ static av_cold int mlp_decode_init(AVCodecContext *avctx)
>          m->substream[substr].lossless_check_data = 0xffffffff;
>      ff_mlpdsp_init(&m->dsp);
>  
> -    avcodec_get_frame_defaults(&m->frame);
> -    avctx->coded_frame = &m->frame;
> -
>      return 0;
>  }
>  
> @@ -904,7 +900,7 @@ static void rematrix_channels(MLPDecodeContext *m, 
> unsigned int substr)
>  /** Write the audio data into the output buffer. */
>  
>  static int output_data(MLPDecodeContext *m, unsigned int substr,
> -                       void *data, int *got_frame_ptr)
> +                       AVFrame *frame, int *got_frame_ptr)
>  {

Doesn't this produce a warning? If not, then patch looks ok.

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

Reply via email to