On Tue, 17 Jan 2023 10:18:42 GMT, Claes Redestad wrote:
>> Using lambdas inside MethodHandles is quite dangerous given that lambdas are
>> initialized using method handles. It may work now because
>> longuestParameterList() is not called when initializing a lambda but it may
>> make any changes in the implementation of lambdas painfull in the future.
>
> Precious little method handle use in lambda bootstrap since JDK 11. Though I
> agree with the sentiment - having fixed a number of bootstrap issues in the
> past - `MethodHandles` is a small step up the abstraction ladder and the code
> in particular already uses a number of method refs and lambdas.
ok, two small changes,
- formatting: usually the method call in a stream are aligned with the '.' at
the beginning
```
stream
.filter(...)
.map(...)
```
instead of at the end.
- the reduce is a max(),
`max(Comparator.comparingInt(List::size))`
-
PR: https://git.openjdk.org/jdk/pull/12025