[x265] [PATCH] stats: rename fields of FrameStats structure

Deepthi Nandakumar deepthi at multicorewareinc.com
Mon Jun 8 14:32:19 CEST 2015


While we're in the renaming business, we might as well rename the percent
variables also, to indicate clearly these are in 8x8 CU counts.

On Fri, Jun 5, 2015 at 6:22 PM, Divya Manivannan <divya at multicorewareinc.com
> wrote:

> # HG changeset patch
> # User Divya Manivannan <divya at multicorewareinc.com>
> # Date 1433508684 -19800
> #      Fri Jun 05 18:21:24 2015 +0530
> # Node ID d790bd34659a36a2028e801a4bff6031fe4a712a
> # Parent  2c806c77fd172f3d4bd5480e4b94ecf4b8601c3d
> stats: rename fields of FrameStats structure
>
> diff -r 2c806c77fd17 -r d790bd34659a source/encoder/frameencoder.cpp
> --- a/source/encoder/frameencoder.cpp   Fri Jun 05 18:20:07 2015 +0530
> +++ b/source/encoder/frameencoder.cpp   Fri Jun 05 18:21:24 2015 +0530
> @@ -562,9 +562,9 @@
>              m_frameStats.mvBits    += m_rows[i].rowStats.mvBits;
>              m_frameStats.coeffBits += m_rows[i].rowStats.coeffBits;
>              m_frameStats.miscBits  += m_rows[i].rowStats.miscBits;
> -            totalI                 += m_rows[i].rowStats.iCuCnt;
> -            totalP                 += m_rows[i].rowStats.pCuCnt;
> -            totalSkip              += m_rows[i].rowStats.skipCuCnt;
> +            totalI                 += m_rows[i].rowStats.intra8x8Cnt;
> +            totalP                 += m_rows[i].rowStats.inter8x8Cnt;
> +            totalSkip              += m_rows[i].rowStats.skip8x8Cnt;
>          }
>          int totalCuCount = totalI + totalP + totalSkip;
>          m_frameStats.percentIntra = (double)totalI / totalCuCount;
> @@ -920,9 +920,9 @@
>              {
>                  /* 1 << shift == number of 8x8 blocks at current depth */
>                  int shift = 2 * (g_maxCUDepth - depth);
> -                curRow.rowStats.iCuCnt += qTreeIntraCnt[depth] << shift;
> -                curRow.rowStats.pCuCnt += qTreeInterCnt[depth] << shift;
> -                curRow.rowStats.skipCuCnt += qTreeSkipCnt[depth] << shift;
> +                curRow.rowStats.intra8x8Cnt += qTreeIntraCnt[depth] <<
> shift;
> +                curRow.rowStats.inter8x8Cnt += qTreeInterCnt[depth] <<
> shift;
> +                curRow.rowStats.skip8x8Cnt  += qTreeSkipCnt[depth] <<
> shift;
>
>                  // clear the row cu data from thread local object
>                  qTreeIntraCnt[depth] = qTreeInterCnt[depth] =
> qTreeSkipCnt[depth] = 0;
> diff -r 2c806c77fd17 -r d790bd34659a source/encoder/ratecontrol.h
> --- a/source/encoder/ratecontrol.h      Fri Jun 05 18:20:07 2015 +0530
> +++ b/source/encoder/ratecontrol.h      Fri Jun 05 18:21:24 2015 +0530
> @@ -53,9 +53,9 @@
>      int         coeffBits; /* Texture bits (DCT coefs) */
>      int         miscBits;
>
> -    int         iCuCnt;
> -    int         pCuCnt;
> -    int         skipCuCnt;
> +    int         intra8x8Cnt;
> +    int         inter8x8Cnt;
> +    int         skip8x8Cnt;
>
>      /* CU type counts stored as percentage */
>      double      percentIntra;
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20150608/b5a23527/attachment.html>


More information about the x265-devel mailing list