<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Sean,<br>
      <br>
      Without even commenting on the merits or necessity I note that you
      cannot possibly<br>
      have even built this patch, much less tested it.<br>
      <br>
      >
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      <span class="changed">625 return Boolean.FLASE;</span><br>
      <br>
      -phil.<br>
      <br>
      On 8/23/12 1:58 AM, Sean Chou wrote:<br>
    </div>
    <blockquote
cite="mid:CAOo=rxWjia+4tKse7UrvmBgYTfyns4DEvmS3-NF=EKuwbcdJcw@mail.gmail.com"
      type="cite">
      <div>Hello,</div>
      <div><br>
      </div>
      <div>    I updated the repository to 2d, the webrev is now: </div>
       <a moz-do-not-send="true"
        href="http://cr.openjdk.java.net/%7Ezhouyx/OJDK-429/webrev.01/">http://cr.openjdk.java.net/~zhouyx/OJDK-429/webrev.01/</a>
       
      <div>
         </div>
      <div>Please take a look.</div>
      <div><br>
        <div class="gmail_quote">---------- Forwarded message ----------<br>
          From: <b class="gmail_sendername">Sean Chou</b> <span
            dir="ltr"><<a moz-do-not-send="true"
              href="mailto:zhouyx@linux.vnet.ibm.com">zhouyx@linux.vnet.ibm.com</a>></span><br>
          Date: Thu, Aug 23, 2012 at 2:24 PM<br>
          Subject: Suggest a modification to isPostscript exception
          handling<br>
          To: <a moz-do-not-send="true"
            href="mailto:2d-dev@openjdk.java.net">2d-dev@openjdk.java.net</a><br>
          <br>
          <br>
          Hello,
          <div><br>
          </div>
          <div>     This is a simple modification
            to sun/print/PSPrinterJob.java. </div>
          <div>     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.</div>
          <div><br>
          </div>
          <div>The webrev is at: <a moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Ezhouyx/OJDK-429/webrev.00/"
              target="_blank">http://cr.openjdk.java.net/~zhouyx/OJDK-429/webrev.00/</a>  .</div>
          <div><br>
          </div>
          <div>Please take a look. </div>
          <span class="HOEnZb"><font color="#888888">
              <div><br>
              </div>
            </font></span></div>
        -- <br>
        Best Regards,<br>
        Sean Chou<br>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>