Re: RFR: 8007333: [launcher] removes multiple back slashes / trivial fix

2013-05-24 Thread Kumar Srinivasan
After some hallway conversation with Akhil, I added some more tests with progressive number of \ handling, just to make sure, besides that there are no other changes. full webrev: http://cr.openjdk.java.net/~ksrini/8007333/webrev.1/ delta webrev

Re: RFR: 8007333: [launcher] removes multiple back slashes / trivial fix

2013-05-23 Thread Alan Bateman
On 22/05/2013 21:30, Kumar Srinivasan wrote: Hello, Please review trivial fix where the launcher did not parse multiple back-slashes correctly. I also added some additional test contributed by Jim Holmlund. http://cr.openjdk.java.net/~ksrini/8007333/webrev.0/ Thanks Kumar This looks okay

RFR: 8007333: [launcher] removes multiple back slashes / trivial fix

2013-05-22 Thread Kumar Srinivasan
Hello, Please review trivial fix where the launcher did not parse multiple back-slashes correctly. I also added some additional test contributed by Jim Holmlund. http://cr.openjdk.java.net/~ksrini/8007333/webrev.0/ Thanks Kumar