[OpenJDK 2D-Dev] Review: Remove private API from graphics code

Phil Race philip.race at oracle.com
Fri Aug 31 18:16:27 UTC 2012


These files were added by Bino to support printing. Quartz isn't used
except for printing in JDK 7, so as I understand it, testing on-screen in
Java2Demo should not exercise this code. I'm surprised that you saw
it being exercised. Did you do any printing testing ?

The matrix inversion seems unlikely to be applied to any non-invertible
matrices, so that's fine, but I wonder if you have lost precision here
due to floating point inaccuracies ?

If you originally had a simple scale or identity, rotated it, and then
applied the inverse to unrotate it, do you really end up with exactly
the same results. The more you do this the more inaccuracies creep in,
which may be part of the reason for the original approach.
I find it a little hard to believe that there isn't a direct public way to
restore a transform.

The changes for mountain lion are safe for snow leopard I presume?
I believe the builds still happen on snow leopard.

Also this should have been sent to 2d-dev, not awt-dev.
These files, APIs, and printing are all 2D, not awt.


I am not sure why they are not used but it
On 8/31/2012 10:45 AM, Scott Kovatch wrote:
> http://cr.openjdk.java.net/~skovatch/7187834/webrev.00/
> This is based on the patch submitted by Marco Dinacci. I had to modify it a bit to get it to compile against 7u-dev, but nothing major.
> The changes in ImageSurfaceData.h were needed as a result of my use of Xcode 4.4.1 on Mountain Lion.
> I ran the Java2Demo and don't see any problems. I paid close attention to the Images tab, since it looks like this code is heavily used by it.
> -- Scott K.
> ----------------------------------------
> Scott Kovatch
> scott.kovatch at oracle.com
> Santa Clara/Pleasanton, CA

More information about the 2d-dev mailing list