Review Request: Add ClassOption.STRONG and default is unspecified

Mandy Chung mandy.chung at oracle.com
Wed Mar 18 17:52:26 UTC 2020


"adjacent" does not seem having equivalent meaning in this case about 
the relationship with the class loader.    “coterminous” is a new word 
to me while I got the meaning once I googled it.  It may be okay for non 
native English speakers.

Alex contributes to this spec (big thanks to Alex).

Mandy

On 3/18/20 1:29 AM, Remi Forax wrote:
> As a non native English reader, i prefer the word "adjacent".
>
> otherwise, looks good.
>
> Rémi
>
> ----- Mail original -----
>> De: "John Rose" <john.r.rose at oracle.com>
>> À: "mandy chung" <mandy.chung at oracle.com>
>> Cc: "Alex Buckley" <alex.buckley at oracle.com>, "valhalla-dev" <valhalla-dev at openjdk.java.net>
>> Envoyé: Mercredi 18 Mars 2020 01:23:51
>> Objet: Re: Review Request: Add ClassOption.STRONG and default is unspecified
>> On Mar 17, 2020, at 1:41 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
>>> The javadoc is updated per offline discussion with Alex.
>>>     http://cr.openjdk.java.net/~mchung/valhalla/webrevs/hidden-classes/weak-strong-class/
>>>
>>> I pushed this version to the sandbox to get the new ClassOption::STRONG in.  We
>>> can continue to improve the javadoc per any feedback.
>> At a glance, it looks nice.  Love the $64 word “coterminous”.
>> Good advice about when to use strong vs. non-strong.



More information about the valhalla-dev mailing list