RFR 8223593 : Refactor code for reallocating storage

Pavel Rappo pavel.rappo at oracle.com
Mon May 13 14:44:14 UTC 2019


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