<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 5, 2013 at 5:36 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 1373020562 -19800<br>
# Node ID c4c1e8043fbc8edba128feef7dedaad63314956d<br>
# Parent  85fe1251277bf343194d76111ecccab1744e832b<br>
xDCT32_C code cleanup<br>
<br>
diff -r 85fe1251277b -r c4c1e8043fbc source/common/dct.cpp<br>
--- a/source/common/dct.cpp     Fri Jul 05 16:00:33 2013 +0530<br>
+++ b/source/common/dct.cpp     Fri Jul 05 16:06:02 2013 +0530<br>
@@ -535,28 +535,28 @@<br>
 #undef N<br>
 }<br>
<br>
-void xDCT32_C(short *pSrc, int *pDst, intptr_t nStride)<br>
+void xDCT32_C(short *src, int *dst, intptr_t nStride)</blockquote><div><br></div><div>nStride here is also hungarian.  The n implies a count.  They've mixed the hungarian styles in a few places.  You don't typically use nStride when other code uses iStride.  In any case, we want to simply use 'stride' here.<br>
</div><div style><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">{<br>
     const int shift_1st = 4;<br>
     const int shift_2nd = 11;<br>
<br>
-    ALIGN_VAR_32(Short, tmp[32 * 32]);<br>
-    ALIGN_VAR_32(Short, tmp1[32 * 32]);<br>
+    ALIGN_VAR_32(Short, coef[32 * 32]);<br>
+    ALIGN_VAR_32(Short, block[32 * 32]);<br>
<br>
     for (int i = 0; i < 32; i++)<br>
     {<br>
-        memcpy(&tmp1[i * 32], &pSrc[i * nStride], 32 * sizeof(short));<br>
+        memcpy(&block[i * 32], &src[i * nStride], 32 * sizeof(short));<br>
     }<br>
<br>
-    partialButterfly32(tmp1, tmp, shift_1st, 32);<br>
-    partialButterfly32(tmp, tmp1, shift_2nd, 32);<br>
+    partialButterfly32(block, coef, shift_1st, 32);<br>
+    partialButterfly32(coef, block, shift_2nd, 32);<br>
<br>
 #define N (32)<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>