[x265] [PATCH] SAO: remove resetSaoUnit()

[chen]

At 2015-11-20 19:58:21,ashok at multicorewareinc.com wrote:
># HG changeset patch
># User Ashok Kumar Mishra<ashok at multicorewareinc.com>
># Date 1445505834 -19800
>#      Thu Oct 22 14:53:54 2015 +0530
># Node ID 5d320d99e14fed8e64505d73f01505cdf0d94861
># Parent  c1575815b250fb02c1fdae803aead10a4997c835
>SAO: remove resetSaoUnit()
>
>diff -r c1575815b250 -r 5d320d99e14f source/encoder/sao.cpp
>--- a/source/encoder/sao.cpp	Thu Oct 29 17:15:15 2015 +0530
>+++ b/source/encoder/sao.cpp	Thu Oct 22 14:53:54 2015 +0530
>@@ -704,16 +704,6 @@
>     std::swap(m_tmpU1[plane], m_tmpU2[plane]);
> }
> 
>-void SAO::resetSaoUnit(SaoCtuParam* saoUnit)
>-{
>-    saoUnit->mergeMode  = SAO_MERGE_NONE;
>-    saoUnit->typeIdx    = -1;
>-    saoUnit->bandPos    = 0;
>-
>-    for (int i = 0; i < SAO_NUM_OFFSET; i++)
>-        saoUnit->offset[i] = 0;
>-}
>-
> void SAO::copySaoUnit(SaoCtuParam* saoUnitDst, const SaoCtuParam* saoUnitSrc)
> {
>     saoUnitDst->mergeMode   = saoUnitSrc->mergeMode;
>@@ -1172,9 +1162,7 @@
>                 }
>             }
> 
>-            saoParam->ctuParam[plane][addr].mergeMode = SAO_MERGE_NONE;
>-            saoParam->ctuParam[plane][addr].typeIdx   = -1;
>-            saoParam->ctuParam[plane][addr].bandPos   = 0;
>+            saoParam->ctuParam[plane][addr].reset();
>             if (saoParam->bSaoFlag[plane > 0])
>                 calcSaoStatsCu(addr, plane);
>         }
>@@ -1326,7 +1314,6 @@
>     int    currentDistortionTableBo[MAX_NUM_SAO_CLASS];
>     double currentRdCostTableBo[MAX_NUM_SAO_CLASS];
> 
>-    resetSaoUnit(lclCtuParam);
>     m_entropyCoder.load(m_rdContexts.temp);
>     m_entropyCoder.resetBits();
>     m_entropyCoder.codeSaoOffset(*lclCtuParam, 0);
>@@ -1386,7 +1373,6 @@
>     m_entropyCoder.store(m_rdContexts.temp);
> 
>     // merge left or merge up
>-
>     for (int mergeIdx = 0; mergeIdx < 2; mergeIdx++)
>     {
>         SaoCtuParam* mergeSrcParam = NULL;
>@@ -1413,8 +1399,6 @@
> 
>             mergeDist[mergeIdx + 1] = ((double)estDist / m_lumaLambda);
>         }
>-        else
>-            resetSaoUnit(&mergeSaoParam[mergeIdx]);
remove directly? mergeSaoParam declare in rdoSaoUnitRow() and never initialize before.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20151120/c001aafb/attachment.html>