<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 5, 2013 at 5:42 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 1373020937 -19800<br>
# Node ID 5ff0843668f49bfd232ef1aa08524ad8278b70b1<br>
# Parent  cd21b61ba794b138aca6d0fc58e46323c45c018d<br>
xIDCT4_C code cleanup<br>
<br>
diff -r cd21b61ba794 -r 5ff0843668f4 source/common/dct.cpp<br>
--- a/source/common/dct.cpp     Fri Jul 05 16:09:17 2013 +0530<br>
+++ b/source/common/dct.cpp     Fri Jul 05 16:12:17 2013 +0530<br>
@@ -591,31 +591,31 @@<br>
     }<br>
 }<br>
<br>
-void xIDCT4_C(int *pSrc, short *pDst, intptr_t stride)<br>
+void xIDCT4_C(int *src, short *dst, intptr_t stride)<br></blockquote><div><br></div><div style>we also need to rename all these methods to idct4_c, etc.</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 = 7;<br>
     const int shift_2nd = 12;<br>
<br>
-    ALIGN_VAR_32(Short, tmp[4 * 4]);<br>
-    ALIGN_VAR_32(Short, tmp2[4 * 4]);<br>
+    ALIGN_VAR_32(Short, coef[4 * 4]);<br>
+    ALIGN_VAR_32(Short, coef2[4 * 4]);<br>
<br>
 #define N (4)<br>
     for (int i = 0; i < N; i++)<br>
     {<br>
         for (int j = 0; j < N; j++)<br>
         {<br>
-            tmp2[i * N + j] = (short)pSrc[i * N + j];<br>
+            coef2[i * N + j] = (short)src[i * N + j];<br>
         }<br>
     }<br>
<br>
 #undef N<br>
<br>
-    partialButterflyInverse4(tmp2, tmp, shift_1st, 4); // Forward DST BY FAST ALGORITHM, block input, tmp output<br>
-    partialButterflyInverse4(tmp, tmp2, shift_2nd, 4); // Forward DST BY FAST ALGORITHM, tmp input, coeff output<br>
+    partialButterflyInverse4(coef2, coef, shift_1st, 4); // Forward DST BY FAST ALGORITHM, block input, coef output<br>
+    partialButterflyInverse4(coef, coef2, shift_2nd, 4); // Forward DST BY FAST ALGORITHM, coef input, coeff output<br>
<br>
     for (int i = 0; i < 4; i++)<br>
     {<br>
-        memcpy(&pDst[i * stride], &tmp2[i * 4], 4 * sizeof(short));<br>
+        memcpy(&dst[i * stride], &coef2[i * 4], 4 * sizeof(short));<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>