On 01/18/2013 01:35 AM, Anton Khirnov wrote: > > On Wed, 9 Jan 2013 12:11:53 -0500, Justin Ruggles <[email protected]> > 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.
It doesn't produce a warning for me. -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
