History

encoder_ext: modify the verbosity level for QP changing

Change the verbosity level of qp changing message from warning to info. Setting QP<=0 means "use the defaults", and that's an intended usecase, so change to info. Discussed in: https://patchwork.ffmpeg.org/project/ffmpeg/patch/1588065363-7104-2-git-send-email-linjie.fu@intel.com/

Linjie Fu committed 6 years ago

Commit f8cea451 parent f74c9201 file · +2 −2Browse files

Showing the changes to codec/encoder/core/src/encoder_ext.cpp (1 of 1 file). Show all

Unified · Split

codec/encoder/core/src/encoder_ext.cpp+2−2
@@ -374,12 +374,12 @@ int32_t ParamValidation (SLogContext* pLogCtx, SWelsSvcCodingParam* pCfg) { · 373 unmodified lines
374374 pCfg->bEnableFrameSkip);
375375 if ((pCfg->iMaxQp <= 0) || (pCfg->iMinQp <= 0)) {
376376 if (pCfg->iUsageType == SCREEN_CONTENT_REAL_TIME) {
377− WelsLog (pLogCtx, WELS_LOG_WARNING, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp, MIN_SCREEN_QP,
377+ WelsLog (pLogCtx, WELS_LOG_INFO, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp, MIN_SCREEN_QP,
378378 MAX_SCREEN_QP);
379379 pCfg->iMinQp = MIN_SCREEN_QP;
380380 pCfg->iMaxQp = MAX_SCREEN_QP;
381381 } else {
382− WelsLog (pLogCtx, WELS_LOG_WARNING, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp,
382+ WelsLog (pLogCtx, WELS_LOG_INFO, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp,
383383 GOM_MIN_QP_MODE, MAX_LOW_BR_QP);
384384 pCfg->iMinQp = GOM_MIN_QP_MODE;
385385 pCfg->iMaxQp = MAX_LOW_BR_QP;

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t