RFR 8223593 : Refactor code for reallocating storage

Roger Riggs Roger.Riggs at oracle.com
Tue May 14 14:50:54 UTC 2019


Hi Ivan,

The updated patch looks fine.

Strictly speaking, the change to Files.readAllBytes is not indicated by 
the bug report
so please update or comment on the bug report to mention that change.

Thanks, Roger


On 05/13/2019 10:44 AM, Pavel Rappo wrote:
> Thanks for updating your patch. The updated code seems fine.
>
> -Pavel
>
>> On 11 May 2019, at 05:01, Ivan Gerasimov <ivan.gerasimov at oracle.com> wrote:
>>
>> Hello!
>>
>> Please help review the updated fix.
>>
>> This new webrev includes changes suggested by Pavel, Peter and Roger.
>>
>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8223593
>> WEBREV: http://cr.openjdk.java.net/~igerasim/8223593/01/webrev/
>>
>> Please note that the behavior of j.n.f.Files.readAllBytes() has changed slightly, so now it *may* be possible to read a file larger than (Integer.MAX_VALUE - 8), if VM is able to allocate that large array.
>>
>> With kind regards,
>> Ivan
>>
>> On 5/8/19 6:50 PM, Ivan Gerasimov wrote:
>>> Hello!
>>>
>>> Jdk has several places with similar logic:  an array needs to be reallocated (by at least some defined amount), taking into account the maximum allowed size of arrays.
>>>
>>> There's clearly an opportunity for refactoring, so it is proposed to introduce a dedicated utility method for calculating the best new size of an array.
>>>
>>> Would you please help review this enhancement?
>>>
>>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8223593
>>> WEBREV: http://cr.openjdk.java.net/~igerasim/8223593/00/webrev/
>>>
>>> Mach5 job ran fine.
>>>
>>> Thanks in advance!
>>>
>> -- 
>> With kind regards,
>> Ivan Gerasimov
>>



More information about the core-libs-dev mailing list