RFR: 8264489: Add more logging to LargeCopyWithMark.java [v2]
Stefan Karlsson
stefank at openjdk.java.net
Wed Mar 31 11:19:08 UTC 2021
On Wed, 31 Mar 2021 09:29:43 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review Alan
>
> test/jdk/java/io/BufferedInputStream/LargeCopyWithMark.java line 29:
>
>> 27: * @summary BufferedInputStream calculates negative array size with large
>> 28: * streams and mark
>> 29: * @run main/othervm -Xmx4G -Xlog:gc,gc+heap,gc+ergo+heap -XX:+CrashOnOutOfMemoryError -XX:+IgnoreUnrecognizedVMOptions -XX:+G1ExitOnExpansionFailure LargeCopyWithMark
>
> Looks okay but I assume you can split this line to avoid having a 170+ line in this file, this helps with future side-by-side review.
Thanks. I've added line breaks to make it easier to read.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3282
More information about the hotspot-gc-dev
mailing list