Need another review for this.
On 2015-12-03 11:11, Marcus Larsson wrote:
Hi,
New webrev:
http://cr.openjdk.java.net/~mlarsson/8144220/webrev.01/
Incremental:
http://cr.openjdk.java.net/~mlarsson/8144220/webrev.00-01/
Added more test cases to the test, and renamed it to
TestQuotedLogOutputs.
The updated test found that partially quoted output names that didn't
start
with the quote character were accepted. This has been fixed.
Test passes in RBT.
Thanks,
Marcus
On 2015-12-02 14:41, Marcus Larsson wrote:
Hi,
Please review the following patch to allow full path names for log
files on windows.
The full path to files will on windows contain a colon (from the C:\
part), and there used to be no way to avoid that colon from being
parsed as the delimiter for the next -Xlog: option. With this patch
it is now possible to avoid this by giving the path within quotation
marks, for example: -Xlog:all:"C:\out.log". Any colons within quotes
will not be parsed as delimiters for -Xlog.
Webrev:
http://cr.openjdk.java.net/~mlarsson/8144220/webrev.00/
Issue:
https://bugs.openjdk.java.net/browse/JDK-8144220
Testing:
New test through RBT.
Thanks,
Marcus