changeset 1039:414db6f4868d

8169069: Module system implementation refresh (11/2016) Reviewed-by: alanb, mchung Contributed-by: alan.bateman@oracle.com, mandy.chung@oracle.com
author alanb
date Thu, 01 Dec 2016 08:57:38 +0000
parents f1042f0aa643
children f41607eb5ffc
files src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/trax/TemplatesImpl.java src/jdk.xml.dom/share/classes/module-info.java test/TEST.ROOT test/javax/xml/jaxp/functional/catalog/DeferFeatureTest.java test/javax/xml/jaxp/module/ServiceProviderTest/src/unnamed/Main.java
diffstat 5 files changed, 7 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/trax/TemplatesImpl.java	Wed Nov 23 16:16:35 2016 +0000
+++ b/src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/trax/TemplatesImpl.java	Thu Dec 01 08:57:38 2016 +0000
@@ -471,8 +471,7 @@
             String pn = _tfactory.getPackageName();
             assert pn != null && pn.length() > 0;
 
-            ModuleDescriptor descriptor
-                = new ModuleDescriptor.Builder(mn)
+            ModuleDescriptor descriptor = ModuleDescriptor.module(mn)
                     .requires("java.xml")
                     .exports(pn)
                     .build();
--- a/src/jdk.xml.dom/share/classes/module-info.java	Wed Nov 23 16:16:35 2016 +0000
+++ b/src/jdk.xml.dom/share/classes/module-info.java	Thu Dec 01 08:57:38 2016 +0000
@@ -24,7 +24,7 @@
  */
 
 module jdk.xml.dom {
-    requires public java.xml;
+    requires transitive java.xml;
     exports org.w3c.dom.css;
     exports org.w3c.dom.html;
     exports org.w3c.dom.stylesheets;
--- a/test/TEST.ROOT	Wed Nov 23 16:16:35 2016 +0000
+++ b/test/TEST.ROOT	Thu Dec 01 08:57:38 2016 +0000
@@ -23,7 +23,7 @@
 groups=TEST.groups
 
 # Minimum jtreg version
-requiredVersion=4.2 b03
+requiredVersion=4.2 b04
 
 # Use new module options
 useNewOptions=true
--- a/test/javax/xml/jaxp/functional/catalog/DeferFeatureTest.java	Wed Nov 23 16:16:35 2016 +0000
+++ b/test/javax/xml/jaxp/functional/catalog/DeferFeatureTest.java	Thu Dec 01 08:57:38 2016 +0000
@@ -45,6 +45,7 @@
  * @test
  * @bug 8077931
  * @library /javax/xml/jaxp/libs
+ * @modules java.xml/javax.xml.catalog:open
  * @run testng/othervm -DrunSecMngr=true catalog.DeferFeatureTest
  * @run testng/othervm catalog.DeferFeatureTest
  * @summary This case tests whether the catalogs specified in delegateSystem,
--- a/test/javax/xml/jaxp/module/ServiceProviderTest/src/unnamed/Main.java	Wed Nov 23 16:16:35 2016 +0000
+++ b/test/javax/xml/jaxp/module/ServiceProviderTest/src/unnamed/Main.java	Thu Dec 01 08:57:38 2016 +0000
@@ -23,6 +23,7 @@
 
 import static javax.xml.XMLConstants.W3C_XML_SCHEMA_NS_URI;
 
+import java.lang.module.ModuleDescriptor.Provides;
 import java.lang.reflect.Layer;
 import java.lang.reflect.Module;
 import java.util.Arrays;
@@ -48,7 +49,8 @@
                 .map(xmlProviderName -> Layer.boot().findModule(xmlProviderName).get())
                 .mapToLong(
                         // services provided by the implementation in provider module
-                        provider -> provider.getDescriptor().provides().keySet().stream()
+                        provider -> provider.getDescriptor().provides().stream()
+                                .map(Provides::service)
                                 .filter(serviceName -> {
                                     allServices.remove(serviceName); // remove service provided by
                                                                      // customized module from allServices