RFR: 8282662: Use List.of() factory method to reduce memory consumption [v5]
Сергей Цыпанов
duke at openjdk.java.net
Wed May 18 14:48:16 UTC 2022
> `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
-------------
Changes: https://git.openjdk.java.net/jdk/pull/7729/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7729&range=04
Stats: 12 lines in 4 files changed: 1 ins; 2 del; 9 mod
Patch: https://git.openjdk.java.net/jdk/pull/7729.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7729/head:pull/7729
PR: https://git.openjdk.java.net/jdk/pull/7729
More information about the security-dev
mailing list