RFR: 8232211: Remove dead code from os.hpp|cpp

David Holmes david.holmes at oracle.com
Wed Oct 16 09:59:51 UTC 2019


On 16/10/2019 6:46 pm, Claes Redestad wrote:
> On 2019-10-16 04:37, David Holmes wrote:
>>> os_version is
>>> never read, which means initialize_os_info can go - too.
>>
>> Hmmmm. This is the kind of information that could be used then not 
>> used then used again on each release, depending on whether we 
>> introduce or remove code that is conditional on the OS version. I 
>> guess we can always add it back ... though I'm willing to bet that if 
>> we need this again the person making the change will reinvent the code 
>> rather than resurrecting it. :(
> 
> I don't have a problem with this. At the very least I'd reinvent it into
> a struct so that major, minor and fix version was addressable and not
> something you need to extract from a 24-bit integer...

The format is intended for simple comparisons as you can see in AIX/PPC 
usage.

David

> /Claes


More information about the hotspot-runtime-dev mailing list