[x265] [PATCH] vbv:reinit cu data when row vbv re-rencode is triggered

Steve Borho steve at borho.org
Fri Apr 18 21:37:28 CEST 2014


On Fri, Apr 18, 2014 at 7:31 AM,  <santhoshini at multicorewareinc.com> wrote:
> # HG changeset patch
> # User Santhoshini Sekar <santhoshini at multicorewareinc.com>
> # Date 1397824258 -19800
> #      Fri Apr 18 18:00:58 2014 +0530
> # Node ID 3d680a36ea9acc45f3db128a5efe557a79a3026b
> # Parent  03aa222ff8eb5ffbe65bd5bf522cad6561210024
> vbv:reinit cu data when row vbv re-rencode is triggered.

space after colon
no need for end punctuation on summary line

>
> diff -r 03aa222ff8eb -r 3d680a36ea9a source/encoder/frameencoder.cpp
> --- a/source/encoder/frameencoder.cpp   Fri Apr 18 17:32:14 2014 +0530
> +++ b/source/encoder/frameencoder.cpp   Fri Apr 18 18:00:58 2014 +0530
> @@ -1100,6 +1100,8 @@
>          {
>              if (!row)
>                  m_pic->m_rowDiagQp[row] = m_pic->m_avgQpRc;
> +            if (cu->m_baseQp != 0)
> +                cu->initEstData(0);

this doesn't look to be the appropriate place for this sort of reset.
If it needs to happen after a row restart shouldn't it be done in the
restart loop below?  Here it looks like it could be calling
initEstData() many times, redundantly.

>
>              if (row >= col && row && m_vbvResetTriggerRow != row)
>                  cu->m_baseQp = m_pic->getCU(cuAddr - numCols + 1)->m_baseQp;
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel



-- 
Steve Borho


More information about the x265-devel mailing list