<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 13, 2017 at 1:55 PM,  <span dir="ltr"><<a href="mailto:praveen@multicorewareinc.com" target="_blank">praveen@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Praveen Tiwari <<a href="mailto:praveen@multicorewareinc.com">praveen@multicorewareinc.com</a>><br>
# Date 1510561483 -19800<br>
#      Mon Nov 13 13:54:43 2017 +0530<br>
# Node ID 67f2dd6203ff8a1e2271ef0ac052ac<wbr>5f1ad99110<br>
# Parent  5ea4fdbdea99a8bdd91d0d7961bcf5<wbr>0764d445b6<br>
analysis: update analysisReuseLevel 7 for analysis sharing<br>
<br>
diff -r 5ea4fdbdea99 -r 67f2dd6203ff source/encoder/analysis.cpp<br>
--- a/source/encoder/analysis.cpp       Mon Nov 13 12:20:50 2017 +0530<br>
+++ b/source/encoder/analysis.cpp       Mon Nov 13 13:54:43 2017 +0530<br>
@@ -2301,7 +2301,7 @@<br>
             for (uint32_t part = 0; part < numPU; part++)<br>
             {<br>
                 PredictionUnit pu(<a href="http://mode.cu" rel="noreferrer" target="_blank">mode.cu</a>, cuGeom, part);<br>
-                if (m_param->analysisReuseLevel == 10)<br>
+                if (m_param->analysisReuseLevel >= 7)<br>
                 {<br>
                     analysis_inter_data* interDataCTU = (analysis_inter_data*)m_frame-<wbr>>m_analysisData.interData;<br>
                     int cuIdx = (mode.cu.m_cuAddr * parentCTU.m_numPartitions) + cuGeom.absPartIdx;<br>
@@ -2407,7 +2407,7 @@<br>
                 if (m_slice->m_pps->bUseDQP && nextDepth <= m_slice->m_pps->maxCuDQPDepth)<br>
                     nextQP = setLambdaFromQP(parentCTU, calculateQpforCuSize(<wbr>parentCTU, childGeom));<br>
<br>
-                int lamdaQP = m_param->analysisReuseLevel == 10 ? nextQP : lqp;<br>
+                int lamdaQP = m_param->analysisReuseLevel >= 7 ? nextQP : lqp;<br></blockquote><div><br></div><div>Code will be more readable if you keep the expression of arithmetic if condition inside a parentheses(). </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
                 if (split)<br>
                     m_param->rdLevel > 4 ? compressInterCU_rd5_6(<wbr>parentCTU, childGeom, nextQP) : compressInterCU_rd0_4(<wbr>parentCTU, childGeom, nextQP);<br>
diff -r 5ea4fdbdea99 -r 67f2dd6203ff source/encoder/encoder.cpp<br>
--- a/source/encoder/encoder.cpp        Mon Nov 13 12:20:50 2017 +0530<br>
+++ b/source/encoder/encoder.cpp        Mon Nov 13 13:54:43 2017 +0530<br>
@@ -3035,7 +3035,7 @@<br>
             CHECKED_MALLOC(interData-><wbr>mergeFlag, uint8_t, analysis->numPartitions * analysis->numCUsInFrame);<br>
         }<br>
<br>
-        if (m_param->analysisReuseLevel == 10)<br>
+        if (m_param->analysisReuseLevel >= 7)<br>
         {<br>
             CHECKED_MALLOC(interData-><wbr>interDir, uint8_t, analysis->numPartitions * analysis->numCUsInFrame);<br>
             for (int dir = 0; dir < numDir; dir++)<br>
@@ -3112,7 +3112,7 @@<br>
                 X265_FREE(((analysis_inter_<wbr>data*)analysis->interData)-><wbr>mergeFlag);<br>
                 X265_FREE(((analysis_inter_<wbr>data*)analysis->interData)-><wbr>partSize);<br>
             }<br>
-            if (m_param->analysisReuseLevel == 10)<br>
+            if (m_param->analysisReuseLevel >= 7)<br>
             {<br>
                 X265_FREE(((analysis_inter_<wbr>data*)analysis->interData)-><wbr>interDir);<br>
                 int numDir = analysis->sliceType == X265_TYPE_P ? 1 : 2;<br>
______________________________<wbr>_________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/<wbr>listinfo/x265-devel</a><br>
</blockquote></div><br></div></div>