RFR: JDK-8282798 java.lang.runtime.Carrier [v11]

Jim Laskey jlaskey at openjdk.java.net
Tue Mar 22 13:00:22 UTC 2022


On Mon, 21 Mar 2022 18:02:07 GMT, ExE Boss <duke at openjdk.java.net> 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 574:
> 
>> 572:             try {
>> 573:                 Lookup lookup = MethodHandles.lookup();
>> 574:                 return lookup.defineHiddenClass(bytes, false, ClassOption.STRONG);
> 
> Actually, this lookup object should probably be kept cached.

Which one, the context lookup or the hidden class lookup?

-------------

PR: https://git.openjdk.java.net/jdk/pull/7744


More information about the core-libs-dev mailing list