Changes to be pushed into jdk8/build

[Kelly O'Hair]

On Sep 13, 2012, at 3:04 AM, Magnus Ihse Bursie wrote:

> On 2012-09-13 10:12, Magnus Ihse Bursie wrote:
>> On 2012-09-13 02:09, David Holmes wrote:
>>> jdk/src/share/demo/jvmti/hprof/hprof_init.c
>>> 
>>> Not sure what this change has to do with anything :) Seems like the code involved here is pretty confused about data type sizes.
>> Yeah. I wonder who changed this and why.
> 
> It was Kelly, with the intent of supressing a warning (or error?) on Mac. My guess is that the compiler might have found it a bit superfluous to check if a unsigned short is ever > 65535. So the change seems sound, since otherwise that check will never happen (and too large numbers will just overflow). I'm not sure if it should be done as part of this push, thouh.
> 
> /Magnus

Yes, I should have pushed this into jdk8/build, I'll take care of it.

You are right in that I saw the warning, and having written this code with NO warnings on any platforms,
I was compelled to investigate my past failure and fix it. ;^)  I consider compiler warnings flaws in the code.

-kto