Looks fine,  Brian.

On 7/15/19 11:58 AM, Brian Burkhalter wrote:
So updated: http://cr.openjdk.java.net/~bpb/8187898/webrev.02/ 
<http://cr.openjdk.java.net/~bpb/8187898/webrev.02/>

Thanks,

Brian

On Jul 12, 2019, at 4:15 PM, Philip Race <philip.r...@oracle.com> wrote:

In that case just delete the comments and the commented out code ...

-phil.

On 7/12/19, 3:12 PM, Brian Burkhalter wrote:
I think I'd be more inclined to make the code at 970 like that at 1036.
Can’t do this exactly as drawImageBGR at 903 does not throw a PrinterException. 
Have to use RuntimeException.

Brian

On Jul 12, 2019, at 1:39 PM, Phil Race <philip.r...@oracle.com 
<mailto:philip.r...@oracle.com>> wrote:

oh well ... in which I suppose this is the best you can do here.

-phil.

On 7/12/19 1:10 PM, Brian Burkhalter wrote:
(2) I don't know why at 1034 you changed from PrinterIOException to 
PrinterException.
There is no PrinterIOException constructor which accepts a String as its only 
parameter and there is no IOException to pass to a PrinterIOException 
constructor.

Reply via email to