RFR: 8267121: Illegal access to private "size" field of ArrayList from build.gradle

Ambarish Rapte arapte at openjdk.java.net
Mon May 17 09:54:00 UTC 2021


On Sat, 15 May 2021 12:42:56 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> A test of our CI build with gradle 7.0.1 and JDK 16.0.1 revealed a latent bug in `build.gradle` in the `fetchExternalTools()` method. It checks the size of an `ArrayList` in two places, but rather than calling the `size()` method it omits the `()` which means it is accessing the private `size` field.
> 
> Starting in JDK 16, which has default strong encapsulation, this no longer works, so we need to fix this before switching. We really should fix it anyway, since accessing the internal field is both unnecessary and wrong.

lgtm, Did not find any other instance where `size` member variable is used.

-------------

Marked as reviewed by arapte (Reviewer).

PR: https://git.openjdk.java.net/jfx/pull/503


More information about the openjfx-dev mailing list