From d1e21b16c94faf9fe7bb210f58de36ea5b2e42c3 Mon Sep 17 00:00:00 2001 From: jlahoda Date: Mon, 19 Jun 2017 05:56:49 +0200 Subject: [PATCH] 8182162: Remove -XD-Xmodule Summary: Removing the undocumented -XD-Xmodule: option. Reviewed-by: jjg --- .../com/sun/tools/javac/comp/Modules.java | 22 +---- .../javac/modules/LegacyXModuleTest.java | 85 ------------------- 2 files changed, 2 insertions(+), 105 deletions(-) delete mode 100644 test/tools/javac/modules/LegacyXModuleTest.java diff --git a/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java b/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java index 32ef920cce..55118280e3 100644 --- a/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java +++ b/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Modules.java @@ -146,8 +146,6 @@ public class Modules extends JCTree.Visitor { public final boolean multiModuleMode; - private final String legacyModuleOverride; - private final Name java_se; private final Name java_; @@ -196,14 +194,6 @@ protected Modules(Context context) { allowAccessIntoSystem = options.isUnset(Option.RELEASE); lintOptions = options.isUnset(Option.XLINT_CUSTOM, "-" + LintCategory.OPTIONS.option); - Collection xmodules = options.keySet() - .stream() - .filter(opt -> opt.startsWith(XMODULES_PREFIX)) - .map(opt -> opt.substring(XMODULES_PREFIX.length())) - .collect(Collectors.toList()); - - legacyModuleOverride = xmodules.size() == 1 ? xmodules.iterator().next() : null; - multiModuleMode = fileManager.hasLocation(StandardLocation.MODULE_SOURCE_PATH); ClassWriter classWriter = ClassWriter.instance(context); classWriter.multiModuleMode = multiModuleMode; @@ -469,9 +459,6 @@ private void setCompilationUnitModules(List trees, Set trees, Set trees) { if (!fileManager.hasLocation(StandardLocation.PATCH_MODULE_PATH)) { - return legacyModuleOverride; + return null; } Set override = new LinkedHashSet<>(); @@ -602,7 +584,7 @@ private String singleModuleOverride(List trees) { } switch (override.size()) { - case 0: return legacyModuleOverride; + case 0: return null; case 1: return override.iterator().next(); default: log.error(Errors.TooManyPatchedModules(override)); diff --git a/test/tools/javac/modules/LegacyXModuleTest.java b/test/tools/javac/modules/LegacyXModuleTest.java deleted file mode 100644 index 36113364e0..0000000000 --- a/test/tools/javac/modules/LegacyXModuleTest.java +++ /dev/null @@ -1,85 +0,0 @@ -/* - * Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved. - * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. - * - * This code is free software; you can redistribute it and/or modify it - * under the terms of the GNU General Public License version 2 only, as - * published by the Free Software Foundation. - * - * This code is distributed in the hope that it will be useful, but WITHOUT - * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or - * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License - * version 2 for more details (a copy is included in the LICENSE file that - * accompanied this code). - * - * You should have received a copy of the GNU General Public License version - * 2 along with this work; if not, write to the Free Software Foundation, - * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. - * - * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA - * or visit www.oracle.com if you need additional information or have any - * questions. - */ - -/* - * @test - * @bug 8178012 - * @summary tests for multi-module mode compilation - * @library /tools/lib - * @modules - * jdk.compiler/com.sun.tools.javac.api - * jdk.compiler/com.sun.tools.javac.main - * @build toolbox.ToolBox toolbox.JavacTask toolbox.ModuleBuilder ModuleTestBase - * @run main LegacyXModuleTest - */ - -import java.nio.file.Path; -import java.util.Arrays; -import java.util.List; -import java.util.Objects; - -import toolbox.JavacTask; -import toolbox.Task; - -public class LegacyXModuleTest extends ModuleTestBase { - - public static void main(String... args) throws Exception { - new LegacyXModuleTest().runTests(); - } - - @Test - public void testLegacyXModule(Path base) throws Exception { - //note: avoiding use of java.base, as that gets special handling on some places: - Path src = base.resolve("src"); - tb.writeJavaFiles(src, "package com.sun.tools.javac.comp; public class Extra { Modules modules; }"); - Path classes = base.resolve("classes"); - tb.createDirectories(classes); - - new JavacTask(tb) - .options("-XD-Xmodule:jdk.compiler") - .outdir(classes) - .files(findJavaFiles(src)) - .run() - .writeAll() - .getOutput(Task.OutputKind.DIRECT); - - List log = new JavacTask(tb) - .options("-XD-Xmodule:java.compiler", - "-XD-Xmodule:jdk.compiler", - "-XDrawDiagnostics") - .outdir(classes) - .files(findJavaFiles(src)) - .run(Task.Expect.FAIL) - .writeAll() - .getOutputLines(Task.OutputKind.DIRECT); - - List actual = - Arrays.asList("Extra.java:1:56: compiler.err.cant.resolve.location: kindname.class, Modules, , , " + - "(compiler.misc.location: kindname.class, com.sun.tools.javac.comp.Extra, null)", - "1 error"); - - if (!Objects.equals(actual, log)) - throw new Exception("expected output not found: " + log); - } - -}