From 2ca4bdab2068f4c360a6a10d53048c53a02d551d Mon Sep 17 00:00:00 2001 From: liuxiang Date: Sun, 10 Aug 2025 11:26:58 +0800 Subject: [PATCH] =?UTF-8?q?=E4=BF=AE=E5=A4=8DAI-Review=E5=91=8A=E8=AD=A6?= =?UTF-8?q?=E9=97=AE=E9=A2=98?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: liuxiang --- .../recorder/server/recorder_server.cpp | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/services/services/recorder/server/recorder_server.cpp b/services/services/recorder/server/recorder_server.cpp index 7dacfa70a..ee0c3d144 100644 --- a/services/services/recorder/server/recorder_server.cpp +++ b/services/services/recorder/server/recorder_server.cpp @@ -69,7 +69,7 @@ std::shared_ptr RecorderServer::Create() { std::shared_ptr server = std::make_shared(); int32_t ret = server->Init(); - CHECK_AND_RETURN_RET_LOG(ret == MSERR_OK, nullptr, "failed to init RecorderServer"); + CHECK_AND_RETURN_RET_LOG(ret == MSERR_OK, nullptr, "failed to init RecorderServer, ret: %{public}d", ret); return server; } @@ -598,16 +598,17 @@ int32_t RecorderServer::SetMetaConfigs(int32_t sourceId) { MEDIA_LOGI("RecorderServer:0x%{public}06" PRIXPTR " SetMetaConfigs in, sourceId(%{public}d)", FAKE_POINTER(this), sourceId); - CHECK_AND_RETURN_RET_LOG(SetMetaMimeType(sourceId, Plugins::MimeType::TIMED_METADATA) == MSERR_OK, - MSERR_EXT_OPERATE_NOT_PERMIT, "set meta mime type failed"); + int32_t ret = SetMetaMimeType(sourceId, Plugins::MimeType::TIMED_METADATA); + CHECK_AND_RETURN_RET_LOG(ret == MSERR_OK, MSERR_EXT_OPERATE_NOT_PERMIT, + "set meta mime type failed, ret: %{public}d", ret); if (config_.metaSource == MetaSourceType::VIDEO_META_MAKER_INFO) { - CHECK_AND_RETURN_RET_LOG( - SetMetaTimedKey(sourceId, VID_DEBUG_INFO_KEY) == MSERR_OK, MSERR_EXT_OPERATE_NOT_PERMIT, - "set meta key failed"); + ret = SetMetaTimedKey(sourceId, VID_DEBUG_INFO_KEY); + CHECK_AND_RETURN_RET_LOG(ret == MSERR_OK, MSERR_EXT_OPERATE_NOT_PERMIT, + "set meta key failed, ret: %{public}d", ret); auto sourceTrackMime = GetVideoMime(config_.videoCodec); - CHECK_AND_RETURN_RET_LOG( - SetMetaSourceTrackMime(sourceId, sourceTrackMime) == MSERR_OK, - MSERR_EXT_OPERATE_NOT_PERMIT, "set meta source track mime failed"); + ret = SetMetaSourceTrackMime(sourceId, sourceTrackMime); + CHECK_AND_RETURN_RET_LOG(ret == MSERR_OK, MSERR_EXT_OPERATE_NOT_PERMIT, + "set meta source track mime failed, ret: %{public}d", ret); } return MSERR_OK; } -- Gitee