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

Jaikiran Pai jpai at openjdk.org
Wed Aug 10 02:54:37 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?

Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:

  Review comment - use assertThrows()

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9806/files
  - new: https://git.openjdk.org/jdk/pull/9806/files/ee516ffd..e61cb275

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9806&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9806&range=00-01

  Stats: 9 lines in 1 file changed: 1 ins; 7 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