[x265] [PATCH] analysis: update analysisReuseLevel 7 for analysis sharing

[Ashok Kumar Mishra]

On Mon, Nov 13, 2017 at 1:55 PM, <praveen at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Praveen Tiwari <praveen at multicorewareinc.com>
> # Date 1510561483 -19800
> #      Mon Nov 13 13:54:43 2017 +0530
> # Node ID 67f2dd6203ff8a1e2271ef0ac052ac5f1ad99110
> # Parent  5ea4fdbdea99a8bdd91d0d7961bcf50764d445b6
> analysis: update analysisReuseLevel 7 for analysis sharing
>
> diff -r 5ea4fdbdea99 -r 67f2dd6203ff source/encoder/analysis.cpp
> --- a/source/encoder/analysis.cpp       Mon Nov 13 12:20:50 2017 +0530
> +++ b/source/encoder/analysis.cpp       Mon Nov 13 13:54:43 2017 +0530
> @@ -2301,7 +2301,7 @@
>              for (uint32_t part = 0; part < numPU; part++)
>              {
>                  PredictionUnit pu(mode.cu, cuGeom, part);
> -                if (m_param->analysisReuseLevel == 10)
> +                if (m_param->analysisReuseLevel >= 7)
>                  {
>                      analysis_inter_data* interDataCTU =
> (analysis_inter_data*)m_frame->m_analysisData.interData;
>                      int cuIdx = (mode.cu.m_cuAddr *
> parentCTU.m_numPartitions) + cuGeom.absPartIdx;
> @@ -2407,7 +2407,7 @@
>                  if (m_slice->m_pps->bUseDQP && nextDepth <=
> m_slice->m_pps->maxCuDQPDepth)
>                      nextQP = setLambdaFromQP(parentCTU,
> calculateQpforCuSize(parentCTU, childGeom));
>
> -                int lamdaQP = m_param->analysisReuseLevel == 10 ? nextQP
> : lqp;
> +                int lamdaQP = m_param->analysisReuseLevel >= 7 ? nextQP :
> lqp;
>

Code will be more readable if you keep the expression of arithmetic if
condition inside a parentheses().

>
>                  if (split)
>                      m_param->rdLevel > 4 ? compressInterCU_rd5_6(parentCTU,
> childGeom, nextQP) : compressInterCU_rd0_4(parentCTU, childGeom, nextQP);
> diff -r 5ea4fdbdea99 -r 67f2dd6203ff source/encoder/encoder.cpp
> --- a/source/encoder/encoder.cpp        Mon Nov 13 12:20:50 2017 +0530
> +++ b/source/encoder/encoder.cpp        Mon Nov 13 13:54:43 2017 +0530
> @@ -3035,7 +3035,7 @@
>              CHECKED_MALLOC(interData->mergeFlag, uint8_t,
> analysis->numPartitions * analysis->numCUsInFrame);
>          }
>
> -        if (m_param->analysisReuseLevel == 10)
> +        if (m_param->analysisReuseLevel >= 7)
>          {
>              CHECKED_MALLOC(interData->interDir, uint8_t,
> analysis->numPartitions * analysis->numCUsInFrame);
>              for (int dir = 0; dir < numDir; dir++)
> @@ -3112,7 +3112,7 @@
>                  X265_FREE(((analysis_inter_data*)analysis->interData)->
> mergeFlag);
>                  X265_FREE(((analysis_inter_data*)analysis->interData)->
> partSize);
>              }
> -            if (m_param->analysisReuseLevel == 10)
> +            if (m_param->analysisReuseLevel >= 7)
>              {
>                  X265_FREE(((analysis_inter_data*)analysis->interData)->
> interDir);
>                  int numDir = analysis->sliceType == X265_TYPE_P ? 1 : 2;
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20171113/357a7549/attachment.html>