Review Request: 8056298 Separate java.awt.datatransfer from the desktop module
Alan.Bateman at oracle.com
Mon Jan 12 20:42:16 UTC 2015
On 12/01/2015 20:29, Sergey Bylokhov wrote:
> Please review a fix for jdk 9.
> In the fix a sun.datatransfer and a java.awt.datatransfer packages
> were moved to the separate module java.datatransfer.
> But sun.awt.datatransfer still located in java.desktop. I tested full
> jdk(all modules included) on osx using a jck, and it works without
> problems, at least no new issues were found. But datatransfer area is
> quite unstable after a bunch of the latest fixes.
> I still cannot replace java.desktop in modules.xml, because of some
> another dependencies(See comments in the bug)
> Bug: https://bugs.openjdk.java.net/browse/JDK-8056298
> Webrev can be found at:
Thanks for doing this. I think it looks okay except for modules.xml
where it looks like there may be a few issues.
1. You've updated the definition of java.corba to depend on
java.datatransfer but I don't think this is needed (is there code in
java.corba that uses this API?).
2. Same thing needs to be checked in java.xml.bind, jdk.jconsole and
jdk.runtime where it's not clear to me that code in these modules uses
2. The update to java.activation to depend on java.datatransfer looks
right but shouldn't you drop the dependency on java.desktop?
I don't have any other comments except to mention that java.xml.soap has
been subsumed into java.xml.ws. Those changes are in jdk9/dev and it
looks like jdk9/client might be a bit out of date.
More information about the build-dev