RFR(XS): 8159901: missing newline char in the exception messages in diagnosticArgument.cpp

[Calvin Cheung]

Fred,

Thanks for your quick review.

Calvin

On 7/13/16, 2:07 PM, Frederic Parain wrote:
> Fix looks good to me.
>
> Thank you fixing this.
>
> Fred
>
> On 13/07/2016 17:33, Calvin Cheung wrote:
>>
>> Please review this small fix for adding the missing newline char in the
>> exception messages in diagnosticArgument.cpp.
>>
>> If the vm_exit_during_initialization() is called after the THROW_MSG()
>> in diagnosticArgument.cpp, the error looks like the following:
>> java.lang.IllegalArgumentException: Integer parsing error nanotime
>> value: syntax errorError occurred during initialization of VM
>> Failed to start tracing backend.
>>
>> After the fix, it becomes:
>> java.lang.IllegalArgumentException: Integer parsing error nanotime
>> value: syntax error
>> Error occurred during initialization of VM
>> Failed to start tracing backend.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8159901
>>          (it is marked confidential)
>>
>> webrev: http://cr.openjdk.java.net/~ccheung/8159901/webrev.00/
>>
>> The fix passed JPRT and related tests in the closed repo.
>>
>> thanks,
>> Calvin
>>
>