Thanks again for going through all the changes in this area.

I've reverted the changes to java/awt/color/ICC_Profile.java and sun/print/ServiceDialog.java as they changes were clearly left over from early exploration into resource encapsulation.

I've decided not to touch com/apple/laf/AquaUtils.java as that is something that should be cleaned up in jdk9/client. The only reason we had to touch this code was because it directly used sun.misc.Launcher, which is now gone.

Yuri needs to look at test/java/awt/xembed/server/TesterClient.java as that addExports usage is not right. I hope Yuri can also help on your comment about why skipTestingEmbeddedFrame=true is commented out in test/java/awt/Mixing/AWT_Mixing/OverlappingTestBase.java.

test/java/awt/Toolkit/Headless/WrappedToolkitTest/WrappedToolkitTest.sh could do with a clean-up too, I assume the @compile was dropped because the test is invoking javac directly now.

-Alan

Reply via email to