[10] RFR(XS): 8180473: Use proper deallocation for FileBuff::_bigbuf

[Zoltán Majó]

Thank you, Vladimir, for the review!

Best regards,


Zoltan


On 05/18/2017 01:46 AM, Vladimir Kozlov wrote:
> Good.
>
> Thanks,
> Vladimir
>
> On 5/17/17 3:52 AM, Zoltán Majó wrote:
>> Hi,
>>
>>
>> please review the following fix for 8180473.
>> https://bugs.openjdk.java.net/browse/JDK-8180473
>> http://cr.openjdk.java.net/~zmajo/8180473/webrev.00/
>>
>> It is a good coding practice to properly deallocate dynamically 
>> allocated resources. In particular, dynamically allocated memory for 
>> arrays using the array-specific new[] operator is supposed to be
>> deallocated using the array-specific delete[] operator. The 
>> deallocation of FileBuff::_bigbuf does not follow this recommendation 
>> and should therefore be changed. Tested with JPRT.
>>
>> Thank you!
>>
>> Best regards,
>>
>>
>> Zoltan
>>