On Tue, 22 Mar 2022 10:20:15 GMT, Daniel Fuchs <[email protected]> wrote:
>> Jim Laskey 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 18 additional
>> commits since the last revision:
>>
>> - Remove LOOKUP static final
>> - Merge branch 'master' into 8282798
>> - Typos
>> - Update Carrier.java
>> - Redo API to use list, bring Carrier.component back
>> - Clean up API
>> - Remove redundant MethodHandle component(MethodType methodType, int i) API
>> - Revert to {@return} style comments.
>> - Clarify primitive store in array carriers.
>> - Use long array for primitives
>> - ... and 8 more:
>> https://git.openjdk.java.net/jdk/compare/b726367e...a8657bbe
>
> src/java.base/share/classes/java/lang/runtime/Carrier.java line 166:
>
>> 164: from = longIndex++;
>> 165: filter = DOUBLE_TO_LONG;
>> 166: } else if(ptype == float.class) {
>
> Trivial: space missing after `if`
Updated
-------------
PR: https://git.openjdk.java.net/jdk/pull/7744
