New webrev: http://cr.openjdk.java.net/~sla/7167157/webrev.01/
On 11 maj 2012, at 12:17, Staffan Larsen wrote:
> Right. Should probably be line.trim().equals("stop")
>
> /Staffan
>
> On 11 maj 2012, at 11:41, Nils Loodin wrote:
>
>> Should it be "startsWith"? I don't know if we ever plan to implement a
>> command that's called stopFeatureX, but still..
>>
>> Regards,
>> Nils Loodin
>>
>> On May 11, 2012, at 11:34 , Staffan Larsen wrote:
>>
>>> Please review the following fix for the fix for 7154822. Previously all
>>> lines in the command file were sent to the JVM as one string, but the size
>>> limit was 1024 bytes so this was changed in 7154822 to sending one line at
>>> a time. Problem with this is that the jcmd utility keeps sending commands
>>> even if it has encountered a 'stop'. So instead of relying on the JVM to
>>> parse the 'stop' command, we must now parse this in the jcmd utility.
>>>
>>> webrev: http://cr.openjdk.java.net/~sla/7167157/webrev.00/
>>>
>>> Thanks,
>>> /Staffan
>>
>
