[x265] [PATCH] corrected chroma partion for blockcopy_pp function

praveen at multicorewareinc.com praveen at multicorewareinc.com
Tue Nov 5 11:40:07 CET 2013


# HG changeset patch
# User Praveen Tiwari
# Date 1383647996 -19800
# Node ID 158f6c0760cb595f6c5a4f0b44aa48f78e76c2df
# Parent  b5bdc10daa02d7d769263c5adb63fe3026ea4ba0
corrected chroma partion for blockcopy_pp function

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


More information about the x265-devel mailing list