RFR: 8193031: Collections.addAll is likely to perform worse than Collection.addAll [v2]

Сергей Цыпанов github.com+10835776+stsypanov at openjdk.java.net
Thu Dec 17 10:47:57 UTC 2020


On Thu, 17 Dec 2020 10:34:38 GMT, Rémi Forax <github.com+828220+forax at openjdk.org> wrote:

>> Сергей Цыпанов has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit:
>> 
>>   8193031: add elements in bulk in Collections.addAll()
>
> src/java.base/share/classes/java/util/Collections.java line 5589:
> 
>> 5587:      */
>> 5588:     @SafeVarargs
>> 5589:     @SuppressWarnings("varargs")
> 
> I don't think you need a SuppressWarnings here

Hi, without this I get failed build:
Compiling 3057 files for java.base
Compiling 89 properties into resource bundles for java.desktop

        return c.addAll(Arrays.asList(elements));
                                      ^
error: warnings found and -Werror specified
1 error
1 warning
make[3]: *** [/home/s.tsypanov/IdeaProjects/jdk-github/build/linux-x86_64-server-release/jdk/modules/java.base/_the.java.base_batch] Error 1
CompileJavaModules.gmk:604: recipe for target '/home/s.tsypanov/IdeaProjects/jdk-github/build/linux-x86_64-server-release/jdk/modules/java.base/_the.java.base_batch' failed
make[2]: *** [java.base-java] Error 2
make[2]: *** Waiting for unfinished jobs....
make/Main.gmk:193: recipe for target 'java.base-java' failed

-------------

PR: https://git.openjdk.java.net/jdk/pull/1764


More information about the core-libs-dev mailing list