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