Attachment #111798 for bug #246066

View | Details | Raw Unified | Return to bug 246066
Collapse All | Expand All




        "      indirectStatic       indirect reference to static member\n" +
        "      intfAnnotation     + annotation type used as super interface\n" +
        "      intfNonInherited   + interface non-inherited method compatibility\n" +
        "      intfRedundant      + find redundant superinterfaces\n" + 
        "      javadoc              invalid javadoc\n" +
        "      localHiding          local variable hiding another variable\n" +
        "      maskedCatchBlock   + hidden catch block\n" +

        "      paramAssign          assignment to a parameter\n" +
        "      pkgDefaultMethod   + attempt to override package-default method\n" +
        "      raw                + usage of raw type\n" +
        "      redundantSuperinterface                + find redundant superinterfaces\n" +
        "      semicolon            unnecessary semicolon, empty statement\n" +
        "      serial             + missing serialVersionUID\n" +
        "      specialParamHiding   constructor or setter parameter hiding a field\n" +

        "      staticReceiver     + non-static reference to static member\n" +
        "      super                overriding a method without making a super invocation\n" +
        "      suppress           + enable @SuppressWarnings\n" +
        "      syncOverride         missing synchronized in synchr. method override\n" +
        "      synthetic-access     same as syntheticAccess\n" +
        "      syntheticAccess      synthetic access for innerclass\n" +
        "      tasks(<tags separated by |>) tasks identified by tags inside comments\n" +
        "      typeHiding         + type parameter hiding another type\n" +
        "      unchecked          + unchecked type operation\n" +
        "      unnecessaryElse      unnecessary else clause\n" +
        "      unqualified-field-access same as unqualifiedField\n" +
        "      unqualifiedField     unqualified reference to field\n" +
        "      unused               macro for unusedArgument, unusedImport, unusedLabel,\n" +
        "                               unusedLocal, unusedPrivate, unusedThrown,\n" +

        "Unrecognized option : @options2.txt\n" /* stderr */,
        true /*shouldFlushOutput*/);
}
//https://bugs.eclipse.org/bugs/show_bug.cgi?id=246066
public void test288_warn_options() {
	this.runConformTest(
		new String[] {
			"X.java",
			"interface IX {}\n" + 
			"class BaseX implements IX {}\n" + 
			"public class X extends BaseX implements IX {\n" + 
			"}\n",
		},
		"\"" + OUTPUT_DIR +  File.separator + "X.java\""
		+ " -sourcepath \"" + OUTPUT_DIR + "\""
		+ " -warn:+intfRedundant -proc:none -d \"" + OUTPUT_DIR + "\"",
		"",
		"----------\n" + 
		"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/X.java (at line 3)\n" + 
		"	public class X extends BaseX implements IX {\n" + 
		"	                                        ^^\n" + 
		"Redundant superinterface IX for the type X, already defined by BaseX\n" + 
		"----------\n" + 
		"1 problem (1 warning)",
		true);
}
//https://bugs.eclipse.org/bugs/show_bug.cgi?id=246066 - variation
public void test289_warn_options() {
	this.runConformTest(
		new String[] {
			"X.java",
			"interface IX {}\n" + 
			"class BaseX implements IX {}\n" + 
			"public class X extends BaseX implements IX {\n" + 
			"}\n",
		},
		"\"" + OUTPUT_DIR +  File.separator + "X.java\""
		+ " -sourcepath \"" + OUTPUT_DIR + "\""
		+ " -warn:+redundantSuperinterface -proc:none -d \"" + OUTPUT_DIR + "\"",
		"",
		"----------\n" + 
		"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/X.java (at line 3)\n" + 
		"	public class X extends BaseX implements IX {\n" + 
		"	                                        ^^\n" + 
		"Redundant superinterface IX for the type X, already defined by BaseX\n" + 
		"----------\n" + 
		"1 problem (1 warning)",
		true);
}
//https://bugs.eclipse.org/bugs/show_bug.cgi?id=246066 - variation
public void test290_warn_options() {
	this.runConformTest(
		new String[] {
			"X.java",
			"interface IX {}\n" + 
			"class BaseX implements IX {}\n" + 
			"public class X extends BaseX implements IX {\n" + 
			"}\n",
		},
		"\"" + OUTPUT_DIR +  File.separator + "X.java\""
		+ " -sourcepath \"" + OUTPUT_DIR + "\""
		+ " -warn:+intfRedundant -warn:-intfRedundant -proc:none -d \"" + OUTPUT_DIR + "\"",
		"",
		"",
		true);
}
}




\      indirectStatic       indirect reference to static member\n\
\      intfAnnotation     + annotation type used as super interface\n\
\      intfNonInherited   + interface non-inherited method compatibility\n\
\      intfRedundant      + find redundant superinterfaces\n\
\      javadoc              invalid javadoc\n\
\      localHiding          local variable hiding another variable\n\
\      maskedCatchBlock   + hidden catch block\n\

\      paramAssign          assignment to a parameter\n\
\      pkgDefaultMethod   + attempt to override package-default method\n\
\      raw                + usage of raw type\n\
\      redundantSuperinterface                + find redundant superinterfaces\n\
\      semicolon            unnecessary semicolon, empty statement\n\
\      serial             + missing serialVersionUID\n\
\      specialParamHiding   constructor or setter parameter hiding a field\n\

\      staticReceiver     + non-static reference to static member\n\
\      super                overriding a method without making a super invocation\n\
\      suppress           + enable @SuppressWarnings\n\
\      syncOverride         missing synchronized in synchr. method override\n\
\      synthetic-access     same as syntheticAccess\n\
\      syntheticAccess      synthetic access for innerclass\n\
\      tasks(<tags separated by |>) tasks identified by tags inside comments\n\
\      typeHiding         + type parameter hiding another type\n\
\      unchecked          + unchecked type operation\n\
\      unnecessaryElse      unnecessary else clause\n\
\      unqualified-field-access same as unqualifiedField\n\
\      unqualifiedField     unqualified reference to field\n\
\      unused               macro for unusedArgument, unusedImport, unusedLabel,\n\
\                               unusedLocal, unusedPrivate, unusedThrown,\n\





		// find out an option name controlling a given problemID
		private String getProblemOptionKey(int problemID) {
			int irritant = ProblemReporter.getIrritant(problemID);
			return CompilerOptions.optionKeyFromIrritant(irritant);
		}


 * Handle a single warning token.
*/
protected void handleWarningToken(String token, boolean isEnabling) throws InvalidInputException {
	if (token.length() == 0) return;
	switch(token.charAt(0)) {
		case 'a' :
			if (token.equals("allDeprecation")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportDeprecation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportDeprecationInDeprecatedCode,
					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
				this.options.put(
					CompilerOptions.OPTION_ReportDeprecationWhenOverridingDeprecatedMethod,
					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
				return;
			} else if (token.equals("allJavadoc")) { //$NON-NLS-1$
				this.warnAllJavadocOn = this.warnJavadocOn = isEnabling;
				return;
			} else if (token.equals("assertIdentifier")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportAssertIdentifier,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'b' :
			if (token.equals("boxing")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportAutoboxing,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'c' :
			if (token.equals("constructorName")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportMethodWithConstructorName,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("conditionAssign")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportPossibleAccidentalBooleanAssignment,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("compareIdentical")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportComparingIdentical,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("charConcat") /*|| token.equals("noImplicitStringConversion")/*backward compatible*/) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportNoImplicitStringConversion,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'd' :
			if (token.equals("deprecation")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportDeprecation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportDeprecationInDeprecatedCode,
					CompilerOptions.DISABLED);
				this.options.put(
					CompilerOptions.OPTION_ReportDeprecationWhenOverridingDeprecatedMethod,
					CompilerOptions.DISABLED);
				return;
			} else if (token.equals("dep-ann")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportMissingDeprecatedAnnotation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("discouraged")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportDiscouragedReference,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'e' :
			if (token.equals("enumSwitch") //$NON-NLS-1$
					|| token.equals("incomplete-switch")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportIncompleteEnumSwitch,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("emptyBlock")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUndocumentedEmptyBlock,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("enumIdentifier")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportEnumIdentifier,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'f' :
			if (token.equals("fieldHiding")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportFieldHiding,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("finalBound")) {//$NON-NLS-1$
				this.options.put(
						CompilerOptions.OPTION_ReportFinalParameterBound,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("finally")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportFinallyBlockNotCompletingNormally,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("forbidden")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportForbiddenReference,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("fallthrough")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportFallthroughCase,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'h' :
			if (token.equals("hiding")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportHiddenCatchBlock,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportLocalVariableHiding,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportFieldHiding,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportTypeParameterHiding,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'i' :
			if (token.equals("indirectStatic")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportIndirectStaticAccess,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("intfNonInherited") || token.equals("interfaceNonInherited")/*backward compatible*/) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportIncompatibleNonInheritedInterfaceMethod,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("intfAnnotation")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportAnnotationSuperInterface,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("intfRedundant") /*|| token.equals("redundantSuperinterface")*/) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportRedundantSuperinterface,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);	
				return;
			}
			break;
		case 'j' :
			if (token.equals("javadoc")) {//$NON-NLS-1$
				this.warnJavadocOn = isEnabling;
				return;
			}
			break;
		case 'l' :
			if (token.equals("localHiding")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportLocalVariableHiding,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'm' :
			if (token.equals("maskedCatchBlock") || token.equals("maskedCatchBlocks")/*backward compatible*/) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportHiddenCatchBlock,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'n' :
			if (token.equals("nls")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportNonExternalizedStringLiteral,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("noEffectAssign")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportNoEffectAssignment,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (/*token.equals("charConcat") ||*/ token.equals("noImplicitStringConversion")/*backward compatible*/) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportNoImplicitStringConversion,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("null")) { //$NON-NLS-1$
				this.options.put(
						CompilerOptions.OPTION_ReportNullReference,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
						CompilerOptions.OPTION_ReportPotentialNullReference,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
						CompilerOptions.OPTION_ReportRedundantNullCheck,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("nullDereference")) { //$NON-NLS-1$
				this.options.put(
						CompilerOptions.OPTION_ReportNullReference,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				if (!isEnabling) {
					this.options.put(
							CompilerOptions.OPTION_ReportPotentialNullReference,
							CompilerOptions.IGNORE);
					this.options.put(
							CompilerOptions.OPTION_ReportRedundantNullCheck,
							CompilerOptions.IGNORE);
				}
				return;
			}
			break;
		case 'o' :
			if (token.equals("over-sync") /*|| token.equals("syncOverride")*/) { //$NON-NLS-1$ 
				this.options.put(
					CompilerOptions.OPTION_ReportMissingSynchronizedOnInheritedMethod,
					isEnabling ? CompilerOptions.ERROR : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("over-ann")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportMissingOverrideAnnotation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'p' :
			if (token.equals("pkgDefaultMethod") || token.equals("packageDefaultMethod")/*backward compatible*/ ) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportOverridingPackageDefaultMethod,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);	
				return;
			} else if (token.equals("paramAssign")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportParameterAssignment,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'r' :
			if (token.equals("raw")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportRawTypeReference,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (/*token.equals("intfRedundant") ||*/ token.equals("redundantSuperinterface")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportRedundantSuperinterface,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);	
				return;
			}
			break;
		case 's' :
			if (token.equals("specialParamHiding")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportSpecialParameterHidingField,
					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
				return;
			} else if (token.equals("syntheticAccess") || token.equals("synthetic-access")) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportSyntheticAccessEmulation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("staticReceiver")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportNonStaticAccessToStatic,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else 	if (/*token.equals("over-sync") ||*/ token.equals("syncOverride")) { //$NON-NLS-1$ 
				this.options.put(
					CompilerOptions.OPTION_ReportMissingSynchronizedOnInheritedMethod,
					isEnabling ? CompilerOptions.ERROR : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("semicolon")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportEmptyStatement,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("serial")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportMissingSerialVersion,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("suppress")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_SuppressWarnings,
					isEnabling ? CompilerOptions.ENABLED : CompilerOptions.DISABLED);
				return;
			} else if (token.equals("static-access")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportNonStaticAccessToStatic,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportIndirectStaticAccess,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("super")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportOverridingMethodWithoutSuperInvocation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 't' :
			if (token.startsWith("tasks")) { //$NON-NLS-1$
				String taskTags = Util.EMPTY_STRING;
				int start = token.indexOf('(');
				int end = token.indexOf(')');
				if (start >= 0 && end >= 0 && start < end){
					taskTags = token.substring(start+1, end).trim();
					taskTags = taskTags.replace('|',',');
				}
				if (taskTags.length() == 0){
					throw new InvalidInputException(this.bind("configure.invalidTaskTag", token)); //$NON-NLS-1$
				}
				this.options.put(
					CompilerOptions.OPTION_TaskTags,
					isEnabling ? taskTags : Util.EMPTY_STRING);
				return;
			} else if (token.equals("typeHiding")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportTypeParameterHiding,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'u' :
			if (token.equals("unusedLocal") || token.equals("unusedLocals")/*backward compatible*/) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedLocal,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unusedArgument") || token.equals("unusedArguments")/*backward compatible*/) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedParameter,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unusedImport") || token.equals("unusedImports")/*backward compatible*/) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedImport,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unusedPrivate")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedPrivateMember,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unusedLabel")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedLabel,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("uselessTypeCheck")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnnecessaryTypeCheck,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unchecked") || token.equals("unsafe")) {//$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportUncheckedTypeOperation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unnecessaryElse")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnnecessaryElse,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unusedThrown")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedDeclaredThrownException,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unqualifiedField") || token.equals("unqualified-field-access")) { //$NON-NLS-1$ //$NON-NLS-2$
				this.options.put(
					CompilerOptions.OPTION_ReportUnqualifiedFieldAccess,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unused")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedLocal,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedParameter,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedImport,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedPrivateMember,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedDeclaredThrownException,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
						CompilerOptions.OPTION_ReportUnusedLabel,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
						CompilerOptions.OPTION_ReportUnusedTypeArgumentsForMethodInvocation,
						isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			} else if (token.equals("unusedTypeArgs")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedTypeArgumentsForMethodInvocation,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'v' :
			if (token.equals("varargsCast")) { //$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportVarargsArgumentNeedCast,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
		case 'w' :
			if (token.equals("warningToken")) {//$NON-NLS-1$
				this.options.put(
					CompilerOptions.OPTION_ReportUnhandledWarningToken,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				this.options.put(
					CompilerOptions.OPTION_ReportUnusedWarningToken,
					isEnabling ? CompilerOptions.WARNING : CompilerOptions.IGNORE);
				return;
			}
			break;
	}
	addPendingErrors(this.bind("configure.invalidWarning", token)); //$NON-NLS-1$
}
/**
 * @deprecated - use {@link #initialize(PrintWriter, PrintWriter, boolean, Map, CompilationProgress)} instead

Return to bug 246066

Lines 1660-1665 Link Here

(-)src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java (-4 / +65 lines)
1660	" indirectStatic indirect reference to static member\n" +	1660	" indirectStatic indirect reference to static member\n" +
1661	" intfAnnotation + annotation type used as super interface\n" +	1661	" intfAnnotation + annotation type used as super interface\n" +
1662	" intfNonInherited + interface non-inherited method compatibility\n" +	1662	" intfNonInherited + interface non-inherited method compatibility\n" +
		1663	" intfRedundant + find redundant superinterfaces\n" +
1663	" javadoc invalid javadoc\n" +	1664	" javadoc invalid javadoc\n" +
1664	" localHiding local variable hiding another variable\n" +	1665	" localHiding local variable hiding another variable\n" +
1665	" maskedCatchBlock + hidden catch block\n" +	1666	" maskedCatchBlock + hidden catch block\n" +
Lines 1671-1677 Link Here
1671	" paramAssign assignment to a parameter\n" +	1672	" paramAssign assignment to a parameter\n" +
1672	" pkgDefaultMethod + attempt to override package-default method\n" +	1673	" pkgDefaultMethod + attempt to override package-default method\n" +
1673	" raw + usage of raw type\n" +	1674	" raw + usage of raw type\n" +
1674	" redundantSuperinterface + find redundant superinterfaces\n" +
1675	" semicolon unnecessary semicolon, empty statement\n" +	1675	" semicolon unnecessary semicolon, empty statement\n" +
1676	" serial + missing serialVersionUID\n" +	1676	" serial + missing serialVersionUID\n" +
1677	" specialParamHiding constructor or setter parameter hiding a field\n" +	1677	" specialParamHiding constructor or setter parameter hiding a field\n" +
Lines 1679-1692 Link Here
1679	" staticReceiver + non-static reference to static member\n" +	1679	" staticReceiver + non-static reference to static member\n" +
1680	" super overriding a method without making a super invocation\n" +	1680	" super overriding a method without making a super invocation\n" +
1681	" suppress + enable @SuppressWarnings\n" +	1681	" suppress + enable @SuppressWarnings\n" +
1682	" syncOverride overriding a synchronized method without synchronizing it\n" +	1682	" syncOverride missing synchronized in synchr. method override\n" +
1683	" synthetic-access same as syntheticAccess\n" +
1684	" syntheticAccess synthetic access for innerclass\n" +	1683	" syntheticAccess synthetic access for innerclass\n" +
1685	" tasks(<tags separated by \|>) tasks identified by tags inside comments\n" +	1684	" tasks(<tags separated by \|>) tasks identified by tags inside comments\n" +
1686	" typeHiding + type parameter hiding another type\n" +	1685	" typeHiding + type parameter hiding another type\n" +
1687	" unchecked + unchecked type operation\n" +	1686	" unchecked + unchecked type operation\n" +
1688	" unnecessaryElse unnecessary else clause\n" +	1687	" unnecessaryElse unnecessary else clause\n" +
1689	" unqualified-field-access same as unqualifiedField\n" +
1690	" unqualifiedField unqualified reference to field\n" +	1688	" unqualifiedField unqualified reference to field\n" +
1691	" unused macro for unusedArgument, unusedImport, unusedLabel,\n" +	1689	" unused macro for unusedArgument, unusedImport, unusedLabel,\n" +
1692	" unusedLocal, unusedPrivate, unusedThrown,\n" +	1690	" unusedLocal, unusedPrivate, unusedThrown,\n" +
Lines 10963-10966 Link Here
10963	"Unrecognized option : @options2.txt\n" /* stderr */,	10961	"Unrecognized option : @options2.txt\n" /* stderr */,
10964	true /shouldFlushOutput/);	10962	true /shouldFlushOutput/);
10965	}	10963	}
		10964	//https://bugs.eclipse.org/bugs/show_bug.cgi?id=246066
		10965	public void test288_warn_options() {
		10966	this.runConformTest(
		10967	new String[] {
		10968	"X.java",
		10969	"interface IX {}\n" +
		10970	"class BaseX implements IX {}\n" +
		10971	"public class X extends BaseX implements IX {\n" +
		10972	"}\n",
		10973	},
		10974	"\"" + OUTPUT_DIR + File.separator + "X.java\""
		10975	+ " -sourcepath \"" + OUTPUT_DIR + "\""
		10976	+ " -warn:+intfRedundant -proc:none -d \"" + OUTPUT_DIR + "\"",
		10977	"",
		10978	"----------\n" +
		10979	"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/X.java (at line 3)\n" +
		10980	" public class X extends BaseX implements IX {\n" +
		10981	" ^^\n" +
		10982	"Redundant superinterface IX for the type X, already defined by BaseX\n" +
		10983	"----------\n" +
		10984	"1 problem (1 warning)",
		10985	true);
		10986	}
		10987	//https://bugs.eclipse.org/bugs/show_bug.cgi?id=246066 - variation
		10988	public void test289_warn_options() {
		10989	this.runConformTest(
		10990	new String[] {
		10991	"X.java",
		10992	"interface IX {}\n" +
		10993	"class BaseX implements IX {}\n" +
		10994	"public class X extends BaseX implements IX {\n" +
		10995	"}\n",
		10996	},
		10997	"\"" + OUTPUT_DIR + File.separator + "X.java\""
		10998	+ " -sourcepath \"" + OUTPUT_DIR + "\""
		10999	+ " -warn:+redundantSuperinterface -proc:none -d \"" + OUTPUT_DIR + "\"",
		11000	"",
		11001	"----------\n" +
		11002	"1. WARNING in ---OUTPUT_DIR_PLACEHOLDER---/X.java (at line 3)\n" +
		11003	" public class X extends BaseX implements IX {\n" +
		11004	" ^^\n" +
		11005	"Redundant superinterface IX for the type X, already defined by BaseX\n" +
		11006	"----------\n" +
		11007	"1 problem (1 warning)",
		11008	true);
		11009	}
		11010	//https://bugs.eclipse.org/bugs/show_bug.cgi?id=246066 - variation
		11011	public void test290_warn_options() {
		11012	this.runConformTest(
		11013	new String[] {
		11014	"X.java",
		11015	"interface IX {}\n" +
		11016	"class BaseX implements IX {}\n" +
		11017	"public class X extends BaseX implements IX {\n" +
		11018	"}\n",
		11019	},
		11020	"\"" + OUTPUT_DIR + File.separator + "X.java\""
		11021	+ " -sourcepath \"" + OUTPUT_DIR + "\""
		11022	+ " -warn:+intfRedundant -warn:-intfRedundant -proc:none -d \"" + OUTPUT_DIR + "\"",
		11023	"",
		11024	"",
		11025	true);
		11026	}
10966	}	11027	}

Lines 257-262 Link Here

(-)batch/org/eclipse/jdt/internal/compiler/batch/messages.properties (-4 / +2 lines)
257	\ indirectStatic indirect reference to static member\n\	257	\ indirectStatic indirect reference to static member\n\
258	\ intfAnnotation + annotation type used as super interface\n\	258	\ intfAnnotation + annotation type used as super interface\n\
259	\ intfNonInherited + interface non-inherited method compatibility\n\	259	\ intfNonInherited + interface non-inherited method compatibility\n\
		260	\ intfRedundant + find redundant superinterfaces\n\
260	\ javadoc invalid javadoc\n\	261	\ javadoc invalid javadoc\n\
261	\ localHiding local variable hiding another variable\n\	262	\ localHiding local variable hiding another variable\n\
262	\ maskedCatchBlock + hidden catch block\n\	263	\ maskedCatchBlock + hidden catch block\n\
Lines 268-274 Link Here
268	\ paramAssign assignment to a parameter\n\	269	\ paramAssign assignment to a parameter\n\
269	\ pkgDefaultMethod + attempt to override package-default method\n\	270	\ pkgDefaultMethod + attempt to override package-default method\n\
270	\ raw + usage of raw type\n\	271	\ raw + usage of raw type\n\
271	\ redundantSuperinterface + find redundant superinterfaces\n\
272	\ semicolon unnecessary semicolon, empty statement\n\	272	\ semicolon unnecessary semicolon, empty statement\n\
273	\ serial + missing serialVersionUID\n\	273	\ serial + missing serialVersionUID\n\
274	\ specialParamHiding constructor or setter parameter hiding a field\n\	274	\ specialParamHiding constructor or setter parameter hiding a field\n\
Lines 276-289 Link Here
276	\ staticReceiver + non-static reference to static member\n\	276	\ staticReceiver + non-static reference to static member\n\
277	\ super overriding a method without making a super invocation\n\	277	\ super overriding a method without making a super invocation\n\
278	\ suppress + enable @SuppressWarnings\n\	278	\ suppress + enable @SuppressWarnings\n\
279	\ syncOverride overriding a synchronized method without synchronizing it\n\	279	\ syncOverride missing synchronized in synchr. method override\n\
280	\ synthetic-access same as syntheticAccess\n\
281	\ syntheticAccess synthetic access for innerclass\n\	280	\ syntheticAccess synthetic access for innerclass\n\
282	\ tasks(<tags separated by \|>) tasks identified by tags inside comments\n\	281	\ tasks(<tags separated by \|>) tasks identified by tags inside comments\n\
283	\ typeHiding + type parameter hiding another type\n\	282	\ typeHiding + type parameter hiding another type\n\
284	\ unchecked + unchecked type operation\n\	283	\ unchecked + unchecked type operation\n\
285	\ unnecessaryElse unnecessary else clause\n\	284	\ unnecessaryElse unnecessary else clause\n\
286	\ unqualified-field-access same as unqualifiedField\n\
287	\ unqualifiedField unqualified reference to field\n\	285	\ unqualifiedField unqualified reference to field\n\
288	\ unused macro for unusedArgument, unusedImport, unusedLabel,\n\	286	\ unused macro for unusedArgument, unusedImport, unusedLabel,\n\
289	\ unusedLocal, unusedPrivate, unusedThrown,\n\	287	\ unusedLocal, unusedPrivate, unusedThrown,\n\