8219196: DataOutputStream.writeUTF may throw unexpected exceptions

Brian Burkhalter brian.burkhalter at oracle.com
Fri Mar 15 21:06:19 UTC 2019


Updated: http://cr.openjdk.java.net/~bpb/8219196/webrev.01/

> On Mar 15, 2019, at 1:05 PM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
> 
>> On Mar 15, 2019, at 12:57 PM, Roger Riggs <Roger.Riggs at oracle.com <mailto:Roger.Riggs at 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.

For this I simply added the requirement to the jtreg tags instead.

>> DataOutputStream.java:
>> 
>> Can you fix the indent lines 387-389, it seems to be only 3 chars.
> 
> Will do.

Thanks,

Brian


More information about the core-libs-dev mailing list