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