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

Brian Burkhalter brian.burkhalter at oracle.com
Fri Jul 12 19:01:46 UTC 2019

Hi Roger,

> On Jul 12, 2019, at 11:17 AM, Roger Riggs <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.

> (The CSR will need to note the source incompatibility due to the removal of a declared exception)




More information about the core-libs-dev mailing list