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

I've added the test for `LinkedHashMap.newLinkedHashMap(int)` in the existing 
`test/jdk/java/util/LinkedHashMap/Basic.java` since that test has tests for 
various APIs of this class.

For `WeakHashMap.newWeakHashMap` and `HashMap.newHashMap`, I have created new 
test classes under relevant locations, since these classes already have test 
classes (almost) per API/feature in those locations.

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

Commit messages:
 - 8285405: add test and check for negative argument to HashMap::newHashMap et 
al

Changes: https://git.openjdk.java.net/jdk/pull/9036/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=9036&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8285405
  Stats: 189 lines in 6 files changed: 186 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/9036.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/9036/head:pull/9036

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

Reply via email to