[x265] [PATCH] TComRdCost::setLambda cleanup

Steve Borho steve at borho.org
Fri Jul 12 08:15:36 CEST 2013


On Fri, Jul 12, 2013 at 12:44 AM, <praveen at multicorewareinc.com> wrote:

> # HG changeset patch
> # User praveentiwari
> # Date 1373607846 -19800
> # Node ID e4eafcb35d23cd00f049b3e6ed72e9b313427213
> # Parent  b518fa481bd4eef520dfa7d5f898ef32c753c88f
> TComRdCost::setLambda cleanup
>

queued.


> diff -r b518fa481bd4 -r e4eafcb35d23 source/Lib/TLibCommon/TComRdCost.cpp
> --- a/source/Lib/TLibCommon/TComRdCost.cpp      Fri Jul 12 11:09:33 2013
> +0530
> +++ b/source/Lib/TLibCommon/TComRdCost.cpp      Fri Jul 12 11:14:06 2013
> +0530
> @@ -51,9 +51,9 @@
>  TComRdCost::~TComRdCost()
>  {}
>
> -Void TComRdCost::setLambda(Double dLambda)
> +Void TComRdCost::setLambda(Double lambda)
>  {
> -    m_lambda2           = dLambda;
> +    m_lambda2           = lambda;
>      m_lambda        = sqrt(m_lambda2);
>

while you're poking around this function, you should clean up the
white-space by lining up the = as near to the vars as is feasible.


>      m_lambdaMotionSAD = (UInt64)floor(65536.0 * m_lambda);
>      m_lambdaMotionSSE = (UInt64)floor(65536.0 * m_lambda2);
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> http://mailman.videolan.org/listinfo/x265-devel
>



-- 
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20130712/899797b0/attachment-0001.html>


More information about the x265-devel mailing list