Hi Roger,

> On Mar 15, 2019, at 12:57 PM, Roger Riggs <roger.ri...@oracle.com> wrote:
> 
> Looks ok.
> 
> In the test, you could probably create the string using.
>     "\u0800".repeat(size);

Good idea.

> Instead of disabling the test statically, you could make it conditional
> on Runtime.maxMemory but the test will fail quickly anyway.
> 
> 
> DataOutputStream.java:
> 
> Can you fix the indent lines 387-389, it seems to be only 3 chars.

Will do.

Thanks,

Brian

Reply via email to