Re: 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: RFR: 8272992: Replace usages of Collections.sort with List.sort call in jdk.* modules

2021-09-13 Thread liach
On Mon, 13 Sep 2021 17:56:14 GMT, Rémi Forax wrote: >> Collections.sort is just a wrapper, so it is better to use an instance >> method directly. > > src/jdk.jfr/share/classes/jdk/jfr/internal/consumer/RepositoryFiles.java line > 215: > >> 213: pathLookup.remove(remove); >> 214