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

Akhil Arora akhil.arora at oracle.com
Sat May 25 00:50:10 UTC 2013


looks good

On 05/24/2013 04:01 PM, Kumar Srinivasan wrote:
> 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