Review Request: JDK-8220282 Add MethodHandle tests on accessing final fields
Lance Andersen
lance.andersen at oracle.com
Tue Apr 2 19:35:09 UTC 2019
Hi Mandy
the updates look good.
> On Mar 26, 2019, at 10:58 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
>
> This is a new version of the patch:
> http://cr.openjdk.java.net/~mchung/jdk13/webrevs/8220282/webrev.01
>
> I made further clean up to this new test.
>
> It extends MethodHandlesTest.HasFields class to include the test
> cases for instance final fields. HasFields is used to test
> findGetter/findStaticGetter, findSetter/findStaticSetter, and
> unreflectSetter/unreflectGetter.
>
> A new HasFields::testCasesFor(int testMode) method is added to
> return the test cases where c[1] == Error.class indicates a
> negative test case.
>
> For getters, all HasFields except bogus_xx fields are positive
> test cases, i.e. no exception is expected.
>
> For findSetter/findStaticSetter, a final field has no write access
> and setting on a final field is a negative test case.
>
> For unreflectSetter, non-final fields and instance final fields
> whose accessible flag is true have write access and hence they
> are positive test cases whereas static final fields are negative
> test cases.
>
> thanks
> Mandy
>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list