RFR(L): JDK-8057777 Cleanup of old and unused VM interfaces
Frederic Parain
frederic.parain at oracle.com
Fri Oct 3 16:02:54 UTC 2014
Alan,
Thank you for your feedback, see my comments below.
On 03/10/2014 17:19, Alan Bateman wrote:
> On 30/09/2014 07:40, Frederic Parain wrote:
>> Hi all,
>>
>> Please review changes for bug JDK-8057777 "Cleanup of old
>> and unused VM interfaces"
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8057777
>>
>> This is basically a big cleanup of VM interfaces that are
>> not used anymore by the JDK but have been kept in our code
>> base for historical reasons (HotSpot Express for instance).
>> These changesets remove these interfaces from both the
>> JDK and the HotSpot side, and also perform some cleanup
>> on code that directly referenced the removed interfaces.
>>
>> These changes do not modify the behavior of the Java
>> classes impacted by the cleanup.
>>
>> VM interfaces removal has been approved by CCC and
>> a Release Note has been prepared that explicitly list
>> all the removed interfaces.
>>
>> Testing: JPRT hotspot + core, vm.quick.testlist, jdk_core
>>
>> Webrevs:
>> http://cr.openjdk.java.net/~fparain/8057777/
> cc'ing core-libs-dev as part of this is clean-up in the library code too.
>
> I think we should deprecate java.lang.Compiler and the Runtime.traceXXX
> methods. They've been non-functional for a long time and having them in
> the API is a bit mis-leading to anyone reading the javadoc. I realize
> you are focused on the removing the old JVM_* functions so we can
> follow-up on that via other issues of course.
Deprecating class and methods would require another CCC approval, so
I'd prefer to post-pone this change to another changeset.
> Can ClassLoader#resolveClass0 can be removed completely? The null check
> can be done in ClassLoader#resolveClass.
Done.
> In the mapfile for libjava then the comment at line 281 says "ZipFile.c
> needs this one". As getLastErrorString is now exported for use by libzip
> then the comment should probably be updated.
Done.
> Otherwise this clean-up looks good to me and the jdk_core group of tests
> is the right group to exercise this area.
Thank you, the new jdk webrev is here:
http://cr.openjdk.java.net/~fparain/8057777/jdk_v02/
Regards,
Fred
--
Frederic Parain - Oracle
Grenoble Engineering Center - France
Phone: +33 4 76 18 81 17
Email: Frederic.Parain at oracle.com
More information about the core-libs-dev
mailing list