Hello,
I updated the repository to 2d, the webrev is now:
http://cr.openjdk.java.net/~zhouyx/OJDK-429/webrev.01/
Please take a look.
---------- Forwarded message ----------
From: Sean Chou <[email protected]>
Date: Thu, Aug 23, 2012 at 2:24 PM
Subject: Suggest a modification to isPostscript exception handling
To: [email protected]
Hello,
This is a simple modification to sun/print/PSPrinterJob.java.
When sun.print.IPPPrintService.isPostscript method checks if the
printer is a postscript printer, if IOException happens, the method assumes
the printer is postscript printer (IPPPrintService.java, line 1605). In
class PSPrinterJob, it invoke isPostscript and assumes all Throwables to
be a postscript printer ( PSPrinterJob.java, line 625 ). I think it should
return false in cases exceptions other than IOException are
caught, IOException should not be expanded to all Throwable.
The webrev is at: http://cr.openjdk.java.net/~zhouyx/OJDK-429/webrev.00/ .
Please take a look.
--
Best Regards,
Sean Chou
