From fa421d165e46cfc3e9d2eea6ecfe821bcbbc58ed Mon Sep 17 00:00:00 2001
From: Skylot <skylot@gmail.com>
Date: Tue, 21 Dec 2021 12:52:52 +0000
Subject: [PATCH] build: disable missing warnings from javadoc

---
 buildSrc/src/main/groovy/jadx-library.gradle             | 2 ++
 jadx-core/src/main/java/jadx/core/deobf/NameMapper.java  | 4 ----
 .../src/main/java/jadx/core/dex/nodes/InsnNode.java      | 5 ++---
 .../src/main/java/jadx/core/dex/nodes/RootNode.java      | 9 +--------
 4 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/buildSrc/src/main/groovy/jadx-library.gradle b/buildSrc/src/main/groovy/jadx-library.gradle
index 161f9880..ed2dc98f 100644
--- a/buildSrc/src/main/groovy/jadx-library.gradle
+++ b/buildSrc/src/main/groovy/jadx-library.gradle
@@ -73,4 +73,6 @@ javadoc {
 	if (JavaVersion.current().isJava9Compatible()) {
 		options.addBooleanOption('html5', true)
 	}
+	// disable 'missing' warnings
+	options.addStringOption('Xdoclint:all,-missing', '-quiet')
 }
diff --git a/jadx-core/src/main/java/jadx/core/deobf/NameMapper.java b/jadx-core/src/main/java/jadx/core/deobf/NameMapper.java
index 104eaffa..ce10bd7f 100644
--- a/jadx-core/src/main/java/jadx/core/deobf/NameMapper.java
+++ b/jadx-core/src/main/java/jadx/core/deobf/NameMapper.java
@@ -143,19 +143,15 @@ public class NameMapper {
 
 	/**
 	 * Return modified string with removed:
-	 * <p>
 	 * <ul>
 	 * <li>not printable chars (including unicode)
 	 * <li>chars not valid for java identifier part
 	 * </ul>
-	 * <p>
 	 * Note: this 'middle' method must be used with prefixed string:
-	 * <p>
 	 * <ul>
 	 * <li>can leave invalid chars for java identifier start (i.e numbers)
 	 * <li>result not checked for reserved words
 	 * </ul>
-	 * <p>
 	 */
 	public static String removeInvalidCharsMiddle(String name) {
 		if (isValidIdentifier(name) && isAllCharsPrintable(name)) {
diff --git a/jadx-core/src/main/java/jadx/core/dex/nodes/InsnNode.java b/jadx-core/src/main/java/jadx/core/dex/nodes/InsnNode.java
index d68d3b40..1c4f12c9 100644
--- a/jadx-core/src/main/java/jadx/core/dex/nodes/InsnNode.java
+++ b/jadx-core/src/main/java/jadx/core/dex/nodes/InsnNode.java
@@ -385,17 +385,16 @@ public class InsnNode extends LineAttrNode {
 
 	/**
 	 * Make copy of InsnNode object.
-	 * <p>
+	 * <br>
 	 * NOTE: can't copy instruction with result argument
 	 * (SSA variable can't be used in two different assigns).
-	 * <p>
+	 * <br>
 	 * Prefer use next methods:
 	 * <ul>
 	 * <li>{@link #copyWithoutResult()} to explicitly state that result not needed
 	 * <li>{@link #copy(RegisterArg)} to provide new result arg
 	 * <li>{@link #copyWithNewSsaVar(MethodNode)} to make new SSA variable for result arg
 	 * </ul>
-	 * <p>
 	 */
 	public InsnNode copy() {
 		if (this.getClass() != InsnNode.class) {
diff --git a/jadx-core/src/main/java/jadx/core/dex/nodes/RootNode.java b/jadx-core/src/main/java/jadx/core/dex/nodes/RootNode.java
index 89009641..6b700f6d 100644
--- a/jadx-core/src/main/java/jadx/core/dex/nodes/RootNode.java
+++ b/jadx-core/src/main/java/jadx/core/dex/nodes/RootNode.java
@@ -336,12 +336,9 @@ public class RootNode {
 
 	/**
 	 * Searches for ClassNode by its full name (original or alias name)
-	 *
+	 * <br>
 	 * Warning: This method has a runtime of O(n) (n = number of classes).
 	 * If you need to call it more than once consider {@link #buildFullAliasClassCache()} instead
-	 *
-	 * @param fullName
-	 * @return
 	 */
 	@Nullable
 	public ClassNode searchClassByFullAlias(String fullName) {
@@ -355,10 +352,6 @@ public class RootNode {
 		return null;
 	}
 
-	/**
-	 *
-	 * @return
-	 */
 	public Map<String, ClassNode> buildFullAliasClassCache() {
 		Map<String, ClassNode> classNameCache = new HashMap<>(classes.size());
 		for (ClassNode cls : classes) {
-- 
GitLab