<div dir="ltr">The patches have been pushed. Please pull from the latest stable tip and recompile. Hopefully your concerns should be addressed now.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Pradeep Ramachandran, PhD<div>Solution Architect at <a href="http://www.multicorewareinc.com/" style="font-size:12.8px" target="_blank">www.multicorewareinc.com/</a></div><div>Adjunct Faculty at<a href="http://www.cse.iitm.ac.in" target="_blank"> www.cse.iitm.ac.in</a>/</div><div><a href="http://pradeeprama.info/" style="font-size:12.8px" target="_blank">pradeeprama.info/</a><br></div><div><span style="font-size:12.8px">Ph:   +91 99627 82018</span><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Wed, Apr 26, 2017 at 11:16 PM, Mario *LigH* Rohkrämer <span dir="ltr"><<a href="mailto:contact@ligh.de" target="_blank">contact@ligh.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi.<br>
<br>
Ma0 already submitted a patch on April 20th, which handles this issue by casting them to unsigned. I guess it was not yet committed.<br>
<br>
By the way, Multicoreware prefers patches in plain text in the message body for easier review.<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
Am 26.04.2017, 16:54 Uhr, schrieb Andrey Semashev <<a href="mailto:andrey.semashev@gmail.com" target="_blank">andrey.semashev@gmail.com</a>>:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
While compiling 2.4 I'm seeind lots of warnings like this:<br>
<br>
.../source/common/ipfilter.cpp<wbr>: In instantiation of ‘void<br>
{anonymous}::interp_horiz_ps_c<wbr>(const pixel*, intptr_t, int16_t*,<br>
intptr_t, int, int) [with int N = 8; int width = 4; int height = 4;<br>
pixel = unsigned char; intptr_t = long int; int16_t = short int]’:<br>
.../source/common/ipfilter.cpp<wbr>:417:5:   required from here<br>
.../source/common/ipfilter.cpp<wbr>:126:36: warning: left shift of negative<br>
value [-Wshift-negative-value]<br>
      int offset = -IF_INTERNAL_OFFS << shift;<br>
                   ~~~~~~~~~~~~~~~~~~^~~~~~~~<br>
<br>
Left-shifting negative signed intehers is undefined behavior in C++.<br>
I've attached a patch that resolves the warnings. The patch assumes 2's<br>
complement signed integers and that the shift does not introduce an<br>
arithmetic overflow.<br>
</blockquote>
<br>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
<br>
Fun and success!<br>
Mario *LigH* Rohkrämer<br>
mailto:<a href="mailto:contact@ligh.de" target="_blank">contact@ligh.de</a><br>
<br>
______________________________<wbr>_________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/l<wbr>istinfo/x265-devel</a><br>
</font></span></blockquote></div><br></div>