[OpenJDK 2D-Dev] [9] RFR JDK-8067059, , PrinterJob.pageDialog() with DialogSelectionType.NATIVE returns a PageFormat when cancelled.

prasanta sadhukhan prasanta.sadhukhan at oracle.com
Fri Nov 13 06:23:42 UTC 2015


Thanks Phil. I was indeed relying on non-presence of equals method in 
PageFormat.

I have modified the webrev to use equality. Can you please review ?
http://cr.openjdk.java.net/~psadhukhan/8067059/webrev.01/

Regards
Prasanta
On 11/13/2015 11:35 AM, Philip Race wrote:
>
>
> On 11/12/15, 9:52 PM, prasanta sadhukhan wrote:
>> Hi Phil,
>>
>> 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.
>
> You are relying on PageFormat not over-riding equals.
> It could do so and then the intent of your equals call would be broken
> in the case that updates to the dialog resulted in a new PageFormat
> but with the equal fieids.
>
> -phil.
>
>>> 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.
>>
>> Regards
>> Prasanta
>>>
>>> -phil.
>>>
>>> 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
>>>> https://docs.oracle.com/javase/8/docs/api/java/awt/print/PrinterJob.html#pageDialog-javax.print.attribute.PrintRequestAttributeSet-
>>>> /[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 
>>>> case.
>>>>
>>>> Regards
>>>> Prasanta
>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20151113/df14d747/attachment.html>


More information about the 2d-dev mailing list