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

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri May 24 23:01:01 UTC 2013


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




More information about the core-libs-dev mailing list