RFR: 8007333: [launcher] removes multiple back slashes (original) (raw)
Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri May 24 23:01:01 UTC 2013
- Previous message: RFR: 8007333: [launcher] removes multiple back slashes / trivial fix
- Next message: RFR: 8007333: [launcher] removes multiple back slashes / trivial fix
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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 http://cr.openjdk.java.net/~ksrini/8007333/webrev.1/webrev.delta/index.html
If there are no more comments I will push this by Tuesday.
Thanks Kumar
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 to me. -Alan
- Previous message: RFR: 8007333: [launcher] removes multiple back slashes / trivial fix
- Next message: RFR: 8007333: [launcher] removes multiple back slashes / trivial fix
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]