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