RFR: 8261407: ReflectionFactory.checkInitted() is not thread-safe [v5]
liach
duke at openjdk.java.net
Fri Feb 11 02:12:15 UTC 2022
On Thu, 10 Feb 2022 22:49:38 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> Can I just write the config class as a record, or does it generate too much boilerplate? Or is this class initialized too early to use records (such as indy is not yet ready)?
>
> Worth a try. Even the regular class, the constructor taking 5 fields isn't too bad to me. In a near future, I hope to remove the old core reflection implementation, `noInflation` and `inflationThreshold` will be removed and fewer fields.
I made a commit with the config class converted into a record. Apparently the tests are passing, and I would assume it would be feasible. Should I apply it?
https://github.com/liachmodded/jdk/commit/8cf5af417a6f906e9fc0c878d60731d6f026b528
-------------
PR: https://git.openjdk.java.net/jdk/pull/6889
More information about the core-libs-dev
mailing list