RFR 8193832: Performance of InputStream.readAllBytes() could be improved

Alan Bateman Alan.Bateman at oracle.com
Thu Dec 21 11:05:46 UTC 2017

On 20/12/2017 22:30, Brian Burkhalter wrote:
> :
> http://cr.openjdk.java.net/~bpb/8193832/webrev.03/
> The patch is updated to:
> * use Peter’s approach to avoid allocating an ArrayList when length <= DEFAULT_BUFFER_SIZE;
> * use the default ArrayList constructor instead of that with a specific initial capacity;
> * update the test to ensure that lengths which require three buffers are covered.
This version looks okay although fragile to maintain due to the code paths.

Have you checked that the updated test covers all cases?


More information about the core-libs-dev mailing list