<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 2, 2013 at 6:52 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 praveentiwari<br>
# Date 1372765951 -19800<br>
# Node ID ef51d05368cf00120fc19f629d803f95af562492<br>
# Parent  879245f5eedca52db300147368c523b8e12c33bc<br>
xDCT16 vector code cleanup<br>
<br>
diff -r 879245f5eedc -r ef51d05368cf source/common/vec/dct.inc<br>
--- a/source/common/vec/dct.inc Tue Jul 02 15:23:41 2013 +0530<br>
+++ b/source/common/vec/dct.inc Tue Jul 02 17:22:31 2013 +0530<br>
@@ -785,27 +785,27 @@<br>
     }<br>
 }<br>
<br>
-void xDCT16(short *pSrc, int *pDst, intptr_t nStride)<br>
+void xDCT16(short *src, int *dst, intptr_t nStride)<br></blockquote><div><br></div><div style>more nStride again; I'll quit commenting on these, just be on the lookout for this Hungarian style</div><div><br></div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 {<br>
     const int shift_1st = 3;<br>
     const int shift_2nd = 10;<br>
<br>
-    ALIGN_VAR_32(Short, tmp[16 * 16]);<br>
-    ALIGN_VAR_32(Short, tmp1[16 * 16]);<br>
+    ALIGN_VAR_32(Short, coef[16 * 16]);<br>
+    ALIGN_VAR_32(Short, block[16 * 16]);<br>
<br>
     for (int i = 0; i < 16; i++)<br>
     {<br>
-        memcpy(&tmp1[i * 16], &pSrc[i * nStride], 16 * sizeof(short));<br>
+        memcpy(&block[i * 16], &src[i * nStride], 16 * sizeof(short));<br>
     }<br>
<br>
-    partialButterfly16(tmp1, tmp, shift_1st, 16);<br>
-    partialButterfly16(tmp, tmp1, shift_2nd, 16);<br>
+    partialButterfly16(block, coef, shift_1st, 16);<br>
+    partialButterfly16(coef, block, shift_2nd, 16);<br>
 #define N (16)<br>
     for (int i = 0; i < N; i++)<br>
     {<br>
         for (int j = 0; j < N; j++)<br>
         {<br>
-            pDst[i * N + j] = tmp1[i * N + j];<br>
+            dst[i * N + j] = block[i * N + j];<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="http://mailman.videolan.org/listinfo/x265-devel" target="_blank">http://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>