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