RFR [15] 8245841: Remove unused com.sun.tools.javac.comp.Modules.XMODULES_PREFIX

Jonathan Gibbons jonathan.gibbons at oracle.com
Tue May 26 20:03:07 UTC 2020


+1

-- Jon

On 5/26/20 12:59 PM, Pavel Rappo wrote:
> Please review this trivial patch:
>
> 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
> --- 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
> @@ -215,8 +215,6 @@
>           limitModsOpt = options.get(Option.LIMIT_MODULES);
>           moduleVersionOpt = options.get(Option.MODULE_VERSION);
>       }
> -    //where
> -        private static final String XMODULES_PREFIX = "-Xmodule:";
>   
>       int depth = -1;
>
> My IDE shows that this field is unused and the tests on our CI system agree with it. The field is probably a leftover from "8182162: Remove -XD-Xmodule".
>
> -Pavel
>
>


More information about the compiler-dev mailing list