<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 12, 2013 at 6:02 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 1373626970 -19800<br>
# Node ID 0ae84c4ad796445eaaaf125298c679ea1ac7284f<br>
# Parent  4dd127bfafdc3ec3a5a01db18ca51c8cc7210234<br>
TComRdCost::xGetSSE32 cleanup<br></blockquote><div><br></div><div>All pushed, thanks.  In the future, if you have a dozen commits that are all simple/cleanup types you can push them to a bitbucket fork and make me a reader and send a pull request.  These types of cleanups don't need to go through the list.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff -r 4dd127bfafdc -r 0ae84c4ad796 source/Lib/TLibCommon/TComRdCost.cpp<br>
--- a/source/Lib/TLibCommon/TComRdCost.cpp      Fri Jul 12 16:23:18 2013 +0530<br>
+++ b/source/Lib/TLibCommon/TComRdCost.cpp      Fri Jul 12 16:32:50 2013 +0530<br>
@@ -1157,45 +1157,45 @@<br>
     return sum;<br>
 }<br>
<br>
-UInt TComRdCost::xGetSSE32(DistParam* pcDtParam)<br>
+UInt TComRdCost::xGetSSE32(DistParam* distParam)<br>
 {<br>
-    if (pcDtParam->applyWeight)<br>
+    if (distParam->applyWeight)<br>
     {<br>
-        assert(pcDtParam->cols == 32);<br>
-        return xGetSSEw(pcDtParam);<br>
+        assert(distParam->cols == 32);<br>
+        return xGetSSEw(distParam);<br>
     }<br>
-    Pel* piOrg   = pcDtParam->fenc;<br>
-    Pel* piCur   = pcDtParam->fref;<br>
-    Int  iRows   = pcDtParam->rows;<br>
-    Int  iStrideOrg = pcDtParam->fencstride;<br>
-    Int  iStrideCur = pcDtParam->frefstride;<br>
+    Pel* org = distParam->fenc;<br>
+    Pel* cur = distParam->fref;<br>
+    Int  rows = distParam->rows;<br>
+    Int  strideOrg = distParam->fencstride;<br>
+    Int  strideCur = distParam->frefstride;<br>
<br>
-    UInt uiSum = 0;<br>
-    UInt uiShift = DISTORTION_PRECISION_ADJUSTMENT((pcDtParam->bitDepth - 8) << 1);<br>
+    UInt sum = 0;<br>
+    UInt shift = DISTORTION_PRECISION_ADJUSTMENT((distParam->bitDepth - 8) << 1);<br>
<br>
-    if ((sizeof(Pel) == 2) || ((piOrg != NULL) && (piCur != NULL)))<br>
+    if ((sizeof(Pel) == 2) || ((org != NULL) && (cur != NULL)))<br>
     {<br>
-        uiSum = xGetSSE32Help<Pel, Pel>(piOrg, iStrideOrg, piCur, iStrideCur, iRows, uiShift);<br>
+        sum = xGetSSE32Help<Pel, Pel>(org, strideOrg, cur, strideCur, rows, shift);<br>
     }<br>
     else<br>
     {<br>
-        Short* piOrg_s;<br>
-        Short* piCur_s;<br>
-        DistParamSSE* DtParam = reinterpret_cast<DistParamSSE*>(pcDtParam);<br>
-        piOrg_s = DtParam->ptr1;<br>
-        piCur_s = DtParam->ptr2;<br>
+        Short* org_s;<br>
+        Short* cur_s;<br>
+        DistParamSSE* dtParam = reinterpret_cast<DistParamSSE*>(distParam);<br>
+        org_s = dtParam->ptr1;<br>
+        cur_s = dtParam->ptr2;<br>
<br>
-        if ((piOrg == NULL) && (piCur == NULL))<br>
+        if ((org == NULL) && (cur == NULL))<br>
         {<br>
-            uiSum = xGetSSE32Help<Short, Short>(piOrg_s, iStrideOrg, piCur_s, iStrideCur, iRows, uiShift);<br>
+            sum = xGetSSE32Help<Short, Short>(org_s, strideOrg, cur_s, strideCur, rows, shift);<br>
         }<br>
         else<br>
         {<br>
-            uiSum = xGetSSE32Help<Short, Pel>(piOrg_s, iStrideOrg, piCur, iStrideCur, iRows, uiShift);<br>
+            sum = xGetSSE32Help<Short, Pel>(org_s, strideOrg, cur, strideCur, rows, shift);<br>
         }<br>
     }<br>
<br>
-    return uiSum;<br>
+    return sum;<br>
 }<br>
<br>
 template<typename T1, typename T2><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>