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

Jim Laskey jlaskey at openjdk.java.net
Fri Mar 11 13:13:44 UTC 2022


On Fri, 11 Mar 2022 10:42:02 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Jim Laskey has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Leave public for the time being
>
> src/java.base/share/classes/java/lang/runtime/Carrier.java line 57:
> 
>> 55:  * @since 19
>> 56:  */
>> 57: /*non-public*/
> 
> What does this non-public comment mean?

We might switch to package private for String Template preview. I forgot that the test can't locate the class if I make it package private,

> src/java.base/share/classes/java/lang/runtime/Carrier.java line 146:
> 
>> 144:      * @return constructor with arguments recasted and reordered
>> 145:      */
>> 146:     private static MethodHandle reshapeConstructor(CarrierShape carrierShape,
> 
> I like how reshape constructors and components is now shared across both implementations

Refactoring is just another tool for the lazy. ;-)

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

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


More information about the core-libs-dev mailing list