RFR: 8263992: Remove dead code NativeLookup::base_library_lookup

David Holmes david.holmes at oracle.com
Tue Mar 23 04:46:24 UTC 2021


On 23/03/2021 2:12 pm, Ioi Lam wrote:
> On Tue, 23 Mar 2021 03:21:52 GMT, David Holmes <dholmes at openjdk.org> wrote:
> 
>> Hi Ioi,
>>
>> Looks good but do we have a problem with the notion of "in base library" ??? Should we perhaps have been using this method elsewhere?
>>
>> Thanks,
>> David
> 
> This function was last used here in 2014. (JDK-8031819: Remove legacy jdk checks and code)
> 
> So one has found any use of it for the past 6+ years. I think it's pretty dead.
> 
> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/diff/afe58d604f28/src/share/vm/prims/jvm.cpp

Yes it looks like this should have been removed as part of 8031819.

But I remain a little confused as to what "in base library" means in the 
rest of the code.

Cheers,
David

> -------------
> 
> PR: https://git.openjdk.java.net/jdk/pull/3139
> 


More information about the hotspot-dev mailing list