Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread Alan Bateman
On 16/12/2014 18:29, Phil Race wrote: Yes .. no technical constraint there. Although I was pondering going further and seeing if we can actually remove the public method java.awt.peer.Peer Component.getPeer() since with the module system the peer package is not accessible and this is then even

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread Phil Race
Yes .. no technical constraint there. Although I was pondering going further and seeing if we can actually remove the public method java.awt.peer.Peer Component.getPeer() since with the module system the peer package is not accessible and this is then even weirder than it is now .. as it effectiv

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread Jim Graham
Stop using them and replace them with new package private methods and a cross-package accessor (similar to the SurfaceManager.ImageAccessor pattern)? What are the technical constraints preventing this? I think the getPeer() method used to be used by AWT applications to tell if an app was on t

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067086: Suppress mac-specific deprecation warnings in the java.desktop module

2014-12-16 Thread Phil Race
Approved -phil. On 12/15/14 10:48 PM, joe darcy wrote: Hello, Please review this (hopefully) last change to suppress the deprecation warnings in the java.desktop module: 8067086: Suppress mac-specific deprecation warnings in the java.desktop module http://cr.openjdk.java.net/~darc

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread joe darcy
Thanks for the review. Just one more fix to review to finish suppressing the deprecation warnings in the desktop module; please also see the review request for JDK-8067086: Suppress mac-specific deprecation warnings in the java.desktop module: http://mail.openjdk.java.net/pipermail/2d-dev/20

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread Phil Race
Approved. We really need to figure out what to do about getPeer() - for this and for jigsaw .. -phil. On 12/16/2014 9:10 AM, joe darcy wrote: On 12/16/2014 3:15 AM, Alan Bateman wrote: On 16/12/2014 06:07, joe darcy wrote: Hello, Please review the next step of suppressing the deprecation

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread joe darcy
On 12/16/2014 3:15 AM, Alan Bateman wrote: On 16/12/2014 06:07, joe darcy wrote: Hello, Please review the next step of suppressing the deprecation warnings the client libraries, this time in windows-specific desktop code: JDK-8067092: Suppress windows-specific deprecation warnings in t

Re: [OpenJDK 2D-Dev] [9] Review Request: 8065373 [macosx] jdk8, jdk7u60 Regression in Graphics2D drawing of derived Fonts

2014-12-16 Thread Andrew Brygin
Hello Sergey, the change looks fine to me. Thanks, Andrew. On 12/12/2014 12:57 AM, Sergey Bylokhov wrote: Hi, Phil. - philip.r...@oracle.com wrote: > > Consistent treatment of concatenating sInverseTx seems right although > I am not sure of the overall picture here. > > Quartz uses "y in

Re: [OpenJDK 2D-Dev] JDK 9 RFR of JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module

2014-12-16 Thread Alan Bateman
On 16/12/2014 06:07, joe darcy wrote: Hello, Please review the next step of suppressing the deprecation warnings the client libraries, this time in windows-specific desktop code: JDK-8067092: Suppress windows-specific deprecation warnings in the java.desktop module http://cr.openjdk