<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Phil,<br>
<br>
<div class="moz-cite-prefix">On 11/13/2015 12:05 AM, Philip Race
wrote:<br>
</div>
<blockquote cite="mid:5644DBE6.10303@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
Shouldn't this use reference equality ?<br>
</blockquote>
It seems Object.java equals() method does == anyways and it seems it
always better to call equals() than to explicitly check for ==<br>
Is it not correct? If it is not, I will change it.<br>
<blockquote cite="mid:5644DBE6.10303@oracle.com" type="cite"> Also
the test does a number of wild card imports instead of<br>
the specific classes it needs. And I am not sure why it<br>
is importing java.awt.* anyway ..<br>
<br>
And I find qualifying "java.lang.String" to be unnecessary.<br>
</blockquote>
will update the test.<br>
<br>
Regards<br>
Prasanta<br>
<blockquote cite="mid:5644DBE6.10303@oracle.com" type="cite"> <br>
-phil.<br>
<br>
On 11/3/15, 12:59 AM, prasanta sadhukhan wrote:
<blockquote cite="mid:56387767.7060101@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=utf-8">
Hi All,<br>
<br>
Please review a fix for jdk9.<br>
<br>
Bug: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8067059">https://bugs.openjdk.java.net/browse/JDK-8067059</a><br>
webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Epsadhukhan/8067059/webrev.00/">http://cr.openjdk.java.net/~psadhukhan/8067059/webrev.00/</a><br>
<br>
The issue was if we "cancel" Printer Job setup dialog which has
been created with DialogSelectionType.NATIVE <br>
it should return "null" as per spec <br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://docs.oracle.com/javase/8/docs/api/java/awt/print/PrinterJob.html#pageDialog-javax.print.attribute.PrintRequestAttributeSet">https://docs.oracle.com/javase/8/docs/api/java/awt/print/PrinterJob.html#pageDialog-javax.print.attribute.PrintRequestAttributeSet</a>-<br>
<i>[If the user cancels the dialog, the attributes will not
reflect any changes made by the user, and the return value
will be null.]<br>
<br>
</i>We were returning PageFormat instead. Fixed to return null
in such case. <br>
<br>
Regards<br>
Prasanta<br>
</blockquote>
</blockquote>
<br>
</body>
</html>