Download
Getting Started
Members
Projects
Community
Marketplace
Events
Planet Eclipse
Newsletter
Videos
Participate
Report a Bug
Forums
Mailing Lists
Wiki
IRC
How to Contribute
Working Groups
Automotive
Internet of Things
LocationTech
Long-Term Support
PolarSys
Science
OpenMDM
More
Community
Marketplace
Events
Planet Eclipse
Newsletter
Videos
Participate
Report a Bug
Forums
Mailing Lists
Wiki
IRC
How to Contribute
Working Groups
Automotive
Internet of Things
LocationTech
Long-Term Support
PolarSys
Science
OpenMDM
Toggle navigation
Bugzilla – Attachment 215728 Details for
Bug 375409
Add missing Javadoc compile options to batch compiler
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
Log In
[x]
|
Terms of Use
|
Copyright Agent
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:
proposed fix v1.0 + regression tests
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\
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 375409
: 215728