<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 12, 2014 at 7:40 AM,  <span dir="ltr"><<a href="mailto:kavitha@multicorewareinc.com" target="_blank">kavitha@multicorewareinc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Kavitha Sampath <<a href="mailto:kavitha@multicorewareinc.com">kavitha@multicorewareinc.com</a>><br>
# Date 1392212379 -19800<br>
#      Wed Feb 12 19:09:39 2014 +0530<br>
# Node ID e2691be42bc7476825f3ba1df2adc31a4b2a8ee1<br>
# Parent  19d7752a4f9ca0863ea5efea80a040bae022be2b<br></blockquote><div><br></div><div>Thanks Kavitha; this was definitely a bug I introduced.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

weightp: fix hash mismatch<br></blockquote><div><br></div><div>In the future, a blank line should separate the summary line and the longer description</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

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