[12] 8206403: ByteArrayOutputStream hugeCapacity method can return invalid capacity
Brian Burkhalter
brian.burkhalter at oracle.com
Mon Jul 23 21:49:21 UTC 2018
Hi Roger,
Updated version: http://cr.openjdk.java.net/~bpb/8206403/webrev.01/
On Jul 23, 2018, at 2:10 PM, Roger Riggs <Roger.Riggs at Oracle.com> wrote:
> You might want to add an @requires of 8Gb or whatever so the test only runs on a system it can succeed on.
Re-tested and changed to @requires 2g and dropped the @ignore.
> I don't see the @randomness in the test. (Other than perhaps available heap).
That was vestigial from copying the header from elsewhere.
> ByteArrayOutputStream:121: A message indicating the nature of the error would be useful.
There was no message in the original file but I concur that one is better so I added one.
> In the test, I don't think you need to fill the array, writing 0 is just as good as 0xff.
Changed.
Thanks for the review.
Brian
More information about the core-libs-dev
mailing list