Skip to content

Commit 7a37816

Browse files
committed
8264866: Remove unneeded WorkArounds.isAutomaticModule
Reviewed-by: jjg
1 parent 51e8201 commit 7a37816

File tree

2 files changed

+1
-13
lines changed

2 files changed

+1
-13
lines changed

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/WorkArounds.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -199,18 +199,6 @@ public TypeElement searchClass(TypeElement klass, String className) {
199199
return elementUtils.getTypeElement(className);
200200
}
201201

202-
203-
// TODO: The following should be replaced by a new method such as Elements.isAutomaticModule
204-
// see JDK-8261625
205-
public boolean isAutomaticModule(ModuleElement me) {
206-
if (me == null) {
207-
return false;
208-
} else {
209-
ModuleSymbol msym = (ModuleSymbol) me;
210-
return (msym.flags() & Flags.AUTOMATIC_MODULE) != 0;
211-
}
212-
}
213-
214202
// TODO: need to re-implement this using j.l.m. correctly!, this has
215203
// implications on testInterface, the note here is that javac's supertype
216204
// does the right thing returning Parameters in scope.

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Extern.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -572,7 +572,7 @@ private String checkLinkCompatibility(String packageName, String moduleName, Str
572572
return DocletConstants.DEFAULT_ELEMENT_NAME;
573573
} else if (moduleName == null) {
574574
// suppress the warning message in the case of automatic modules
575-
if (!configuration.workArounds.isAutomaticModule(me) && issueWarning) {
575+
if (!utils.elementUtils.isAutomaticModule(me) && issueWarning) {
576576
configuration.getReporter().print(Kind.WARNING,
577577
resources.getText("doclet.linkMismatch_ModuleLinkedtoPackage", path));
578578
}

0 commit comments

Comments
 (0)