[OpenJDK 2D-Dev]  RFR JDK-8067059, , PrinterJob.pageDialog() with DialogSelectionType.NATIVE returns a PageFormat when cancelled.
prasanta.sadhukhan at oracle.com
Fri Nov 13 05:52:13 UTC 2015
On 11/13/2015 12:05 AM, Philip Race wrote:
> Shouldn't this use reference equality ?
It seems Object.java equals() method does == anyways and it seems it
always better to call equals() than to explicitly check for ==
Is it not correct? If it is not, I will change it.
> Also the test does a number of wild card imports instead of
> the specific classes it needs. And I am not sure why it
> is importing java.awt.* anyway ..
> And I find qualifying "java.lang.String" to be unnecessary.
will update the test.
> On 11/3/15, 12:59 AM, prasanta sadhukhan wrote:
>> Hi All,
>> Please review a fix for jdk9.
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8067059
>> webrev: http://cr.openjdk.java.net/~psadhukhan/8067059/webrev.00/
>> The issue was if we "cancel" Printer Job setup dialog which has been
>> created with DialogSelectionType.NATIVE
>> it should return "null" as per spec
>> /[If the user cancels the dialog, the attributes will not reflect any
>> changes made by the user, and the return value will be null.]
>> /We were returning PageFormat instead. Fixed to return null in such
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the 2d-dev