RFR: 8300543 Compiler Implementation for Pattern Matching for switch [v2]

[Vicente Romero]

On Thu, 20 Apr 2023 13:33:17 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> src/java.base/share/classes/java/lang/runtime/SwitchBootstraps.java line 170:
>> 
>>> 168:             } else if (label instanceof EnumDesc<?> enumDesc) {
>>> 169:                 if (target.getClass().isEnum() &&
>>> 170:                     ((Enum<?>) target).describeConstable().stream().anyMatch(d -> d.equals(enumDesc))) {
>> 
>> probably a matter of style but isn't it a bit too much to use streams here? there will be always only one element right?
>
> Technically, one element, but the type is optional, so formally could be no elements. I could use `.get()`, but that feels a bit ugly and against optional?

yep I know, up to you I'm not strong on this, I guess I was thinking about performance but probably not using streams won't make a difference in this case

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13074#discussion_r1172609916