Changeset: d3dde3f775b8 Author: jlahoda Date: 2016-11-02 16:48 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/d3dde3f775b8
8166538: Improve error reporting for compiling against unexported package Summary: When a type cannot be found, look into other modules, search for possible viable types, and report them conveniently to the user. Reviewed-by: mcimadamore, jjg ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TypeEnter.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties ! test/tools/javac/PackageClassAmbiguity/util.out ! test/tools/javac/T5003235/T5003235a.java ! test/tools/javac/T5003235/T5003235a.out ! test/tools/javac/T5003235/T5003235b.java ! test/tools/javac/T5003235/T5003235b.out ! test/tools/javac/diags/Example.java ! test/tools/javac/diags/RunExamples.java ! test/tools/javac/diags/examples.not-yet.txt + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/NotDefAccessClassIntfCantAccessReason.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api1/api1/Api.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api1/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api2/api2/Api.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api2/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/impl/impl/Impl.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/impl/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/NotDefAccessClassIntfCantAccessReason.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api1/api1/Api.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api1/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api2/api2/Api.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api2/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/impl/impl/Impl.java + test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/impl/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/NotDefAccessClassPackageCantAccess.java - test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m1/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m1/p1/C1.java - test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m2/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m2/p2/C2.java + test/tools/javac/diags/examples/NotDefAccessDoesNotRead/NotDefAccessDoesNotRead.java + test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/api/api/Api.java + test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/api/module-info.java + test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/impl/impl/Impl.java + test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/impl/module-info.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadFromUnnamed/NotDefAccessDoesNotReadFromUnnamed.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadFromUnnamed/modulepath/api/api/Api.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadFromUnnamed/modulepath/api/module-info.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/NotDefAccessDoesNotReadUnnamed.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/classpath/api/Api.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/auxiliary/auxiliary/Auxiliary.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/auxiliary/module-info.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/impl/impl/Impl.java + test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/impl/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExported/NotDefAccessNotExported.java + test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/api/api/Api.java + test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/api/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/impl/impl/Impl.java + test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/impl/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExportedFromUnnamed/NotDefAccessNotExportedFromUnnamed.java + test/tools/javac/diags/examples/NotDefAccessNotExportedFromUnnamed/modulepath/api/api/Api.java + test/tools/javac/diags/examples/NotDefAccessNotExportedFromUnnamed/modulepath/api/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/NotDefAccessNotExportedToModule.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/api/api/Api.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/api/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/impl/impl/Impl.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/impl/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/other/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/NotDefAccessNotExportedToModuleFromUnnamed.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/modulepath/api/api/Api.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/modulepath/api/module-info.java + test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/modulepath/other/module-info.java ! test/tools/javac/modules/AddLimitMods.java ! test/tools/javac/modules/AddReadsTest.java ! test/tools/javac/modules/AutomaticModules.java + test/tools/javac/modules/ConvenientAccessErrorsTest.java ! test/tools/javac/modules/EdgeCases.java ! test/tools/javac/modules/GraphsTest.java ! test/tools/javac/modules/PackageMultipleModules.java ! test/tools/javac/modules/RequiresPublicTest.java ! test/tools/javac/modules/ResolveTest.java ! test/tools/javac/modules/UsesTest.java ! test/tools/javac/modules/XModuleTest.java Changeset: 1fc501869aa8 Author: jlahoda Date: 2016-11-02 20:21 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/1fc501869aa8 8169074: Build is failing after JDK-8166538 Summary: Reverting d3dde3f775b8 Reviewed-by: darcy ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TypeEnter.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties ! test/tools/javac/PackageClassAmbiguity/util.out ! test/tools/javac/T5003235/T5003235a.java ! test/tools/javac/T5003235/T5003235a.out ! test/tools/javac/T5003235/T5003235b.java ! test/tools/javac/T5003235/T5003235b.out ! test/tools/javac/diags/Example.java ! test/tools/javac/diags/RunExamples.java ! test/tools/javac/diags/examples.not-yet.txt - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/NotDefAccessClassIntfCantAccessReason.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api1/api1/Api.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api1/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api2/api2/Api.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/api2/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/impl/impl/Impl.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReason/modulesourcepath/impl/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/NotDefAccessClassIntfCantAccessReason.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api1/api1/Api.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api1/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api2/api2/Api.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/api2/module-info.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/impl/impl/Impl.java - test/tools/javac/diags/examples/NotDefAccessClassIntfCantAccessReasonFragment/modulesourcepath/impl/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/NotDefAccessClassPackageCantAccess.java + test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m1/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m1/p1/C1.java + test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m2/module-info.java + test/tools/javac/diags/examples/NotDefAccessClassPackageCantAccess/modulesourcepath/m2/p2/C2.java - test/tools/javac/diags/examples/NotDefAccessDoesNotRead/NotDefAccessDoesNotRead.java - test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/api/api/Api.java - test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/api/module-info.java - test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/impl/impl/Impl.java - test/tools/javac/diags/examples/NotDefAccessDoesNotRead/modulesourcepath/impl/module-info.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadFromUnnamed/NotDefAccessDoesNotReadFromUnnamed.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadFromUnnamed/modulepath/api/api/Api.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadFromUnnamed/modulepath/api/module-info.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/NotDefAccessDoesNotReadUnnamed.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/classpath/api/Api.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/auxiliary/auxiliary/Auxiliary.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/auxiliary/module-info.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/impl/impl/Impl.java - test/tools/javac/diags/examples/NotDefAccessDoesNotReadUnnamed/modulesourcepath/impl/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExported/NotDefAccessNotExported.java - test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/api/api/Api.java - test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/api/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/impl/impl/Impl.java - test/tools/javac/diags/examples/NotDefAccessNotExported/modulesourcepath/impl/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExportedFromUnnamed/NotDefAccessNotExportedFromUnnamed.java - test/tools/javac/diags/examples/NotDefAccessNotExportedFromUnnamed/modulepath/api/api/Api.java - test/tools/javac/diags/examples/NotDefAccessNotExportedFromUnnamed/modulepath/api/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/NotDefAccessNotExportedToModule.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/api/api/Api.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/api/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/impl/impl/Impl.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/impl/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModule/modulesourcepath/other/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/NotDefAccessNotExportedToModuleFromUnnamed.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/modulepath/api/api/Api.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/modulepath/api/module-info.java - test/tools/javac/diags/examples/NotDefAccessNotExportedToModuleFromUnnamed/modulepath/other/module-info.java ! test/tools/javac/modules/AddLimitMods.java ! test/tools/javac/modules/AddReadsTest.java ! test/tools/javac/modules/AutomaticModules.java - test/tools/javac/modules/ConvenientAccessErrorsTest.java ! test/tools/javac/modules/EdgeCases.java ! test/tools/javac/modules/GraphsTest.java ! test/tools/javac/modules/PackageMultipleModules.java ! test/tools/javac/modules/RequiresPublicTest.java ! test/tools/javac/modules/ResolveTest.java ! test/tools/javac/modules/UsesTest.java ! test/tools/javac/modules/XModuleTest.java Changeset: 035532fe695b Author: smarks Date: 2016-11-02 14:22 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/035532fe695b 8169078: add bug IDs to jdeprscan tests Reviewed-by: jjg ! test/tools/jdeprscan/tests/jdk/jdeprscan/TestLoad.java ! test/tools/jdeprscan/tests/jdk/jdeprscan/TestScan.java Changeset: afaafd7267cc Author: rfield Date: 2016-11-02 16:29 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/afaafd7267cc 8167636: jshell tool: Edit Pad should be in its own module Reviewed-by: jlahoda ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/ConsoleIOContext.java - src/jdk.jshell/share/classes/jdk/internal/jshell/tool/EditPad.java - src/jdk.jshell/share/classes/jdk/internal/jshell/tool/ExternalEditor.java ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/JShellTool.java ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/resources/l10n.properties ! src/jdk.jshell/share/classes/module-info.java - test/jdk/jshell/EditorPadTest.java Changeset: 692418f52877 Author: jlahoda Date: 2016-11-03 11:46 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/692418f52877 8166635: getEnclosedElements() on package causes BadClassFile error Summary: Properly reading typevars from enclosing scopes for local classes. Reviewed-by: mcimadamore ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassReader.java + test/tools/javac/processing/model/nestedTypeVars/NestedTypeVars.java + test/tools/javac/processing/model/nestedTypeVars/NestedTypeVars.out Changeset: 08e3ce8305c9 Author: ksrini Date: 2016-11-02 15:42 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/08e3ce8305c9 8166857: langtools build.xml broken on windows Reviewed-by: jjg, jlahoda, mcimadamore ! make/build.xml Changeset: 46f2219faf5a Author: rfield Date: 2016-11-03 09:12 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/46f2219faf5a 8161969: jshell tool: /var value is not truncated per feedback setting 8166637: jshell tool: confusing truncation of long result values 8154513: JShell tool: welcome message should match feedback mode 8167552: jshell tool: Typo in jshell command '/? /reload' description Reviewed-by: jlahoda ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/Feedback.java ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/JShellTool.java ! src/jdk.jshell/share/classes/jdk/internal/jshell/tool/resources/l10n.properties ! test/jdk/jshell/ToolFormatTest.java ! test/jdk/jshell/ToolRetainTest.java ! test/jdk/jshell/ToolSimpleTest.java Changeset: 80b576bd3631 Author: jjg Date: 2016-11-03 13:59 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/80b576bd3631 8167975: align javac --add-* modules options with launcher Reviewed-by: jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTaskImpl.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Arguments.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Option.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/javac.properties ! src/jdk.compiler/share/classes/com/sun/tools/javac/util/AbstractDiagnosticFormatter.java ! test/tools/doclint/moduleTests/bad/module-info.java ! test/tools/doclint/moduleTests/good/module-info.java ! test/tools/javac/T6403466.java + test/tools/javac/diags/examples/BadNameForOption.java - test/tools/javac/diags/examples/CantFindModule/CantFindModule.java + test/tools/javac/diags/examples/ModuleForOptionNotFound.java - test/tools/javac/diags/examples/XaddexportsMalformedEntry.java - test/tools/javac/diags/examples/XaddexportsTooMany.java - test/tools/javac/diags/examples/XaddreadsMalformedEntry.java - test/tools/javac/diags/examples/XaddreadsTooMany.java + test/tools/javac/modules/AddExportsTest.java + test/tools/javac/modules/AddModulesTest.java ! test/tools/javac/modules/AddReadsTest.java ! test/tools/javac/modules/AnnotationProcessorsInModulesTest.java ! test/tools/javac/modules/EdgeCases.java + test/tools/javac/modules/LimitModulesTest.java ! test/tools/javac/modules/ModuleTestBase.java ! test/tools/javac/modules/XModuleTest.java Changeset: 74a391527c2c Author: lana Date: 2016-11-04 17:52 +0000 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/74a391527c2c Merge - src/jdk.jshell/share/classes/jdk/internal/jshell/tool/EditPad.java - src/jdk.jshell/share/classes/jdk/internal/jshell/tool/ExternalEditor.java - test/jdk/jshell/EditorPadTest.java - test/tools/javac/diags/examples/CantFindModule/CantFindModule.java - test/tools/javac/diags/examples/XaddexportsMalformedEntry.java - test/tools/javac/diags/examples/XaddexportsTooMany.java - test/tools/javac/diags/examples/XaddreadsMalformedEntry.java - test/tools/javac/diags/examples/XaddreadsTooMany.java Changeset: 16f0cf12da2d Author: rfield Date: 2016-11-04 14:47 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/16f0cf12da2d 8129559: JShell: compilation fails if class, method or field is annotated and has modifiers 8080354: JShell: Runtime visible annotations cannot be retrieved Reviewed-by: jlahoda ! src/jdk.jshell/share/classes/jdk/jshell/MaskCommentsAndModifiers.java ! test/jdk/jshell/ClassMembersTest.java ! test/jdk/jshell/ClassesTest.java ! test/jdk/jshell/CompletenessTest.java ! test/jdk/jshell/IgnoreTest.java ! test/jdk/jshell/ModifiersTest.java Changeset: 5892a55c7bf9 Author: rfield Date: 2016-11-06 22:50 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/5892a55c7bf9 8161983: JShell API: Clean-up following 8160127 et. al. Reviewed-by: jlahoda ! src/jdk.jshell/share/classes/jdk/jshell/JShell.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIDefaultExecutionControl.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIEventHandler.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIExecutionControl.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIInitiator.java + src/jdk.jshell/share/classes/jdk/jshell/execution/JdiDefaultExecutionControl.java + src/jdk.jshell/share/classes/jdk/jshell/execution/JdiEventHandler.java + src/jdk.jshell/share/classes/jdk/jshell/execution/JdiExecutionControl.java + src/jdk.jshell/share/classes/jdk/jshell/execution/JdiInitiator.java ! src/jdk.jshell/share/classes/jdk/jshell/execution/RemoteExecutionControl.java ! src/jdk.jshell/share/classes/jdk/jshell/execution/Util.java ! src/jdk.jshell/share/classes/jdk/jshell/spi/ExecutionControl.java ! test/jdk/jshell/FailOverExecutionControlTest.java - test/jdk/jshell/JDILaunchingExecutionControlTest.java - test/jdk/jshell/JDIListeningExecutionControlTest.java - test/jdk/jshell/JDIListeningLocalhostExecutionControlTest.java + test/jdk/jshell/JdiLaunchingExecutionControlTest.java + test/jdk/jshell/JdiListeningExecutionControlTest.java + test/jdk/jshell/JdiListeningLocalhostExecutionControlTest.java - test/jdk/jshell/UserJDIUserRemoteTest.java + test/jdk/jshell/UserJdiUserRemoteTest.java Changeset: 82a50c8d9a43 Author: mcimadamore Date: 2016-11-07 11:16 +0000 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/82a50c8d9a43 8168854: javac erroneously reject a a service interface inner class in a provides clause Summary: Java SE 9 proposed spec changes do not explicitly forbid a service interface to be an inner class - javac should follow Reviewed-by: jjg ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties - test/tools/javac/diags/examples/ServiceDefinitionInner/ServiceDefinitionInner.java - test/tools/javac/diags/examples/ServiceDefinitionInner/modulesourcepath/m/module-info.java - test/tools/javac/diags/examples/ServiceDefinitionInner/modulesourcepath/m/p1/C1.java - test/tools/javac/diags/examples/ServiceDefinitionInner/modulesourcepath/m/p2/C2.java ! test/tools/javac/modules/ProvidesTest.java + test/tools/javac/modules/T8168854/module-info.java + test/tools/javac/modules/T8168854/pack1/Outer.java + test/tools/javac/modules/T8168854/pack1/Outer1.java Changeset: 47871e348144 Author: mcimadamore Date: 2016-11-07 16:08 +0000 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/47871e348144 8169093: Generics, javac not matching actual and formal arguments. Summary: Make diagnostic for type-args arity mismatch consistent with the class case Reviewed-by: vromero ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties + test/tools/javac/diags/examples/WrongNumberTypeArgsFragment.java Changeset: ed2e9ef4ce42 Author: ksrini Date: 2016-11-08 12:56 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/ed2e9ef4ce42 8166379: IAE while invoking javadoc with --patch-module Reviewed-by: jjg ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/Start.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/ToolOption.java ! test/jdk/javadoc/tool/modules/Modules.java Changeset: d4deb115da83 Author: jlahoda Date: 2016-11-10 17:27 +0100 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/d4deb115da83 8166700: NPE during invoking getEnclosedElements() on javax.lang.model.element.Element instance representing a package Summary: Should not strip first parameter of a local class constructor for local classes in initializers. Reviewed-by: jjg ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Scope.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassReader.java + test/tools/javac/processing/model/LocalClasses/LocalClassesModel.java + test/tools/javac/processing/model/LocalClasses/LocalClassesModel.out + test/tools/javac/processing/model/LocalClasses/LocalTest$1Local.jcod + test/tools/javac/processing/model/LocalClasses/LocalTest$2Local.jcod + test/tools/javac/processing/model/LocalClasses/LocalTest$3Local.jcod + test/tools/javac/processing/model/LocalClasses/LocalTest$4Local.jcod + test/tools/javac/processing/model/LocalClasses/LocalTest$5Local.jcod + test/tools/javac/processing/model/LocalClasses/LocalTest.jcod + test/tools/javac/processing/model/LocalClasses/LocalTest.orign Changeset: 6afd59d40256 Author: jjg Date: 2016-11-10 13:29 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/6afd59d40256 8169447: javac should detect/reject repeated use of --patch-module on command line Reviewed-by: jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Option.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/javac.properties ! test/tools/javac/modules/PatchModulesTest.java Changeset: cdb8c9dd402f Author: jjg Date: 2016-11-10 15:58 -0800 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/cdb8c9dd402f Merge ! make/build.xml ! src/jdk.compiler/share/classes/com/sun/tools/javac/api/JavacTaskImpl.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Scope.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TypeEnter.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassReader.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Arguments.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Option.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler.properties ! src/jdk.compiler/share/classes/com/sun/tools/javac/resources/javac.properties ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/Start.java ! src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/ToolOption.java - src/jdk.jshell/share/classes/jdk/internal/jshell/tool/EditPad.java - src/jdk.jshell/share/classes/jdk/internal/jshell/tool/ExternalEditor.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIDefaultExecutionControl.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIEventHandler.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIExecutionControl.java - src/jdk.jshell/share/classes/jdk/jshell/execution/JDIInitiator.java ! src/jdk.jshell/share/classes/module-info.java ! test/jdk/javadoc/tool/modules/Modules.java - test/jdk/jshell/EditorPadTest.java - test/jdk/jshell/JDILaunchingExecutionControlTest.java - test/jdk/jshell/JDIListeningExecutionControlTest.java - test/jdk/jshell/JDIListeningLocalhostExecutionControlTest.java ! test/jdk/jshell/ToolFormatTest.java - test/jdk/jshell/UserJDIUserRemoteTest.java ! test/tools/javac/T6403466.java ! test/tools/javac/diags/Example.java - test/tools/javac/diags/examples/CantFindModule/CantFindModule.java - test/tools/javac/diags/examples/ServiceDefinitionInner/ServiceDefinitionInner.java - test/tools/javac/diags/examples/ServiceDefinitionInner/modulesourcepath/m/module-info.java - test/tools/javac/diags/examples/ServiceDefinitionInner/modulesourcepath/m/p1/C1.java - test/tools/javac/diags/examples/ServiceDefinitionInner/modulesourcepath/m/p2/C2.java - test/tools/javac/diags/examples/XaddexportsMalformedEntry.java - test/tools/javac/diags/examples/XaddexportsTooMany.java - test/tools/javac/diags/examples/XaddreadsMalformedEntry.java - test/tools/javac/diags/examples/XaddreadsTooMany.java ! test/tools/javac/modules/AddLimitMods.java ! test/tools/javac/modules/AddReadsTest.java ! test/tools/javac/modules/AnnotationProcessorsInModulesTest.java ! test/tools/javac/modules/AutomaticModules.java ! test/tools/javac/modules/EdgeCases.java ! test/tools/javac/modules/GraphsTest.java ! test/tools/javac/modules/ModuleTestBase.java ! test/tools/javac/modules/PackageMultipleModules.java ! test/tools/javac/modules/PatchModulesTest.java ! test/tools/javac/modules/ProvidesTest.java ! test/tools/javac/modules/RequiresTransitiveTest.java ! test/tools/javac/modules/ResolveTest.java + test/tools/javac/modules/T8168854/pack1/Outer.java + test/tools/javac/modules/T8168854/pack1/Outer1.java ! test/tools/javac/modules/UsesTest.java ! test/tools/javac/modules/XModuleTest.java + test/tools/javac/processing/model/LocalClasses/LocalClassesModel.java