[OpenJDK 2D-Dev] 8187898: PrintStream should override FilterOutputStream#write(byte[]) with a method that has no throws clause

[Brian Burkhalter]

Here is new webrev incorporating the two changes below.

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

Thanks,

Brian

> On Jul 12, 2019, at 12:01 PM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
> 
>> On Jul 12, 2019, at 11:17 AM, Roger Riggs <Roger.Riggs at oracle.com <mailto:Roger.Riggs at oracle.com>> wrote:
>> 
>> Would it be appropriate to add @Override to the new method (and perhaps existing overridden methods).
> 
> Yes, I think so.
> 
>> Previously, calling FilterOutputStream.write(byte[]) would delegate to write(byte[], 0, length).
>> The proposed change duplicates the code and changes the ways that overridden classes might see the call.
>> What's the benefit of duplicating the code and calling out.write(buf)?
> 
> Probably nothing. Probably better to call write(b,0,b.length) directly.
> 
>> Not my call, but in PSPrinterJob.java (970-977) it might be cleaner to just delete the unused code and comment.
> 
> I agree but I was waiting for a suggestion from a 2D reviewer.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/2d-dev/attachments/20190712/3de1f384/attachment.html>