shithub: openh264

Download patch

ref: f8cea450ff42ffa0f1956ca8abe0f83d50842da6
parent: f74c920129b6cfe6bbaca23baf3216f4e6b930d0
author: Linjie Fu <[email protected]>
date: Tue Apr 28 20:31:11 EDT 2020 

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/[email protected]/

Suggested-by: Martin Storsjö <[email protected]>
Signed-off-by: Linjie Fu <[email protected]>


--- a/codec/encoder/core/src/encoder_ext.cpp
+++ b/codec/encoder/core/src/encoder_ext.cpp
@@ -374,12 +374,12 @@
                  pCfg->bEnableFrameSkip);
     if ((pCfg->iMaxQp <= 0) || (pCfg->iMinQp <= 0)) {
       if (pCfg->iUsageType == SCREEN_CONTENT_REAL_TIME) {
-        WelsLog (pLogCtx, WELS_LOG_WARNING, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp, MIN_SCREEN_QP,
+        WelsLog (pLogCtx, WELS_LOG_INFO, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp, MIN_SCREEN_QP,
                  MAX_SCREEN_QP);
         pCfg->iMinQp = MIN_SCREEN_QP;
         pCfg->iMaxQp = MAX_SCREEN_QP;
       } else {
-        WelsLog (pLogCtx, WELS_LOG_WARNING, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp,
+        WelsLog (pLogCtx, WELS_LOG_INFO, "Change QP Range from(%d,%d) to (%d,%d)", pCfg->iMinQp, pCfg->iMaxQp,
                  GOM_MIN_QP_MODE, MAX_LOW_BR_QP);
         pCfg->iMinQp = GOM_MIN_QP_MODE;
         pCfg->iMaxQp = MAX_LOW_BR_QP;