[x265] [PATCH] weightp: fix hash mismatch

[Steve Borho]

On Wed, Feb 12, 2014 at 7:40 AM, <kavitha at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Kavitha Sampath <kavitha at multicorewareinc.com>
> # Date 1392212379 -19800
> #      Wed Feb 12 19:09:39 2014 +0530
> # Node ID e2691be42bc7476825f3ba1df2adc31a4b2a8ee1
> # Parent  19d7752a4f9ca0863ea5efea80a040bae022be2b
>

Thanks Kavitha; this was definitely a bug I introduced.


> weightp: fix hash mismatch
>

In the future, a blank line should separate the summary line and the longer
description


> all references should have same luma and chroma denominator
>
> diff -r 19d7752a4f9c -r e2691be42bc7 source/encoder/weightPrediction.cpp
> --- a/source/encoder/weightPrediction.cpp       Wed Feb 12 00:35:27 2014
> -0600
> +++ b/source/encoder/weightPrediction.cpp       Wed Feb 12 19:09:39 2014
> +0530
> @@ -500,9 +500,9 @@
>              {
>                  for (int ref = 0; ref < slice.getNumRefIdx(list); ref++)
>                  {
> -                    SET_WEIGHT(wp[list][ref][0], false, 64, 6, 0);
> -                    SET_WEIGHT(wp[list][ref][1], false, 64, 6, 0);
> -                    SET_WEIGHT(wp[list][ref][2], false, 64, 6, 0);
> +                    SET_WEIGHT(wp[list][ref][0], false, 1<<denom, denom,
> 0);
> +                    SET_WEIGHT(wp[list][ref][1], false, 1<<denom, denom,
> 0);
> +                    SET_WEIGHT(wp[list][ref][2], false, 1<<denom, denom,
> 0);
>                  }
>              }
>              if (weightp::tryCommonDenom(slice, param, wp, temp, denom))
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>



-- 
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20140212/b4a962ad/attachment-0001.html>