RFR: 8285405: add test and check for negative argument to HashMap::newHashMap et al [v2]
Stuart Marks
smarks at openjdk.org
Wed Aug 10 23:59:28 UTC 2022
On Wed, 10 Aug 2022 02:54:37 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> (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()
Thanks for following up on this. Looks good.
-------------
Marked as reviewed by smarks (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9806
More information about the core-libs-dev
mailing list