jmx-dev RFR: 8285149: Using HashMap.newHashMap to replace new HashMap(int)

2022-04-20 Thread XenoAmess
These are the changes that too many to be reviewed in 8186958, thus split some of them out. - Commit messages: - 9073085: ues HashMap.newHashMap to replace new HashMap(int) Changes: https://git.openjdk.java.net/jdk/pull/8301/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk

Re: jmx-dev RFR: 8285149: Using HashMap.newHashMap to replace new HashMap(int)

2022-04-20 Thread liach
On Tue, 19 Apr 2022 17:44:10 GMT, XenoAmess wrote: > These are the changes that too many to be reviewed in 8186958, thus split > some of them out. Would you consider replacing usages of `new HashMap<>(size, 1)`, such as in `AnnotationInvocationHandler`, as well? Such maps have a smaller table

Re: jmx-dev RFR: 8285149: Using HashMap.newHashMap to replace new HashMap(int) [v2]

2022-04-20 Thread XenoAmess
> These are the changes that too many to be reviewed in 8186958, thus split > some of them out. XenoAmess has updated the pull request incrementally with two additional commits since the last revision: - add more replaces - add more replaces - Changes: - all: https://git.openj

Re: jmx-dev RFR: 8285149: Using HashMap.newHashMap to replace new HashMap(int) [v3]

2022-04-20 Thread XenoAmess
> These are the changes that too many to be reviewed in 8186958, thus split > some of them out. XenoAmess has updated the pull request incrementally with one additional commit since the last revision: add more replaces - Changes: - all: https://git.openjdk.java.net/jdk/pull/83