On Tue, 22 Jun 2021 20:43:03 GMT, Maurizio Cimadamore <[email protected]>
wrote:
>> Jan Lahoda has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Updating javadoc, as suggested.
>
> src/java.base/share/classes/java/lang/runtime/SwitchBootstraps.java line 175:
>
>> 173: * Bootstrap method for linking an {@code invokedynamic} call site
>> that
>> 174: * implements a {@code switch} on a target of an enum type. The
>> static
>> 175: * arguments are used to encode the case labels associated to the
>> switch
>
> `String` and `Class` should appear in code blocks perhaps, or link tags?
> Also, I think this text could be improved by splitting the sentence by using
> a bullet list:
>
>
> The static arguments are used to encode the case labels associated to the
> switch
> construct, where each label can be encoded in two ways:
> * as a String value, which represents the name of the enum constant
> associated with the label
> * as a Class value, which represents the enum type associated with a type
> test pattern
Thanks! I've (tried to) updated the javadoc as suggested.
-------------
PR: https://git.openjdk.java.net/jdk17/pull/81
