RFR(M) JDK-8031819: Remove legacy jdk checks and code

Lois Foltan lois.foltan at oracle.com
Wed Jun 4 15:51:46 UTC 2014


Great clean up!  Looks good.  Only a tiny minor comment. In 
reflectionUtils.cpp, I would also remove the comment "The following 
class fields do not exist in previous version of jdk".  I find that more 
confusing than helpful.  I don't need to see a new webrev though.

Lois


On 6/4/2014 11:34 AM, harold seigel wrote:
> Hi,
>
> Please review this JDK 9 fix for bug JDK-8031819.  The fix removes 
> legacy code for old JDK versions.  Although the webrev contains lots 
> of files, the changes are not complicated.
>
> Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8031819/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8031819
>
> The fix was tested with the Hotspot JTREG tests, JCK Lang, VM, and 
> API/java_lang tests, nsk Quick tests, and JPRT.
>
> Thanks! Harold



More information about the hotspot-runtime-dev mailing list