RFR(M): 7178703: Fix handling of quoted arguments and better error messages in dcmd

[Rickard Bäckman]

Looks good.

/R

On 06/25/2012 03:49 PM, Staffan Larsen wrote:
> Here is an updated webrev. The last one didn't compile on Solaris.
>
> http://cr.openjdk.java.net/~sla/7178703/webrev.02/
>
> Thanks,
> /Staffan
>
> On 21 jun 2012, at 13:30, Staffan Larsen wrote:
>
>> Please review the following fix to the diagnostic command framework to improve information and error messages as well as handling of quoted arguments.
>>
>> webrev: http://cr.openjdk.java.net/~sla/7178703/webrev.01/
>>
>> The bug hasn't been made public yet so here is the description from the bug:
>> ----
>> Error messages in the dcmd parser can be improved to include information on which argument failed to parse or did not exists.
>>
>> Dcmd parser does not handle quoted arguments correctly (fails to parse them).
>> ----
>>
>> A new test has been added to verify the handling of quoted arguments.
>>
>> Thanks,
>> /Staffan
>
>