RFR: 8147500: The HashtableTextDump::get_num() should check for integer overflow

[David Holmes]

PS. Don't forget copyright updates :)

On 22/01/2016 2:37 PM, David Holmes wrote:
> Hi Jiangli,
>
>> Please review the following improvements to
>> HashtableTextDump::get_num() and HashtableTextDump::corrupted_if()
>> described in JDK-8147500
>> <https://bugs.openjdk.java.net/browse/JDK-8147500>.
>>
>>  - The HashtableTextDump::get_num() is use to parse an integer value
>> from the input file. It should check for integer overflow and report
>> error without continuing.
>>  - The HashtableTextDump::corrupted_if() should take an additional
>> argument so caller can pass a meaningful error message.
>>
>> Webrev: http://cr.openjdk.java.net/~jiangli/8147500/webrev.00/
>
> As improvements on the existing code this seems fine.
>
> Thanks,
> David