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