On Fri, 9 May 2025 09:21:14 GMT, Viktor Klang <vkl...@openjdk.org> wrote:
>> Per Minborg 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 24 additional >> commits since the last revision: >> >> - Revert unintended change >> - Revert unintended change >> - Remove unused factories and add comment >> - Merge branch 'master' into stable-updaters >> - Add composition of functions and MHs >> - Revert changes in stable classes >> - Add lazy CallSite methods >> - Reformat >> - Add convenience methods and documentations >> - Add low level variants >> - ... and 14 more: https://git.openjdk.org/jdk/compare/b9531363...9a58db6a > > src/java.base/share/classes/jdk/internal/lang/stable/StableFieldUpdater.java > line 697: > >> 695: return f; >> 696: } >> 697: } > > Is there a specific reason not to use `Class::getDeclaredField(name)`? Class::getDeclaredField(name) throws NoSuchFieldException if not found. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25040#discussion_r2081336037
