Looks good, but needs a test.

> On 2 sep 2015, at 20:21, Jaroslav Bachorik <jaroslav.bacho...@oracle.com> 
> wrote:
> 
> Please, review the following change
> 
> Issue : https://bugs.openjdk.java.net/browse/JDK-8134420
> Webrev: http://cr.openjdk.java.net/~jbachorik/8134420/webrev.00
> 
> Currently, 'jps' will print multi-line vm args as they are - eg. splitting 
> the output into multiple lines. This kind of output is not very easy to read 
> and, incidentally, it can break this test.
> 
> My proposal is to replace all '\n' and '\r' characters with "\n" and "\r" 
> strings respectively to keep the output format being 'one line per one JVM'.
> 
> Thanks,
> 
> -JB-

Reply via email to