Sorry about the repository, I'll update the webrev. On Thu, Aug 23, 2012 at 2:24 PM, Sean Chou <[email protected]>wrote:
> 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 > > -- Best Regards, Sean Chou
