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

[Brian Burkhalter]

On Dec 21, 2017, at 3:05 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> 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?

I think it covers all of them except the OOME. I’ll review it again.

Brian