<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 5, 2013 at 4:40 AM,  <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<br>
# Date 1383647996 -19800<br>
# Node ID 158f6c0760cb595f6c5a4f0b44aa48f78e76c2df<br>
# Parent  b5bdc10daa02d7d769263c5adb63fe3026ea4ba0<br>
corrected chroma partion for blockcopy_pp function<br></blockquote><div><br></div><div>I liked it better the old way</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

diff -r b5bdc10daa02 -r 158f6c0760cb source/test/pixelharness.cpp<br>
--- a/source/test/pixelharness.cpp      Tue Nov 05 15:28:56 2013 +0530<br>
+++ b/source/test/pixelharness.cpp      Tue Nov 05 16:09:56 2013 +0530<br>
@@ -670,15 +670,6 @@<br>
         }<br>
     }<br>
<br>
-    if (opt.chroma_copy_pp[part])<br>
-    {<br>
-        if (!check_block_copy_pp(ref.chroma_copy_pp[part], opt.chroma_copy_pp[part]))<br>
-        {<br>
-            printf("chroma_copy_pp[%s] failed\n", chromaPartStr[part]);<br>
-            return false;<br>
-        }<br>
-    }<br>
-<br>
     if (opt.luma_copy_sp[part])<br>
     {<br>
         if (!check_block_copy_sp(ref.luma_copy_sp[part], opt.luma_copy_sp[part]))<br>
@@ -854,7 +845,19 @@<br>
             return false;<br>
         }<br>
     }<br>
-    return true;<br>
+<br>
+    for (int i = 0; i < NUM_CHROMA_PARTITIONS; i++)<br>
+    {<br>
+      if (opt.chroma_copy_pp[i])<br>
+      {<br>
+          if (!check_block_copy_pp(ref.chroma_copy_pp[i], opt.chroma_copy_pp[i]))<br>
+          {<br>
+              printf("chroma_copy_pp[%s] failed\n", chromaPartStr[i]);<br>
+              return false;<br>
+          }<br>
+      }<br>
+    }<br>
+     return true;<br>
 }<br>
<br>
 void PixelHarness::measurePartition(int part, const EncoderPrimitives& ref, const EncoderPrimitives& opt)<br>
@@ -922,12 +925,6 @@<br>
         REPORT_SPEEDUP(opt.luma_copy_pp[part], ref.luma_copy_pp[part], pbuf1, 64, pbuf2, 128);<br>
     }<br>
<br>
-    if (opt.chroma_copy_pp[part])<br>
-    {<br>
-        printf("ccpy_pp[%s]", chromaPartStr[part]);<br>
-        REPORT_SPEEDUP(opt.chroma_copy_pp[part], ref.chroma_copy_pp[part], pbuf1, 64, pbuf2, 128);<br>
-    }<br>
-<br>
     if (opt.luma_copy_sp[part])<br>
     {<br>
         printf("lcpy_sp[%s]", lumaPartStr[part]);<br>
@@ -1055,4 +1052,13 @@<br>
         printf("downscale");<br>
         REPORT_SPEEDUP(opt.frame_init_lowres_core, ref.frame_init_lowres_core, pbuf2, pbuf1, pbuf2, pbuf3, pbuf4, 64, 64, 64, 64);<br>
     }<br>
+<br>
+    for (int i = 0; i < NUM_CHROMA_PARTITIONS; i++)<br>
+    {<br>
+        if (opt.chroma_copy_pp[i])<br>
+        {<br>
+            printf("ccpy_pp[%s]", chromaPartStr[i]);<br>
+            REPORT_SPEEDUP(opt.chroma_copy_pp[i], ref.chroma_copy_pp[i], pbuf1, 64, pbuf2, 128);<br>
+        }<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" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>