On Wed, 31 Mar 2021 09:29:43 GMT, Alan Bateman <al...@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
