hg: jigsaw/jake/jdk: Revert provides change until javadoc supports it.
Changeset: cc82cebf8689 Author:mchung Date: 2016-11-17 22:58 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/cc82cebf8689 Revert provides change until javadoc supports it. ! src/java.desktop/share/classes/module-info.java ! src/jdk.jdi/share/classes/module-info.java ! src/jdk.jlink/share/classes/module-info.java ! src/jdk.jvmstat/share/classes/module-info.java ! src/jdk.localedata/share/classes/module-info.java ! src/jdk.security.auth/share/classes/module-info.java
hg: jigsaw/jake/jaxws: Merge multiple provides of the same service type
Changeset: 77a73360ee6b Author:mchung Date: 2016-11-17 21:09 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/77a73360ee6b Merge multiple provides of the same service type ! src/jdk.xml.bind/share/classes/module-info.java ! src/jdk.xml.ws/share/classes/module-info.java
hg: jigsaw/jake/jdk: Merge multiple provides of the same service type
Changeset: e2110950087f Author:mchung Date: 2016-11-17 22:06 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/e2110950087f Merge multiple provides of the same service type ! src/java.desktop/share/classes/module-info.java ! src/jdk.jdi/share/classes/module-info.java ! src/jdk.jlink/share/classes/module-info.java ! src/jdk.jvmstat/share/classes/module-info.java ! src/jdk.localedata/share/classes/module-info.java ! src/jdk.security.auth/share/classes/module-info.java
hg: jigsaw/jake/hotspot: Merge multiple provides of the same service type
Changeset: 1273e7cc048b Author:mchung Date: 2016-11-17 21:10 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/1273e7cc048b Merge multiple provides of the same service type ! src/jdk.vm.ci/share/classes/module-info.java
hg: jigsaw/jake: 4 new changesets
Changeset: 3b59ad58041d Author:simonis Date: 2016-11-14 11:48 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/3b59ad58041d 8169630: Fix wrong cpu build flag for Linux/ppc64le build Reviewed-by: erikj ! common/autoconf/flags.m4 ! common/autoconf/generated-configure.sh Changeset: ff98aa9ec9fa Author:erikj Date: 2016-11-15 10:00 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/ff98aa9ec9fa 8169632: Update compare script for clean compare Reviewed-by: tbell ! common/bin/compare.sh ! common/bin/compare_exceptions.sh.incl Changeset: b6c985a906c9 Author:lana Date: 2016-11-17 17:50 + URL: http://hg.openjdk.java.net/jigsaw/jake/rev/b6c985a906c9 Added tag jdk-9+145 for changeset ff98aa9ec9fa ! .hgtags Changeset: 6e2540d701ac Author:mchung Date: 2016-11-17 18:04 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/rev/6e2540d701ac Merge ! common/autoconf/flags.m4 ! common/autoconf/generated-configure.sh ! common/bin/compare.sh
hg: jigsaw/jake/corba: 2 new changesets
Changeset: ecd74b41ab65 Author:lana Date: 2016-11-17 17:50 + URL: http://hg.openjdk.java.net/jigsaw/jake/corba/rev/ecd74b41ab65 Added tag jdk-9+145 for changeset a44b156ae7f0 ! .hgtags Changeset: be54656af21d Author:mchung Date: 2016-11-17 18:04 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/corba/rev/be54656af21d Merge
hg: jigsaw/jake/jaxws: 2 new changesets
Changeset: e997d835cfee Author:lana Date: 2016-11-17 17:50 + URL: http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/e997d835cfee Added tag jdk-9+145 for changeset 29277a4e7307 ! .hgtags Changeset: 8e8d9cd2f887 Author:mchung Date: 2016-11-17 18:04 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/8e8d9cd2f887 Merge
hg: jigsaw/jake/jdk: 26 new changesets
Changeset: 762dad364838 Author:jlaskey Date: 2016-11-07 14:36 -0400 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/762dad364838 8155756: Better context for some jlink exceptions Reviewed-by: mchung ! src/jdk.jlink/share/classes/jdk/tools/jlink/internal/DirArchive.java ! src/jdk.jlink/share/classes/jdk/tools/jlink/internal/TaskHelper.java ! src/jdk.jlink/share/classes/jdk/tools/jlink/resources/jlink.properties Changeset: 361fb5404741 Author:xuelei Date: 2016-11-08 06:22 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/361fb5404741 8169318: Dump the reproduced packet in DTLSOverDatagram.java Reviewed-by: mullan ! test/javax/net/ssl/DTLS/DTLSOverDatagram.java Changeset: 60837db5d445 Author:nishjain Date: 2016-11-08 14:48 +0530 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/60837db5d445 8166994: Improve sun.util.locale.LocaleMatcher Reviewed-by: okutsu, naoto, peytoia ! src/java.base/share/classes/sun/util/locale/LocaleMatcher.java + test/java/util/Locale/Bug8166994.java Changeset: 3f560167843d Author:redestad Date: 2016-11-08 13:23 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/3f560167843d 8168911: Increased number of classes initialized during initialization of SignatureFileVerifier Reviewed-by: ascarpino ! src/java.base/share/classes/sun/security/util/AbstractAlgorithmConstraints.java ! src/java.base/share/classes/sun/security/util/AlgorithmDecomposer.java ! src/java.base/share/classes/sun/security/util/DisabledAlgorithmConstraints.java Changeset: befc4757d253 Author:psandoz Date: 2016-11-08 12:36 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/befc4757d253 8165782: java.util.jar.JarFile.runtimeVersion() spec needs clarification Reviewed-by: mchung, alanb ! src/java.base/share/classes/java/util/jar/JarFile.java Changeset: a641b410ee9b Author:psandoz Date: 2016-11-08 12:36 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/a641b410ee9b 8166735: JarFile#getVersion spec clarification for unversioned jars Reviewed-by: mchung, alanb ! src/java.base/share/classes/java/util/jar/JarFile.java Changeset: 36cf65a9e2f9 Author:sherman Date: 2016-11-08 14:29 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/36cf65a9e2f9 8168862: Tighten permissions granted to the jdk.zipfs module Reviewed-by: mullan, mchung, lancea, bpb ! src/java.base/share/lib/security/default.policy Changeset: 9bb8dc7e972d Author:asmotrak Date: 2016-11-08 15:55 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/9bb8dc7e972d 8168882: keytool doesn't print certificate info if disabled algorithm was used for signing a jar Reviewed-by: weijun, mullan ! src/java.base/share/classes/sun/security/tools/keytool/Main.java + test/lib/security/SecurityTools.java ! test/sun/security/tools/keytool/KeyToolTest.java ! test/sun/security/tools/keytool/PrintSSL.java + test/sun/security/tools/keytool/ReadJar.java - test/sun/security/tools/keytool/readjar.sh Changeset: 4407b0525631 Author:wetmore Date: 2016-11-08 16:54 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/4407b0525631 8169229: RSAClientKeyExchange debug info is incorrect Reviewed-by: xuelei ! src/java.base/share/classes/sun/security/ssl/RSAClientKeyExchange.java Changeset: 7f7091c1dd33 Author:rpatil Date: 2016-11-08 18:02 +0530 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/7f7091c1dd33 8169191: (tz) Support tzdata2016i Reviewed-by: martin, okutsu ! make/data/tzdata/VERSION ! make/data/tzdata/antarctica ! make/data/tzdata/asia ! make/data/tzdata/australasia ! make/data/tzdata/europe ! make/data/tzdata/zone.tab ! test/sun/util/calendar/zi/tzdata/VERSION ! test/sun/util/calendar/zi/tzdata/antarctica ! test/sun/util/calendar/zi/tzdata/asia ! test/sun/util/calendar/zi/tzdata/australasia ! test/sun/util/calendar/zi/tzdata/europe ! test/sun/util/calendar/zi/tzdata/zone.tab Changeset: a02d6daeba2a Author:psandoz Date: 2016-11-09 10:36 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/a02d6daeba2a 8164934: Optional.map() javadoc code example Reviewed-by: forax, chegar ! src/java.base/share/classes/java/util/Optional.java Changeset: 60d4214cc8ba Author:xuelei Date: 2016-11-10 08:59 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/60d4214cc8ba 8169362: Interop automated testing with Chrome Reviewed-by: wetmore, jnimeh, asmotrak + test/javax/net/ssl/interop/ClientHelloChromeInterOp.java + test/javax/net/ssl/interop/ClientHelloInterOp.java Changeset: df914079c901 Author:skovalev Date: 2016-11-09 14:24 +0300 URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/df914079c901 8169196: [TESTBUG] Three tests from sun/net/www have undeclared dependencies Reviewed-by: dfuchs ! test/sun/net/www/http/HttpClient/RetryPost.java - test/sun/net/www/http/HttpClient/RetryPost.sh !
hg: jigsaw/jake/langtools: 10 new changesets
Changeset: 6cc2220006f2 Author:lana Date: 2016-11-11 05:56 + URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/6cc2220006f2 Merge Changeset: eb48b040fbc2 Author:jlahoda Date: 2016-11-11 12:54 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/eb48b040fbc2 8145838: JShell: restrict RemoteAgent connection socket to localhost Summary: Also reviewed by Chris Ries Reviewed-by: rfield ! src/jdk.jshell/share/classes/jdk/jshell/execution/JdiDefaultExecutionControl.java ! src/jdk.jshell/share/classes/jdk/jshell/execution/RemoteExecutionControl.java Changeset: 4c650abde107 Author:jlahoda Date: 2016-11-11 17:45 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/4c650abde107 8169599: Several JShell tests are failing on Solaris after JDK-8145838 Summary: Partially reverting eb48b040fbc2 Reviewed-by: rfield ! src/jdk.jshell/share/classes/jdk/jshell/execution/JdiDefaultExecutionControl.java ! src/jdk.jshell/share/classes/jdk/jshell/execution/RemoteExecutionControl.java Changeset: c5d46cf53534 Author:rfield Date: 2016-11-11 11:50 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/c5d46cf53534 8166333: jshell tool: shortcut var does not import its type Reviewed-by: jlahoda ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/ConsoleIOContext.java Changeset: 768f685bceb7 Author:mchung Date: 2016-11-11 17:32 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/768f685bceb7 8168386: Fix jdeps verbose options Reviewed-by: dfuchs, lancea ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/ModuleAnalyzer.java ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/resources/jdeps.properties ! test/tools/jdeps/DotFileTest.java + test/tools/jdeps/Options.java ! test/tools/jdeps/lib/JdepsRunner.java Changeset: c18249ecb690 Author:mchung Date: 2016-11-12 09:26 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/c18249ecb690 8169606: jdeps --list-reduced-deps should not show java.base as all modules require it Reviewed-by: dfuchs ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsConfiguration.java ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/Module.java ! src/jdk.jdeps/share/classes/com/sun/tools/jdeps/ModuleExportsAnalyzer.java ! test/tools/jdeps/listdeps/ListModuleDeps.java + test/tools/jdeps/listdeps/src/hi/Hi.java Changeset: 120957324d6e Author:sadayapalam Date: 2016-11-14 19:45 +0530 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/120957324d6e 8168312: javac throws NPE if annotation processor is specified and module is declared in a file named arbitrarily Reviewed-by: jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symtab.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java + test/tools/javac/modules/AnnotationProcessingWithModuleInfoInWrongPlace.java Changeset: cb9e896265ef Author:ksrini Date: 2016-11-14 16:33 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/cb9e896265ef 8167967: javadoc should identify the ordinal value of enum constants Reviewed-by: jjg ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AbstractMemberWriter.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/ConfigurationImpl.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/ConstructorWriterImpl.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/AbstractMemberBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/AnnotationTypeFieldBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/AnnotationTypeRequiredMemberBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/ConstantsSummaryBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/ConstructorBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/EnumConstantBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/FieldBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/MemberSummaryBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/MethodBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/builders/PropertyBuilder.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Utils.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/VisibleMemberMap.java ! test/jdk/javadoc/doclet/testJavaFX/TestJavaFX.java ! test/jdk/javadoc/doclet/testOrdering/TestOrdering.java -
hg: jigsaw/jake/nashorn: 5 new changesets
Changeset: cb625e28d779 Author:attila Date: 2016-11-11 15:50 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/cb625e28d779 8168373: don't emit conversions for symbols outside their lexical scope Reviewed-by: hannesw, sundar ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/LocalVariableTypesCalculator.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/debug/PrintVisitor.java + test/script/basic/es6/JDK-8168373.js Changeset: ac5035ff7ee9 Author:hannesw Date: 2016-11-11 18:56 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/ac5035ff7ee9 8168049: Fix Performance of Lexer.isJSWhitespace Reviewed-by: sundar, attila ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/Lexer.java Changeset: 4a68dd740be8 Author:sdama Date: 2016-11-14 22:33 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/4a68dd740be8 8156615: Catch parameter can be a BindingPattern in ES6 mode Summary: Added parser support for catch parameter being a binding pattern Reviewed-by: sundar, hannesw, attila Contributed-by: srinivas.d...@oracle.com ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/api/tree/IRTranslator.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/AssignSymbols.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/CodeGenerator.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/LocalVariableTypesCalculator.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/Lower.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/CatchNode.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/Parser.java ! test/script/basic/es6/destructuring.js ! test/script/basic/es6/destructuring.js.EXPECTED + test/script/nosecurity/treeapi/destructuring_catch.js + test/script/nosecurity/treeapi/destructuring_catch.js.EXPECTED Changeset: f9d3c20b9585 Author:lana Date: 2016-11-17 17:50 + URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/f9d3c20b9585 Added tag jdk-9+145 for changeset 4a68dd740be8 ! .hgtags Changeset: e7c817adf098 Author:mchung Date: 2016-11-17 18:04 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/e7c817adf098 Merge ! .hgtags
hg: jigsaw/jake/jaxp: 2 new changesets
Changeset: 4f68472a3623 Author:lana Date: 2016-11-17 17:50 + URL: http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/4f68472a3623 Added tag jdk-9+145 for changeset 71558b38bad7 ! .hgtags Changeset: dac05614acb1 Author:mchung Date: 2016-11-17 18:04 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/dac05614acb1 Merge
hg: jigsaw/jake/nashorn: 8169920: Fix Nashorn JTReg Jake test failures
Changeset: 51851824d373 Author:hseigel Date: 2016-11-17 16:12 -0500 URL: http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/51851824d373 8169920: Fix Nashorn JTReg Jake test failures Summary: add needed :open to @modules Reviewed-by: alanb ! test/src/jdk/nashorn/internal/runtime/doubleconv/test/BignumTest.java ! test/src/jdk/nashorn/internal/runtime/doubleconv/test/DiyFpTest.java
hg: jigsaw/jake/hotspot: Clean-up of JVMTI AddModuleExports/AddModuleOpens
Changeset: e9f9a8d55756 Author:sspitsyn Date: 2016-11-17 10:29 + URL: http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/e9f9a8d55756 Clean-up of JVMTI AddModuleExports/AddModuleOpens ! make/test/JtregNative.gmk ! src/share/vm/prims/jvmtiEnv.cpp ! src/share/vm/prims/jvmtiExport.cpp ! src/share/vm/prims/jvmtiExport.hpp - test/serviceability/jvmti/AddModuleExports/MyPackage/AddModuleExportsTest.java - test/serviceability/jvmti/AddModuleExports/libAddModuleExportsTest.c + test/serviceability/jvmti/AddModuleExportsAndOpens/MyPackage/AddModuleExportsAndOpensTest.java + test/serviceability/jvmti/AddModuleExportsAndOpens/libAddModuleExportsAndOpensTest.c
Re: uses is useless ?
On 16/11/2016 19:41, fo...@univ-mlv.fr wrote: : My point is that in teory, that's cool but in reality the module-info of java.base (or java.sql) declares a lot of 'uses' making the whole idea moot. To avoid java.base to use 'uses', we need a way to be able to have a way to delegate the power of the ServiceLoader to another code, so the other code will have to use 'uses'. If a consumer is making use of an API in java.base and the implementation of that API makes use of services under the covers then it is transparent to the consumer of the API. The consumer of the API shouldn't need to know anything about the "SPI" types and would be very surprising if the user of an API needed to declare `uses` on such types. In the JDK then I'm only aware of two cases where the consumer passes a service type to an API that doesn't know anything about the service type (the two cases are JVMCI and a utility method in javac). Both of these cases use addUses to get through the hygiene check. jlink is a bit special in that the user of the tool needs at least some global knowledge when choosing the service provider modules to include in the run time image. As I said, the tool could help more but having it do eager service binding is problematic as it tends to maximize rather than minimize the number of the modules. -Alan.
