From d8ee02a071e8044687a34bed646718deb9d13d37 Mon Sep 17 00:00:00 2001 From: Matthias Hunstock Date: Sun, 10 Jul 2016 13:02:37 +0200 Subject: [PATCH] decklink: enhancement: format selection with AVRational v2 When finding a video mode based on frame size and frame rate, use AVRational methods instead of simple comparison. Add debug information. Signed-off-by: Matthias Hunstock Signed-off-by: Michael Niedermayer --- libavdevice/decklink_common.cpp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libavdevice/decklink_common.cpp b/libavdevice/decklink_common.cpp index 53004b372f..a3bc58dd6a 100644 --- a/libavdevice/decklink_common.cpp +++ b/libavdevice/decklink_common.cpp @@ -143,6 +143,9 @@ int ff_decklink_set_format(AVFormatContext *avctx, int i = 1; HRESULT res; + av_log(avctx, AV_LOG_DEBUG, "Trying to find mode for frame size %dx%d, frame timing %d/%d, direction %d, mode number %d\n", + width, height, tb_num, tb_den, direction, num); + if (ctx->duplex_mode) { DECKLINK_BOOL duplex_supported = false; @@ -178,11 +181,7 @@ int ff_decklink_set_format(AVFormatContext *avctx, return AVERROR(EIO); } - - if (tb_num == 1) { - tb_num *= 1000; - tb_den *= 1000; - } + AVRational target_tb = av_make_q(tb_num, tb_den); ctx->bmd_mode = bmdModeUnknown; while ((ctx->bmd_mode == bmdModeUnknown) && itermode->Next(&mode) == S_OK) { BMDTimeValue bmd_tb_num, bmd_tb_den; @@ -190,9 +189,10 @@ int ff_decklink_set_format(AVFormatContext *avctx, int bmd_height = mode->GetHeight(); mode->GetFrameRate(&bmd_tb_num, &bmd_tb_den); + AVRational mode_tb = av_make_q(bmd_tb_num, bmd_tb_den); if ((bmd_width == width && bmd_height == height && - bmd_tb_num == tb_num && bmd_tb_den == tb_den) || i == num) { + !av_cmp_q(mode_tb, target_tb)) || i == num) { ctx->bmd_mode = mode->GetDisplayMode(); ctx->bmd_width = bmd_width; ctx->bmd_height = bmd_height; @@ -200,7 +200,7 @@ int ff_decklink_set_format(AVFormatContext *avctx, ctx->bmd_tb_num = bmd_tb_num; ctx->bmd_field_dominance = mode->GetFieldDominance(); av_log(avctx, AV_LOG_INFO, "Found Decklink mode %d x %d with rate %.2f%s\n", - bmd_width, bmd_height, (float)bmd_tb_den/(float)bmd_tb_num, + bmd_width, bmd_height, 1/av_q2d(mode_tb), (ctx->bmd_field_dominance==bmdLowerFieldFirst || ctx->bmd_field_dominance==bmdUpperFieldFirst)?"(i)":""); }