<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 17, 2013 at 10:25 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 1382023511 -19800<br>
# Node ID b5375476fdedcce6e60c71c215ed95c923b9df43<br>
# Parent  dfae391107c3d2823adabe284f1c08278669d8f0<br>
created partition size and function pointer for luma filter<br>
<br>
diff -r dfae391107c3 -r b5375476fded source/common/primitives.h<br>
--- a/source/common/primitives.h        Thu Oct 17 16:05:37 2013 +0530<br>
+++ b/source/common/primitives.h        Thu Oct 17 20:55:11 2013 +0530<br>
@@ -148,6 +148,20 @@<br>
     NUM_CHROMA_PARTITIONS<br>
 };<br>
<br>
+//Each luma size of table will be used to genrate 3 partion sizes, except few exceptions.<br></blockquote><div><br></div><div>Comments need a space after //</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+//WxH will be used for WxH, Wx(H + 1) and (W + 1)x(H), like 4x8 will genrate 4x8, 4x9 and 5x9<br></blockquote><div><br></div><div>generate </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+//exceptions are 16x64, 48x64, 64x16, 64x48 because they don't have corresponding odd block sizes i.e (W + 1)xH and Wx(H + 1)<br></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+enum LumaPartions<br></blockquote><div><br></div><div>LumaPartitions</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+{<br>
+    LUMA_PARTITION_4x8,        LUMA_PARTITION_4x16 = 3,   LUMA_PARTITION_8x4 = 6,    LUMA_PARTITION_8x8 = 9,<br>
+    LUMA_PARTITION_8x16 = 12,  LUMA_PARTITION_8x32 = 15,  LUMA_PARTITION_12x16 = 18, LUMA_PARTITION_16x4 = 21,<br>
+    LUMA_PARTITION_16x8 = 24,  LUMA_PARTITION_16x12 = 27, LUMA_PARTITION_16x16 = 30, LUMA_PARTITION_16x32 = 33,<br>
+    LUMA_PARTITION_16x64 = 36, LUMA_PARTITION_24x32,      LUMA_PARTITION_32x8 = 40,  LUMA_PARTITION_32x16 = 43,<br>
+    LUMA_PARTITION_32x24 = 46, LUMA_PARTITION_32x32 = 49, LUMA_PARTITION_32x64 = 52, LUMA_PARTITION_48x64 = 55,<br>
+    LUMA_PARTITION_64x16,      LUMA_PARTITION_64x32,      LUMA_PARTITION_64x48 = 60, LUMA_PARTITION_64x64,<br>
+    NUM_LUMA_PARTITIONS = 64<br>
+};<br></blockquote><div><br></div><div>I don't see the point to making a new sparse enum that lines up with the existing Partitions enum.  If you're not going to pack these so NUM_LUMA_PARTITIONS is 24, why don't you just use Partitions directly?</div>
</div><div><br></div>-- <br>Steve Borho
</div></div>