Hi Duncan,<div><br></div><div>Yes.  I like "1+~Val" better.  It is lighter and cleaner.  </div><div><br></div><div>The updated patch is attached with both your and Aaron's comments.</div><div><br></div><div>Thanks,</div>

<div>-Peng</div><div><br><div class="gmail_quote">On Wed, Feb 27, 2013 at 9:57 AM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Peng,<div class="im"><br>
<br>
On 27/02/13 15:28, Peng Cheng wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Suppressing warning was the my first attempt to fix this warning, and just as<br>
Aaron said, the warning suppressing was added into multiple places, looking ugly.<br>
<br>
The attached patch is the second attempt,  People, who wrote "-Val" with "Val"<br>
as unsigned , normally understand this change with the original code as comments<br>
as Aaron suggests.<br>
</blockquote>
<br></div>
can't you just replace -Val with 1+~Val ?<br>
<br>
Ciao, Duncan.<br>
</blockquote></div><br></div>