<div dir="ltr">Modes chosen by the lookahead are something that could potentially be used in analysis (maybe as a search start point?). This as an idea merits investigation.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 2, 2016 at 9:49 PM, Ximing Cheng <span dir="ltr"><<a href="mailto:chengximing1989@foxmail.com" target="_blank">chengximing1989@foxmail.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 Ximing Cheng <<a href="mailto:ximingcheng@tencent.com">ximingcheng@tencent.com</a>><br>
# Date 1464884336 -28800<br>
#      Fri Jun 03 00:18:56 2016 +0800<br>
# Node ID a8b9ed784477d7feb6f4d900d963b037ca2e435e<br>
# Parent  6098ba3e0cf16b110cff3b2519ce2d997ecac396<br>
lookahead: remove unused intra mode<br>
<br>
diff -r 6098ba3e0cf1 -r a8b9ed784477 source/common/lowres.cpp<br>
--- a/source/common/lowres.cpp  Tue May 31 14:06:55 2016 +0100<br>
+++ b/source/common/lowres.cpp  Fri Jun 03 00:18:56 2016 +0800<br>
@@ -69,7 +69,6 @@<br>
     lowresPlane[3] = buffer[3] + padoffset;<br>
<br>
     CHECKED_MALLOC(intraCost, int32_t, cuCount);<br>
-    CHECKED_MALLOC(intraMode, uint8_t, cuCount);<br>
<br>
     for (int i = 0; i < bframes + 2; i++)<br>
     {<br>
@@ -98,7 +97,6 @@<br>
 {<br>
     X265_FREE(buffer[0]);<br>
     X265_FREE(intraCost);<br>
-    X265_FREE(intraMode);<br>
<br>
     for (int i = 0; i < bframes + 2; i++)<br>
     {<br>
diff -r 6098ba3e0cf1 -r a8b9ed784477 source/common/lowres.h<br>
--- a/source/common/lowres.h    Tue May 31 14:06:55 2016 +0100<br>
+++ b/source/common/lowres.h    Fri Jun 03 00:18:56 2016 +0800<br>
@@ -124,7 +124,6 @@<br>
     int32_t*  rowSatds[X265_BFRAME_MAX + 2][X265_BFRAME_MAX + 2];<br>
     int       intraMbs[X265_BFRAME_MAX + 2];<br>
     int32_t*  intraCost;<br>
-    uint8_t*  intraMode;<br>
     int64_t   satdCost;<br>
     uint16_t* lowresCostForRc;<br>
     uint16_t(*lowresCosts[X265_BFRAME_MAX + 2][X265_BFRAME_MAX + 2]);<br>
diff -r 6098ba3e0cf1 -r a8b9ed784477 source/encoder/slicetype.cpp<br>
--- a/source/encoder/slicetype.cpp      Tue May 31 14:06:55 2016 +0100<br>
+++ b/source/encoder/slicetype.cpp      Fri Jun 03 00:18:56 2016 +0800<br>
@@ -313,7 +313,6 @@<br>
<br>
             fenc.lowresCosts[0][0][cuXY] = (uint16_t)(X265_MIN(icost, LOWRES_COST_MASK) | (0 << LOWRES_COST_SHIFT));<br>
             fenc.intraCost[cuXY] = icost;<br>
-            fenc.intraMode[cuXY] = (uint8_t)ilowmode;<br>
<br>
             /* do not include edge blocks in the frame cost estimates, they are not very accurate */<br>
             const bool bFrameScoreCU = (cuX > 0 && cuX < widthInCU - 1 &&<br>
<br>
<br>
_______________________________________________<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/listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div>Deepthi Nandakumar<br></div>Engineering Manager, x265<br></div>Multicoreware, Inc<br></div></div>
</div>