Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-07-04 Thread Jayathirth D V
Hi Prasanta,   Changes are working fine.   Thanks, Jay   From: Phil Race Sent: Friday, July 01, 2016 11:55 PM To: Prasanta Sadhukhan Cc: 2d-dev Subject: Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified   +1  ... although I am

Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-07-01 Thread Phil Race
+1 ... although I am a little nervous about FP error causing false results in some cases. -phil. On 06/30/2016 11:10 PM, Prasanta Sadhukhan wrote: Hi Phil, Right. I have rectified the calculation. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/6789262/webrev.02/

Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-07-01 Thread Prasanta Sadhukhan
Hi Phil, Right. I have rectified the calculation. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/6789262/webrev.02/ Regards Prasanta On 7/1/2016 3:29 AM, Phil Race wrote: Shouldn't these all be >= and <= ? Else (eg) for X==1 in both cases you would return false when it

Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-06-30 Thread Phil Race
Shouldn't these all be >= and <= ? Else (eg) for X==1 in both cases you would return false when it should be true .. f ((mpa.getX(units) > mediaPrintables[i].getX(units)) && -phil. On 06/30/2016 03:31 AM, Prasanta Sadhukhan wrote: Hi Phil, I modified the method to check for "fit" rather

Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-06-30 Thread Prasanta Sadhukhan
Hi Phil, I modified the method to check for "fit" rather than perfect match as you suggested. Please find the modified webrev http://cr.openjdk.java.net/~psadhukhan/6789262/webrev.01/ Regards Prasanta On 6/29/2016 11:29 PM, Philip Race wrote: This is actually a little subtle. The check you

Re: [OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-06-29 Thread Philip Race
This is actually a little subtle. The check you are deleting has been there since 2000 .. and it can't just be removed. But it need adjusting. This is the subtlety. Most of the attribute checks are yes/no. I think for this one we need to look at the actual dimensions and return true if it is

[OpenJDK 2D-Dev] [9] RFR JDK-6789262: getPageFormat doesn't apply PrintRequestAttributeSet specified

2016-06-27 Thread Prasanta Sadhukhan
Hi All, Please review a fix for an issue where it was found that setting left margin of 2" is ignored and when we query PrinterJob.job.getPageFormat(PrintRequestAttributeSet) we get the default paper mediapritable area of x=1" instead of the set 2". Bug: