RFR: 8300237: Minor improvements in MethodHandles [v2]
Rémi Forax
forax at openjdk.org
Tue Jan 17 15:33:18 UTC 2023
On Tue, 17 Jan 2023 10:18:42 GMT, Claes Redestad <redestad at openjdk.org> 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
More information about the core-libs-dev
mailing list