On Thu, 10 Feb 2022 22:49:38 GMT, Mandy Chung <mch...@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