On Dec 21, 2017, at 3:05 AM, Alan Bateman <alan.bate...@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