<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 15, 2014 at 11:31 PM, Steve Borho <span dir="ltr"><<a href="mailto:steve@borho.org" target="_blank">steve@borho.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Apr 15, 2014 at 11:34 AM,  <<a href="mailto:aarthi@multicorewareinc.com">aarthi@multicorewareinc.com</a>> wrote:<br>

> # HG changeset patch<br>
> # User Aarthi Thirumalai<br>
> # Date 1397579661 -19800<br>
> #      Tue Apr 15 22:04:21 2014 +0530<br>
> # Node ID bf48002755a3f5593732ca039ad38a3c799da808<br>
> # Parent  0a95a6bb0f8e71d7a7d0f8e3803ec2878ad558fe<br>
> vbv: clear row diagonal Satd costs, cu Satd costs when vbv row reset is triggered.<br>
><br>
> diff -r 0a95a6bb0f8e -r bf48002755a3 source/encoder/frameencoder.cpp<br>
> --- a/source/encoder/frameencoder.cpp   Tue Apr 15 12:34:36 2014 +0900<br>
> +++ b/source/encoder/frameencoder.cpp   Tue Apr 15 22:04:21 2014 +0530<br>
> @@ -1192,6 +1192,8 @@<br>
>                          m_pic->m_qpaRc[r] = 0;<br>
>                          m_pic->m_rowEncodedBits[r] = 0;<br>
>                          m_pic->m_numEncodedCusPerRow[r] = 0;<br>
> +                        m_pic->m_rowDiagSatd[r] = 0;<br>
> +                        m_pic->m_rowDiagIntraSatd[r] = 0;<br>
>                      }<br>
><br>
>                      m_bAllRowsStop = false;<br>
> @@ -1248,6 +1250,13 @@<br>
>      x265_emms();<br>
>      double qp = baseQp;<br>
><br>
> +    /*clear cuCostsForVbv when vbv row reset is triggered. */<br>
> +    if (m_pic->m_cuCostsForVbv[cuAddr] > 0 || m_pic->m_intraCuCostsForVbv[cuAddr] > 0)<br>
> +    {<br>
> +        m_pic->m_cuCostsForVbv[cuAddr] = 0;<br>
> +        m_pic->m_intraCuCostsForVbv[cuAddr] = 0;<br>
> +    }<br>
> +<br>
<br>
</div></div>Can we just unconditionally clear these two values?<br></blockquote><div> yea, we can remove the if condition and clear them always.. they will be normally be  0 at this point unless the cus are again encoded after vbv row resets in which case, we need to clear them necessarily.   </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Steve Borho<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</font></span></blockquote></div><br></div></div>