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

Claes Redestad claes.redestad at oracle.com
Wed Oct 16 08:46:30 UTC 2019


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...

/Claes


More information about the hotspot-runtime-dev mailing list