On Mon, 21 Feb 2022 16:35:09 GMT, liach <d...@openjdk.java.net> wrote:
>> liach has updated the pull request with a new target base due to a merge or >> a rebase. The incremental webrev excludes the unrelated changes brought in >> by the merge/rebase. The pull request contains 13 additional commits since >> the last revision: >> >> - Merge branch 'master' into 8261407-reflectionfactory >> - Whitespace issues >> - Refine docs, pull members out of the config record >> - The fast path should always come first. good lesson learned! >> restore config field comments >> - Try making the config a record and see if it works >> - Make config a pojo, move loading code into config class >> - use peter's model of separate data object >> - Merge branch 'master' into 8261407-reflectionfactory >> - Include the stable annotation >> - Merge branch 'master' into 8261407-reflectionfactory >> - ... and 3 more: >> https://git.openjdk.java.net/jdk/compare/73eb6e94...e97ea278 > > peter and mandy, would you mind review again today? Latest version looks good to me, @liach. Thanks for doing this. Regards, Peter ------------- PR: https://git.openjdk.java.net/jdk/pull/6889