From 71ad83667d32f64c658034baa3cc56246ae67363 Mon Sep 17 00:00:00 2001 From: Zhao Zhili Date: Sun, 25 Apr 2021 16:06:04 +0800 Subject: [PATCH] lavc/videotoolboxenc: set DataRateLimits for hevc From the comment it's not available on old version. It works now by testing on macOS 11.2.1. There is no document about since when. So trying to set the configuration and ignore the error for hevc. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index e576ff7d7e..e4f890f7f7 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec/videotoolboxenc.c @@ -1113,8 +1113,8 @@ static int vtenc_create_encoder(AVCodecContext *avctx, return AVERROR_EXTERNAL; } - if (vtctx->codec_id == AV_CODEC_ID_H264 && max_rate > 0) { - // kVTCompressionPropertyKey_DataRateLimits is not available for HEVC + if ((vtctx->codec_id == AV_CODEC_ID_H264 || vtctx->codec_id == AV_CODEC_ID_HEVC) + && max_rate > 0) { bytes_per_second_value = max_rate >> 3; bytes_per_second = CFNumberCreate(kCFAllocatorDefault, kCFNumberSInt64Type, @@ -1152,7 +1152,11 @@ static int vtenc_create_encoder(AVCodecContext *avctx, if (status) { av_log(avctx, AV_LOG_ERROR, "Error setting max bitrate property: %d\n", status); - return AVERROR_EXTERNAL; + // kVTCompressionPropertyKey_DataRateLimits is available for HEVC + // now but not on old release. There is no document about since + // when. So ignore the error if it failed for hevc. + if (vtctx->codec_id != AV_CODEC_ID_HEVC) + return AVERROR_EXTERNAL; } }