[8u-dev] Request for Review and Approval: 8066612: Add a test that will call getDeclaredFields() on all classes and try to set them accessible.

Daniel Fuchs daniel.fuchs at oracle.com
Wed Dec 17 18:21:25 UTC 2014


Hi,

This is a request for approval to backport:

   8066612: Add a test that will call getDeclaredFields() on all
            classes and try to set them accessible.
   https://bugs.openjdk.java.net/browse/JDK-8066612

to 8u-dev.

The test code is slightly different on 8u compared to 9 so I'm asking
for review here as well:

On 8u-dev the test is different because it can't use the jrt:/
filesystem that 9 was using. Instead it parses the jars found on
the bootclasspath to get the list of classes.

The changes are however very limited:

   - The inner class ClassNameJrtStreamBuilder is replaced with
     ClassNameStreamBuilder
   - The class loader passed to Class.forName is now 'null'

Here is the webrev for 8:
http://cr.openjdk.java.net/~dfuchs/webrev_8066612/webrev-jdk8.01/

To be compared with the changeset pushed in 9:
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/70e68970bdee

And in case that's needed - the review thread for 9 started here:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-December/030078.html

best regards

-- daniel



More information about the jdk8u-dev mailing list