Attachment 215728 Details for Bug 375409 – proposed fix v1.0 + regression tests

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] proposed fix v1.0 + regression tests

Fixed-bug-375409-Add-missing-Javadoc-compile-options.patch (text/plain), 22.29 KB, created by Ayushman Jain

on 2012-05-16 14:25:57 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ayushman Jain

Created: 2012-05-16 14:25:57 EDT

Size: 22.29 KB

patch

obsolete

>diff --git a/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java b/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java
>index 971b03e..84f4c17 100644
>--- a/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java
>+++ b/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java
>@@ -81,7 +81,7 @@
> 			"}\n";
> 
> 	static {
>-//		TESTS_NAMES = new String[] { "test315_warn_options_a" };
>+//		TESTS_NAMES = new String[] { "testBug375409" };
> //		TESTS_NUMBERS = new int[] { 306 };
> //		TESTS_RANGE = new int[] { 298, -1 };
> 	}
>@@ -1734,15 +1734,32 @@
>         "      intfAnnotation     + annotation type used as super interface\n" + 
>         "      intfNonInherited   + interface non-inherited method compatibility\n" + 
>         "      intfRedundant        find redundant superinterfaces\n" + 
>+        "      invalidJavadoc       all warnings for malformed javadoc tags\n" + 
>+        "      invalidJavadocTag    validate javadoc tag arguments\n" + 
>+        "      invalidJavadocTagDep validate deprecated references in javadoc tag args\n" + 
>+        "      invalidJavadocTagNotVisible  validate non-visible references in javadoc\n" + 
>+        "							tag args\n" + 
>+        "      invalidJavadocVisibility(<visibility>)  specify visibility modifier\n" + 
>+        "							for malformed javadoc tag warnings\n" + 
>         "      javadoc              invalid javadoc\n" + 
>         "      localHiding          local variable hiding another variable\n" + 
>         "      maskedCatchBlock   + hidden catch block\n" + 
>+        "      missingJavadocTags   missing Javadoc tags\n" + 
>+        "      missingJavadocTagsOverriding missing Javadoc tags in overriding methods\n" + 
>+        "      missingJavadocTagsMethod missing Javadoc tags for method type parameter\n" + 
>+        "      missingJavadocTagsVisibility(<visibility>)  specify visibility modifier\n" + 
>+        "							for missing javadoc tags warnings\n" + 
>+        "      missingJavadocComments   missing Javadoc comments\n" + 
>+        "      missingJavadocCommentsOverriding   missing Javadoc tags in overriding\n" + 
>+        "							methods\n" + 
>+        "      missingJavadocCommentsVisibility(<visibility>)  specify visibility\n" + 
>+        "							modifier for missing javadoc comments warnings\n" + 
>         "      nls                  string literal lacking non-nls tag //$NON-NLS-<n>$\n" + 
>         "      noEffectAssign     + assignment without effect\n" + 
>         "      null                 potential missing or redundant null check\n" + 
>-        "      nullAnnot(<annot. names separated by |>) + annotation based null analysis,\n" +
>-        "                           nullable|nonnull|nonnullbydefault annotation types\n" +
>-        "                           optionally specified using fully qualified names\n" +
>+        "      nullAnnot(<annot. names separated by |>) + annotation based null analysis,\n" + 
>+        "                           nullable|nonnull|nonnullbydefault annotation types\n" + 
>+        "                           optionally specified using fully qualified names\n" + 
>         "      nullDereference    + missing null check\n" + 
>         "      over-ann             missing @Override annotation (superclass)\n" + 
>         "      paramAssign          assignment to a parameter\n" + 
>@@ -1758,7 +1775,7 @@
>         "      super                overriding a method without making a super invocation\n" + 
>         "      suppress           + enable @SuppressWarnings\n" + 
>         "                           When used with -err:, it can also silent optional\n" + 
>-        "                           errors and warnings\n" +
>+        "                           errors and warnings\n" + 
>         "      switchDefault      + switch statement lacking a default case\n" + 
>         "      syncOverride         missing synchronized in synchr. method override\n" + 
>         "      syntheticAccess      synthetic access for innerclass\n" + 
>@@ -1777,8 +1794,17 @@
>         "      unusedImport       + unused import declaration\n" + 
>         "      unusedLabel        + unused label\n" + 
>         "      unusedLocal        + unread local variable\n" + 
>+        "      unusedParam		    unused parameter\n" + 
>+        "      unusedParamOverriding unused parameter for overriding method\n" + 
>+        "      unusedParamImplementing unused parameter for implementing method\n" + 
>+        "      unusedParamIncludeDoc unused parameter documented in comment tag\n" + 
>         "      unusedPrivate      + unused private member declaration\n" + 
>         "      unusedThrown         unused declared thrown exception\n" + 
>+        "      unusedThrown         unused declared thrown exception, overriding method\n" + 
>+        "      unusedThrownIncludeDocComment     unused declared thrown exception,\n" + 
>+        "							documented in a comment tag\n" + 
>+        "      unusedThrownExemptExceptionThrowable  unused declared thrown exception,\n" + 
>+        "							exempt Exception and Throwable\n" + 
>         "      unusedTypeArgs     + unused type arguments for method and constructor\n" + 
>         "      uselessTypeCheck     unnecessary cast/instanceof operation\n" + 
>         "      varargsCast        + varargs argument need explicit cast\n" + 
>@@ -13178,4 +13204,128 @@
> 			"1 problem (1 warning)",
> 			true);
> }
>+
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=375409
>+public void testBug375409a() {
>+	this.runConformTest(
>+		new String[] {
>+				"p/X.java",
>+				"package p;\n" +
>+				"public class X {\n" +
>+				"/** \n" +
>+				"* Description {@see String}, {@category cat}\n" +
>+				"* @param a\n" + 
>+				"*/\n" +
>+				"public void foo(int i) {}}\n" 
>+		},
>+		"\"" + OUTPUT_DIR +  File.separator + "p" + File.separator + "X.java\""
>+		+ " -sourcepath \"" + OUTPUT_DIR + "\""
>+		+ " -1.5"
>+		+ " -warn:invalidJavadoc -proc:none -d \"" + OUTPUT_DIR + "\"",
>+		"",
>+		"----------\n" + 
>+		"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/p/X.java (at line 4)\n" + 
>+		"	* Description {@see String}, {@category cat}\n" + 
>+		"	                ^^^\n" + 
>+		"Javadoc: Unexpected tag\n" + 
>+		"----------\n" + 
>+		"2. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/p/X.java (at line 4)\n" + 
>+		"	* Description {@see String}, {@category cat}\n" + 
>+		"	                               ^^^^^^^^\n" + 
>+		"Javadoc: Unexpected tag\n" + 
>+		"----------\n" + 
>+		"3. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/p/X.java (at line 5)\n" + 
>+		"	* @param a\n" + 
>+		"	         ^\n" + 
>+		"Javadoc: Parameter a is not declared\n" + 
>+		"----------\n" + 
>+		"3 problems (3 warnings)",
>+		true);
>+}
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=375409
>+public void testBug375409b() {
>+	this.runConformTest(
>+		new String[] {
>+				"p/X.java",
>+				"package p;\n" +
>+				"public class X {\n" +
>+				"/** \n" +
>+				"* Description {@see String}, {@category cat}\n" +
>+				"* @param a\n" + 
>+				"*/\n" +
>+				"public void foo(int i) {}}\n" 
>+		},
>+		"\"" + OUTPUT_DIR +  File.separator + "p" + File.separator + "X.java\""
>+		+ " -sourcepath \"" + OUTPUT_DIR + "\""
>+		+ " -1.5"
>+		+ " -warn:missingJavadocTags,missingJavadocTagsVisibility(public) -proc:none -d \"" + OUTPUT_DIR + "\"",
>+		"",
>+		"----------\n" + 
>+		"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/p/X.java (at line 7)\n" + 
>+		"	public void foo(int i) {}}\n" + 
>+		"	                    ^\n" + 
>+		"Javadoc: Missing tag for parameter i\n" + 
>+		"----------\n" + 
>+		"1 problem (1 warning)",
>+		true);
>+}
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=375409
>+public void testBug375409c() {
>+	this.runConformTest(
>+		new String[] {
>+				"p/X.java",
>+				"package p;\n" +
>+				"public class X {\n" +
>+				"/** \n" +
>+				"* Description {@see String}, {@category cat}\n" +
>+				"* @param a\n" + 
>+				"*/\n" +
>+				"public void foo(int i) {}}\n" 
>+		},
>+		"\"" + OUTPUT_DIR +  File.separator + "p" + File.separator + "X.java\""
>+		+ " -sourcepath \"" + OUTPUT_DIR + "\""
>+		+ " -1.5"
>+		+ " -warn:missingJavadocComments -proc:none -d \"" + OUTPUT_DIR + "\"",
>+		"",
>+		"----------\n" + 
>+		"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/p/X.java (at line 2)\n" + 
>+		"	public class X {\n" + 
>+		"	             ^\n" + 
>+		"Javadoc: Missing comment for public declaration\n" + 
>+		"----------\n" + 
>+		"1 problem (1 warning)",
>+		true);
>+}
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=375409
>+public void testBug375409d() {
>+	this.runConformTest(
>+		new String[] {
>+				"p/X.java",
>+				"package p;\n" +
>+				"public class X {\n" +
>+				"/** \n" +
>+				"* Description {@see String}, {@category cat}\n" +
>+				"* @param a\n" + 
>+				"*/\n" +
>+				"void foo(int i) {}\n" +
>+				"/** \n" +
>+				"* Description {@see String}, {@category cat}\n" +
>+				"* @param a\n" + 
>+				"*/\n" +
>+				"public void foo2(int i2) {}}\n" 
>+		},
>+		"\"" + OUTPUT_DIR +  File.separator + "p" + File.separator + "X.java\""
>+		+ " -sourcepath \"" + OUTPUT_DIR + "\""
>+		+ " -1.5"
>+		+ " -warn:missingJavadocTags,missingJavadocTagsVisibility(public) -proc:none -d \"" + OUTPUT_DIR + "\"",
>+		"",
>+		"----------\n" + 
>+		"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/p/X.java (at line 12)\n" + 
>+		"	public void foo2(int i2) {}}\n" + 
>+		"	                     ^^\n" + 
>+		"Javadoc: Missing tag for parameter i2\n" + 
>+		"----------\n" + 
>+		"1 problem (1 warning)",
>+		true);
>+}
> }
>diff --git a/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/Main.java b/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/Main.java
>index 8fb5ce1..a77d580 100644
>--- a/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/Main.java
>+++ b/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/Main.java
>@@ -3446,7 +3446,60 @@
> 						CompilerOptions.OPTION_IncludeNullInfoFromAsserts,
> 						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
> 				return;
>-			} 
>+			} else if (token.equals("invalidJavadoc")) { //$NON-NLS-1$
>+				setSeverity(CompilerOptions.OPTION_ReportInvalidJavadoc, severity, isEnabling);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportInvalidJavadocTags,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportInvalidJavadocTagsDeprecatedRef,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportInvalidJavadocTagsNotVisibleRef,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				if (isEnabling) {
>+					this.options.put(
>+							CompilerOptions.OPTION_DocCommentSupport,
>+							CompilerOptions.ENABLED);
>+					this.options.put(
>+						CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility,
>+						CompilerOptions.PRIVATE);
>+				}
>+				return;
>+			} else if (token.equals("invalidJavadocTag")) { //$NON-NLS-1$
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportInvalidJavadocTags,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("invalidJavadocTagDep")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportInvalidJavadocTagsDeprecatedRef,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("invalidJavadocTagNotVisible")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportInvalidJavadocTagsNotVisibleRef,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.startsWith("invalidJavadocTagVisibility")) { //$NON-NLS-1$
>+				int start = token.indexOf('(');
>+				int end = token.indexOf(')');
>+				String visibility = null;
>+				if (isEnabling && start >= 0 && end >= 0 && start < end){
>+					visibility = token.substring(start+1, end).trim();
>+				}
>+				if (visibility != null && visibility.equals(CompilerOptions.PUBLIC)
>+						|| visibility.equals(CompilerOptions.PRIVATE)
>+						|| visibility.equals(CompilerOptions.PROTECTED)
>+						|| visibility.equals(CompilerOptions.DEFAULT)) {
>+					this.options.put(
>+							CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility,
>+							visibility);
>+					return;
>+				} else {
>+					throw new IllegalArgumentException(this.bind("configure.invalidJavadocTagVisibility", token)); //$NON-NLS-1$
>+				}
>+			}
> 			break;
> 		case 'j' :
> 			if (token.equals("javadoc")) {//$NON-NLS-1$
>@@ -3466,6 +3519,89 @@
> 			if (token.equals("maskedCatchBlock") || token.equals("maskedCatchBlocks")/*backward compatible*/) { //$NON-NLS-1$ //$NON-NLS-2$
> 				setSeverity(CompilerOptions.OPTION_ReportHiddenCatchBlock, severity, isEnabling);
> 				return;
>+			} else if (token.equals("missingJavadocTags")) { //$NON-NLS-1$
>+				setSeverity(CompilerOptions.OPTION_ReportMissingJavadocTags, severity, isEnabling);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportMissingJavadocTagsOverriding,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportMissingJavadocTagsMethodTypeParameters,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				if (isEnabling) {
>+					this.options.put(
>+							CompilerOptions.OPTION_DocCommentSupport,
>+							CompilerOptions.ENABLED);
>+					this.options.put(
>+						CompilerOptions.OPTION_ReportMissingJavadocTagsVisibility,
>+						CompilerOptions.PRIVATE);
>+				}
>+				return;
>+			} else if (token.equals("missingJavadocTagsOverriding")) { //$NON-NLS-1$
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportMissingJavadocTagsOverriding,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("missingJavadocTagsMethod")) { //$NON-NLS-1$
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportMissingJavadocTagsMethodTypeParameters,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.startsWith("missingJavadocTagsVisibility")) { //$NON-NLS-1$
>+				int start = token.indexOf('(');
>+				int end = token.indexOf(')');
>+				String visibility = null;
>+				if (isEnabling && start >= 0 && end >= 0 && start < end){
>+					visibility = token.substring(start+1, end).trim();
>+				}
>+				if (visibility != null && visibility.equals(CompilerOptions.PUBLIC)
>+						|| visibility.equals(CompilerOptions.PRIVATE)
>+						|| visibility.equals(CompilerOptions.PROTECTED)
>+						|| visibility.equals(CompilerOptions.DEFAULT)) {
>+					this.options.put(
>+							CompilerOptions.OPTION_ReportMissingJavadocTagsVisibility,
>+							visibility);
>+					return;
>+				} else {
>+					throw new IllegalArgumentException(this.bind("configure.missingJavadocTagsVisibility", token)); //$NON-NLS-1$
>+				}
>+			} else if (token.equals("missingJavadocComments")) { //$NON-NLS-1$
>+				setSeverity(CompilerOptions.OPTION_ReportMissingJavadocComments, severity, isEnabling);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportMissingJavadocCommentsOverriding,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				if (isEnabling) {
>+					this.options.put(
>+							CompilerOptions.OPTION_DocCommentSupport,
>+							CompilerOptions.ENABLED);
>+					this.options.put(
>+						CompilerOptions.OPTION_ReportMissingJavadocCommentsVisibility,
>+						CompilerOptions.PRIVATE);
>+				}
>+				return;
>+			} else if (token.equals("missingJavadocCommentsOverriding")) { //$NON-NLS-1$
>+				setSeverity(CompilerOptions.OPTION_ReportMissingJavadocComments, severity, isEnabling);
>+				this.options.put(
>+					CompilerOptions.OPTION_ReportMissingJavadocCommentsOverriding,
>+					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.startsWith("missingJavadocCommentsVisibility")) { //$NON-NLS-1$
>+				int start = token.indexOf('(');
>+				int end = token.indexOf(')');
>+				String visibility = null;
>+				if (isEnabling && start >= 0 && end >= 0 && start < end){
>+					visibility = token.substring(start+1, end).trim();
>+				}
>+				if (visibility != null && visibility.equals(CompilerOptions.PUBLIC)
>+						|| visibility.equals(CompilerOptions.PRIVATE)
>+						|| visibility.equals(CompilerOptions.PROTECTED)
>+						|| visibility.equals(CompilerOptions.DEFAULT)) {
>+					this.options.put(
>+							CompilerOptions.OPTION_ReportMissingJavadocCommentsVisibility,
>+							visibility);
>+					return;
>+				} else {
>+					throw new IllegalArgumentException(this.bind("configure.missingJavadocCommentsVisibility", token)); //$NON-NLS-1$
>+				}
> 			}
> 			break;
> 		case 'n' :
>@@ -3666,6 +3802,21 @@
> 			} else if (token.equals("unusedThrown")) { //$NON-NLS-1$
> 				setSeverity(CompilerOptions.OPTION_ReportUnusedDeclaredThrownException, severity, isEnabling);
> 				return;
>+			} else if (token.equals("unusedThrownWhenOverriding")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportUnusedDeclaredThrownExceptionWhenOverriding,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("unusedThrownIncludeDocComment")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportUnusedDeclaredThrownExceptionIncludeDocCommentReference,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("unusedThrownExemptExceptionThrowable")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportUnusedDeclaredThrownExceptionExemptExceptionAndThrowable,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
> 			} else if (token.equals("unqualifiedField") || token.equals("unqualified-field-access")) { //$NON-NLS-1$ //$NON-NLS-2$
> 				setSeverity(CompilerOptions.OPTION_ReportUnqualifiedFieldAccess, severity, isEnabling);
> 				return;
>@@ -3679,7 +3830,25 @@
> 				setSeverity(CompilerOptions.OPTION_ReportUnusedTypeArgumentsForMethodInvocation, severity, isEnabling);
> 				setSeverity(CompilerOptions.OPTION_ReportRedundantSpecificationOfTypeArguments, severity, isEnabling);
> 				return;
>-			} else if (token.equals("unusedTypeArgs")) { //$NON-NLS-1$
>+			} else if (token.equals("unusedParam")) { //$NON-NLS-1$
>+				setSeverity(CompilerOptions.OPTION_ReportUnusedParameter, severity, isEnabling);
>+				return;
>+			} else if (token.equals("unusedParamIncludeDoc")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportUnusedParameterIncludeDocCommentReference,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("unusedParamOverriding")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportUnusedParameterWhenOverridingConcrete,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			} else if (token.equals("unusedParamImplementing")) { //$NON-NLS-1$
>+				this.options.put(
>+						CompilerOptions.OPTION_ReportUnusedParameterWhenImplementingAbstract,
>+						isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
>+				return;
>+			}  else if (token.equals("unusedTypeArgs")) { //$NON-NLS-1$
> 				setSeverity(CompilerOptions.OPTION_ReportUnusedTypeArgumentsForMethodInvocation, severity, isEnabling);
> 				setSeverity(CompilerOptions.OPTION_ReportRedundantSpecificationOfTypeArguments, severity, isEnabling);
> 				return;
>diff --git a/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/messages.properties b/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/messages.properties
>index 5e8f78c..67f2f43 100644
>--- a/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/messages.properties
>+++ b/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/messages.properties
>@@ -320,10 +320,26 @@
> \      intfAnnotation     + annotation type used as super interface\n\
> \      intfNonInherited   + interface non-inherited method compatibility\n\
> \      intfRedundant        find redundant superinterfaces\n\
>+\      invalidJavadoc       all warnings for malformed javadoc tags\n\
>+\      invalidJavadocTag    validate javadoc tag arguments\n\
>+\      invalidJavadocTagDep validate deprecated references in javadoc tag args\n\
>+\      invalidJavadocTagNotVisible  validate non-visible references in javadoc\n\
>+\							tag args\n\
>+\      invalidJavadocVisibility(<visibility>)  specify visibility modifier\n\
>+\							for malformed javadoc tag warnings\n\
> \      javadoc              invalid javadoc\n\
> \      localHiding          local variable hiding another variable\n\
> \      maskedCatchBlock   + hidden catch block\n\
>-\	   missingJavadocTags   missing Javadoc tags\n\
>+\      missingJavadocTags   missing Javadoc tags\n\
>+\      missingJavadocTagsOverriding missing Javadoc tags in overriding methods\n\
>+\      missingJavadocTagsMethod missing Javadoc tags for method type parameter\n\
>+\      missingJavadocTagsVisibility(<visibility>)  specify visibility modifier\n\
>+\							for missing javadoc tags warnings\n\
>+\      missingJavadocComments   missing Javadoc comments\n\
>+\      missingJavadocCommentsOverriding   missing Javadoc tags in overriding\n\
>+\							methods\n\
>+\      missingJavadocCommentsVisibility(<visibility>)  specify visibility\n\
>+\							modifier for missing javadoc comments warnings\n\
> \      nls                  string literal lacking non-nls tag //$NON-NLS-<n>$\n\
> \      noEffectAssign     + assignment without effect\n\
> \      null                 potential missing or redundant null check\n\
>@@ -364,8 +380,17 @@
> \      unusedImport       + unused import declaration\n\
> \      unusedLabel        + unused label\n\
> \      unusedLocal        + unread local variable\n\
>+\      unusedParam		    unused parameter\n\
>+\      unusedParamOverriding unused parameter for overriding method\n\
>+\      unusedParamImplementing unused parameter for implementing method\n\
>+\      unusedParamIncludeDoc unused parameter documented in comment tag\n\
> \      unusedPrivate      + unused private member declaration\n\
> \      unusedThrown         unused declared thrown exception\n\
>+\      unusedThrown         unused declared thrown exception, overriding method\n\
>+\      unusedThrownIncludeDocComment     unused declared thrown exception,\n\
>+\							documented in a comment tag\n\
>+\      unusedThrownExemptExceptionThrowable  unused declared thrown exception,\n\
>+\							exempt Exception and Throwable\n\
> \      unusedTypeArgs     + unused type arguments for method and constructor\n\
> \      uselessTypeCheck     unnecessary cast/instanceof operation\n\
> \      varargsCast        + varargs argument need explicit cast\n\

Actions: View | Diff

Attachments on bug 375409: 215728