RFR: 8285405: add test and check for negative argument to HashMap::newHashMap et al

Jaikiran Pai jpai at openjdk.org
Tue Aug 9 09:44:59 UTC 2022


(This is a recreation of a previous pull request which had received some reviews https://github.com/openjdk/jdk/pull/9036. I had to delete that personal branch and recreate it due to some git issues)

Can I please get a review of this change which addresses https://bugs.openjdk.java.net/browse/JDK-8285405?

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

Commit messages:
 - incorporate review comments from Stuart
 - Merge latest from master branch
 - add newline at the end of the new file
 - Add tests for the new HashSet#newHashSet and LinkedHashSet#newLinkedHashSet APIs
 - add a check for negative numElements very early in the new APIs of HashSet and LinkedHashSet
 - 8285405: add test and check for negative argument to HashMap::newHashMap et al

Changes: https://git.openjdk.org/jdk/pull/9806/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9806&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8285405
  Stats: 48 lines in 6 files changed: 46 ins; 1 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/9806.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9806/head:pull/9806

PR: https://git.openjdk.org/jdk/pull/9806


More information about the core-libs-dev mailing list