RFR: 8282662: Use List/Set.of() factory methods to reduce memory consumption [v2]
Сергей Цыпанов
duke at openjdk.java.net
Wed Mar 9 08:35:45 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 incrementally with one additional commit since the last revision:
8282662: Revert dubious changes
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7729/files
- new: https://git.openjdk.java.net/jdk/pull/7729/files/b31edfcd..5bbe8c4e
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7729&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7729&range=00-01
Stats: 6 lines in 2 files changed: 1 ins; 0 del; 5 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