RFR 8223593 : Refactor code for reallocating storage

Ivan Gerasimov ivan.gerasimov at oracle.com
Sat May 11 04:01:46 UTC 2019


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