[x265] [PATCH] lookahead: remove unused intra mode

[Deepthi Nandakumar]

Modes chosen by the lookahead are something that could potentially be used
in analysis (maybe as a search start point?). This as an idea merits
investigation.

On Thu, Jun 2, 2016 at 9:49 PM, Ximing Cheng <chengximing1989 at foxmail.com>
wrote:

> # HG changeset patch
> # User Ximing Cheng <ximingcheng at tencent.com>
> # Date 1464884336 -28800
> #      Fri Jun 03 00:18:56 2016 +0800
> # Node ID a8b9ed784477d7feb6f4d900d963b037ca2e435e
> # Parent  6098ba3e0cf16b110cff3b2519ce2d997ecac396
> lookahead: remove unused intra mode
>
> diff -r 6098ba3e0cf1 -r a8b9ed784477 source/common/lowres.cpp
> --- a/source/common/lowres.cpp  Tue May 31 14:06:55 2016 +0100
> +++ b/source/common/lowres.cpp  Fri Jun 03 00:18:56 2016 +0800
> @@ -69,7 +69,6 @@
>      lowresPlane[3] = buffer[3] + padoffset;
>
>      CHECKED_MALLOC(intraCost, int32_t, cuCount);
> -    CHECKED_MALLOC(intraMode, uint8_t, cuCount);
>
>      for (int i = 0; i < bframes + 2; i++)
>      {
> @@ -98,7 +97,6 @@
>  {
>      X265_FREE(buffer[0]);
>      X265_FREE(intraCost);
> -    X265_FREE(intraMode);
>
>      for (int i = 0; i < bframes + 2; i++)
>      {
> diff -r 6098ba3e0cf1 -r a8b9ed784477 source/common/lowres.h
> --- a/source/common/lowres.h    Tue May 31 14:06:55 2016 +0100
> +++ b/source/common/lowres.h    Fri Jun 03 00:18:56 2016 +0800
> @@ -124,7 +124,6 @@
>      int32_t*  rowSatds[X265_BFRAME_MAX + 2][X265_BFRAME_MAX + 2];
>      int       intraMbs[X265_BFRAME_MAX + 2];
>      int32_t*  intraCost;
> -    uint8_t*  intraMode;
>      int64_t   satdCost;
>      uint16_t* lowresCostForRc;
>      uint16_t(*lowresCosts[X265_BFRAME_MAX + 2][X265_BFRAME_MAX + 2]);
> diff -r 6098ba3e0cf1 -r a8b9ed784477 source/encoder/slicetype.cpp
> --- a/source/encoder/slicetype.cpp      Tue May 31 14:06:55 2016 +0100
> +++ b/source/encoder/slicetype.cpp      Fri Jun 03 00:18:56 2016 +0800
> @@ -313,7 +313,6 @@
>
>              fenc.lowresCosts[0][0][cuXY] = (uint16_t)(X265_MIN(icost,
> LOWRES_COST_MASK) | (0 << LOWRES_COST_SHIFT));
>              fenc.intraCost[cuXY] = icost;
> -            fenc.intraMode[cuXY] = (uint8_t)ilowmode;
>
>              /* do not include edge blocks in the frame cost estimates,
> they are not very accurate */
>              const bool bFrameScoreCU = (cuX > 0 && cuX < widthInCU - 1 &&
>
>
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>



-- 
Deepthi Nandakumar
Engineering Manager, x265
Multicoreware, Inc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20160613/a7cb8ed7/attachment-0001.html>