From 8fbf0300d8248400e1c21784c3abe931f553383b Mon Sep 17 00:00:00 2001 From: wm4 Date: Mon, 17 Mar 2014 18:19:03 +0100 Subject: vd_lavc: reinit hwdec on profile changes Needed in theory. I don't know if there are even any real-world files which change the profile mid-stream. --- video/decode/lavc.h | 1 + video/decode/vd_lavc.c | 7 +++++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/video/decode/lavc.h b/video/decode/lavc.h index 35c66fc19b..16d3251725 100644 --- a/video/decode/lavc.h +++ b/video/decode/lavc.h @@ -40,6 +40,7 @@ typedef struct lavc_ctx { int hwdec_fmt; int hwdec_w; int hwdec_h; + int hwdec_profile; } vd_ffmpeg_ctx; struct vd_lavc_hwdec { diff --git a/video/decode/vd_lavc.c b/video/decode/vd_lavc.c index 632d5cacd7..969c78f54c 100644 --- a/video/decode/vd_lavc.c +++ b/video/decode/vd_lavc.c @@ -463,14 +463,17 @@ static enum AVPixelFormat get_format_hwdec(struct AVCodecContext *avctx, if (ctx->hwdec->image_format) { for (int i = 0; fmt[i] != AV_PIX_FMT_NONE; i++) { if (ctx->hwdec->image_format == pixfmt2imgfmt(fmt[i])) { - // There could be more reasons for a change, e.g. profile change. + // There could be more reasons for a change, and it's possible + // that we miss some. (Might also depend on the hwaccel type.) bool change = ctx->hwdec_w != avctx->width || ctx->hwdec_h != avctx->height || - ctx->hwdec_fmt != ctx->hwdec->image_format; + ctx->hwdec_fmt != ctx->hwdec->image_format || + ctx->hwdec_profile != avctx->profile; ctx->hwdec_w = avctx->width; ctx->hwdec_h = avctx->height; ctx->hwdec_fmt = ctx->hwdec->image_format; + ctx->hwdec_profile = avctx->profile; if (ctx->hwdec->init_decoder && change) { if (ctx->hwdec->init_decoder(ctx, ctx->hwdec_fmt, ctx->hwdec_w, ctx->hwdec_h) < 0) -- cgit v1.2.3