Commit 63930386 authored by Michael Niedermayer's avatar Michael Niedermayer

Merge commit 'e2274aa5'

* commit 'e2274aa5':
  mjpegdec: use the AVFrame API properly.

Conflicts:
	libavcodec/mjpegdec.c
Merged-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parents 2fade10c e2274aa5
...@@ -87,9 +87,12 @@ av_cold int ff_mjpeg_decode_init(AVCodecContext *avctx) ...@@ -87,9 +87,12 @@ av_cold int ff_mjpeg_decode_init(AVCodecContext *avctx)
{ {
MJpegDecodeContext *s = avctx->priv_data; MJpegDecodeContext *s = avctx->priv_data;
if (!s->picture_ptr) if (!s->picture_ptr) {
s->picture_ptr = &s->picture; s->picture = av_frame_alloc();
avcodec_get_frame_defaults(&s->picture); if (!s->picture)
return AVERROR(ENOMEM);
s->picture_ptr = s->picture;
}
s->avctx = avctx; s->avctx = avctx;
ff_hpeldsp_init(&s->hdsp, avctx->flags); ff_hpeldsp_init(&s->hdsp, avctx->flags);
...@@ -1370,7 +1373,7 @@ next_field: ...@@ -1370,7 +1373,7 @@ next_field:
s->last_dc[i] = (4 << s->bits); s->last_dc[i] = (4 << s->bits);
if (s->lossless) { if (s->lossless) {
av_assert0(s->picture_ptr == &s->picture); av_assert0(s->picture_ptr == s->picture);
if (CONFIG_JPEGLS_DECODER && s->ls) { if (CONFIG_JPEGLS_DECODER && s->ls) {
// for () { // for () {
// reset_ls_coding_parameters(s, 0); // reset_ls_coding_parameters(s, 0);
...@@ -1391,7 +1394,7 @@ next_field: ...@@ -1391,7 +1394,7 @@ next_field:
} }
} else { } else {
if (s->progressive && predictor) { if (s->progressive && predictor) {
av_assert0(s->picture_ptr == &s->picture); av_assert0(s->picture_ptr == s->picture);
if ((ret = mjpeg_decode_scan_progressive_ac(s, predictor, if ((ret = mjpeg_decode_scan_progressive_ac(s, predictor,
ilv, prev_shift, ilv, prev_shift,
point_transform)) < 0) point_transform)) < 0)
...@@ -2037,7 +2040,10 @@ av_cold int ff_mjpeg_decode_end(AVCodecContext *avctx) ...@@ -2037,7 +2040,10 @@ av_cold int ff_mjpeg_decode_end(AVCodecContext *avctx)
av_log(avctx, AV_LOG_INFO, "Single field\n"); av_log(avctx, AV_LOG_INFO, "Single field\n");
} }
if (s->picture_ptr) if (s->picture) {
av_frame_free(&s->picture);
s->picture_ptr = NULL;
} else if (s->picture_ptr)
av_frame_unref(s->picture_ptr); av_frame_unref(s->picture_ptr);
av_free(s->buffer); av_free(s->buffer);
......
...@@ -90,7 +90,7 @@ typedef struct MJpegDecodeContext { ...@@ -90,7 +90,7 @@ typedef struct MJpegDecodeContext {
int h_max, v_max; /* maximum h and v counts */ int h_max, v_max; /* maximum h and v counts */
int quant_index[4]; /* quant table index for each component */ int quant_index[4]; /* quant table index for each component */
int last_dc[MAX_COMPONENTS]; /* last DEQUANTIZED dc (XXX: am I right to do that ?) */ int last_dc[MAX_COMPONENTS]; /* last DEQUANTIZED dc (XXX: am I right to do that ?) */
AVFrame picture; /* picture structure */ AVFrame *picture; /* picture structure */
AVFrame *picture_ptr; /* pointer to picture structure */ AVFrame *picture_ptr; /* pointer to picture structure */
int got_picture; ///< we found a SOF and picture is valid, too. int got_picture; ///< we found a SOF and picture is valid, too.
int linesize[MAX_COMPONENTS]; ///< linesize << interlaced int linesize[MAX_COMPONENTS]; ///< linesize << interlaced
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment