[x265] [PATCH] Support for HLG-graded content and pic_struct

Vittorio Giovara vittorio.giovara at gmail.com
Thu Apr 12 16:49:34 CEST 2018


On Thu, Apr 12, 2018 at 1:44 PM, <ashok at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Ashok Kumar Mishra <ashok at multicorewareinc.com>
> # Date 1523532439 -19800
> #      Thu Apr 12 16:57:19 2018 +0530
> # Node ID ace55bce0e3c6e0beb52712fd71e01025bd73a22
> # Parent  04a337abd70de269cef7d9655365f3a3ebde02aa
> Support for HLG-graded content and pic_struct
>
> diff -r 04a337abd70d -r ace55bce0e3c source/common/param.cpp
> --- a/source/common/param.cpp   Thu Apr 12 15:10:59 2018 +0530
> +++ b/source/common/param.cpp   Thu Apr 12 16:57:19 2018 +0530
> @@ -134,7 +134,8 @@
>      param->bEmitInfoSEI = 1;
>      param->bEmitHDRSEI = 0;
>      param->bEmitIDRRecoverySEI = 0;
> -    /* CU definitions */
> +
> +       /* CU definitions */
>      param->maxCUSize = 64;
>      param->minCUSize = 8;
>      param->tuQTMaxInterDepth = 1;
> @@ -192,7 +193,8 @@
>      param->bEnableSAO = 1;
>      param->bSaoNonDeblocked = 0;
>      param->bLimitSAO = 0;
> -    /* Coding Quality */
> +
> +       /* Coding Quality */
>      param->cbQpOffset = 0;
>      param->crQpOffset = 0;
>      param->rdPenalty = 0;
> @@ -302,6 +304,10 @@
>      param->bLowPassDct = 0;
>      param->bMVType = 0;
>      param->bSingleSeiNal = 0;
> +
> +       /* SEI messages */
> +       param->preferredTransferCharacteristics = -1;
> +       param->pictureStructure = -1;
>  }
>
>  int x265_param_default_preset(x265_param* param, const char* preset,
> const char* tune)
> @@ -1030,6 +1036,8 @@
>          OPT("max-ausize-factor") p->maxAUSizeFactor = atof(value);
>          OPT("dynamic-refine") p->bDynamicRefine = atobool(value);
>          OPT("single-sei") p->bSingleSeiNal = atobool(value);
> +               OPT("atc-sei") p->preferredTransferCharacteristics =
> atoi(value);
> +               OPT("pic-struct") p->pictureStructure = atoi(value);
>          else
>              return X265_PARAM_BAD_NAME;
>      }
> diff -r 04a337abd70d -r ace55bce0e3c source/encoder/encoder.cpp
> --- a/source/encoder/encoder.cpp        Thu Apr 12 15:10:59 2018 +0530
> +++ b/source/encoder/encoder.cpp        Thu Apr 12 16:57:19 2018 +0530
> @@ -338,10 +338,12 @@
>
>      if (m_param->bEmitHRDSEI)
>          m_rateControl->initHRD(m_sps);
> +
>      if (!m_rateControl->init(m_sps))
>          m_aborted = true;
>      if (!m_lookahead->create())
>          m_aborted = true;
> +
>      initRefIdx();
>      if (m_param->analysisSave && m_param->bUseAnalysisFile)
>      {
>

For the future, could you please keep cosmetic changes (such as the ones
above) in a separate patch?
It gets confusing really fast and makes reading history needlessly fast.
-- 
Vittorio
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20180412/ce0869a6/attachment.html>


More information about the x265-devel mailing list