On Tue, 8 Feb 2022 14:16:15 GMT, Vladimir Ivanov <[email protected]> wrote:
>> Aleksey Shipilev 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 three additional
>> commits since the last revision:
>>
>> - Comment
>> - Merge branch 'master' into JDK-8281168-int-varform
>> - Fix
>
> src/java.base/share/classes/java/lang/invoke/VarForm.java line 112:
>
>> 110: @ForceInline
>> 111: final MemberName getMemberName(int mode) {
>> 112: MemberName mn = memberName_table[mode];
>
> It makes sense to leave a comment here why inlining `getMemberNameOrNull()`
> makes sense. Otherwise, looks good.
Added comment.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7333