<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 16, 2013 at 10: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 1381938013 -19800<br>
# Node ID 5914800260d95a9bac7ce3eec2291e5c781e8422<br>
# Parent  86f1dcc53a1a101b7acd169d608b1d089efeb888<br>
check_IPFilterChroma_primitive, stride made equal to min width 2, fix for 2XN block<br></blockquote><div><br></div><div>Queued, but you'll need to explain to me why this is a necessary or an improvement.</div><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
diff -r 86f1dcc53a1a -r 5914800260d9 source/test/ipfilterharness.cpp<br>
--- a/source/test/ipfilterharness.cpp   Wed Oct 16 20:30:59 2013 +0530<br>
+++ b/source/test/ipfilterharness.cpp   Wed Oct 16 21:10:13 2013 +0530<br>
@@ -270,31 +270,26 @@<br>
<br>
 bool IPFilterHarness::check_IPFilterChroma_primitive(filter_pp_t ref, filter_pp_t opt)<br>
 {<br>
-    int rand_srcStride, rand_dstStride, rand_coeffIdx;<br>
+    int srcStride, dstStride, rand_coeffIdx;<br>
<br>
     for (int i = 0; i <= 100; i++)<br>
     {<br>
<br>
         rand_coeffIdx = rand() % 8;                // Random coeffIdex in the filter<br>
-        rand_srcStride = rand() % 100;              // Randomly generated srcStride<br>
-        rand_dstStride = rand() % 100;              // Randomly generated dstStride<br>
<br>
-        if (rand_srcStride < 32)<br>
-            rand_srcStride = 32;<br>
+         srcStride = 2;<br>
+         dstStride = 2;<br>
<br>
-        if (rand_dstStride < 32)<br>
-            rand_dstStride = 32;<br>
-<br>
-        opt(pixel_buff + 3 * rand_srcStride,<br>
-            rand_srcStride,<br>
+        opt(pixel_buff + 3 * srcStride,<br>
+            srcStride,<br>
             IPF_vec_output_p,<br>
-            rand_dstStride,<br>
+            dstStride,<br>
             rand_coeffIdx<br>
             );<br>
-        ref(pixel_buff + 3 * rand_srcStride,<br>
-            rand_srcStride,<br>
+        ref(pixel_buff + 3 * srcStride,<br>
+            srcStride,<br>
             IPF_C_output_p,<br>
-            rand_dstStride,<br>
+            dstStride,<br>
             rand_coeffIdx<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>