RFR: 8282662: Use List.of() factory method to reduce memory consumption [v8]
Roger Riggs
rriggs at openjdk.java.net
Tue May 31 19:36:44 UTC 2022
On Tue, 31 May 2022 19:30:07 GMT, Сергей Цыпанов <duke at openjdk.java.net> wrote:
>> `List.of()` along with `Set.of()` create unmodifiable `List/Set` but with smaller footprint comparing to `Arrays.asList()` / `new HashSet()` when called with vararg of size 0, 1, 2.
>>
>> In general replacement of `Arrays.asList()` with `List.of()` is dubious as the latter is null-hostile, however in some cases we are sure that arguments are non-null. Into this PR I've included the following cases (in addition to those where the argument is proved to be non-null at compile-time):
>> - `MethodHandles.longestParameterList()` never returns null
>> - parameter types are never null
>> - interfaces used for proxy construction and returned from `Class.getInterfaces()` are never null
>> - exceptions types of method signature are never null
>
> Сергей Цыпанов has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>
> - 8282662: Revert wrong copyright year change
> - 8282662: Revert ProxyGenerator
> - 8282662: Revert ProxyGenerator
> - 8282662: Revert dubious changes in MethodType
> - 8282662: Revert dubious changes
> - 8282662: Use List/Set.of() factory methods to save memory
Why the force push? They are discouraged, making it harder to review.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7729
More information about the security-dev
mailing list