Attachment #45150 for bug #96237




	private final int DEFAULT_VISIBILITY = 2;
	private final int PRIVATE_VISIBILITY = 3;

	private static String INVALID_CLASS_JAVADOC_REF = "	/**\n" + 
		"	 * @see X_dep\n" + 
		"	 * @see X.X_priv\n" + 
		"	 * @see X.Unknown\n" + 
		"	 * @see X#X(int)\n" + 

		"	 * @see X#foo_dep(String)\n" + 
		"	 * @see X#unknown()\n" + 
		"	 */\n";
	private static String INVALID_METHOD_JAVADOC_REF = "	/**\n" + 
		"	 * @param str\n" + 
		"	 * @param str\n" + 
		"	 * @param xxx\n" + 

		"	 * @throws IllegalArgumentException\n" + 
		"	 * @throws java.io.IOException\n" + 
		"	 * @throws Unknown\n" + 
		"	 * @see X_dep\n" + 
		"	 * @see X.X_priv\n" + 
		"	 * @see X.Unknown\n" + 
		"	 * @see X#X(int)\n" + 

		"	 * @see X#foo_dep(String)\n" + 
		"	 * @see X#unknown()\n" + 
		"	 */\n";
	private static String DEP_CLASS =
		"/** @deprecated */\n" + 
		"public class X_dep {}\n";
	private static String REF_CLASS =
		"public class X {\n" + 
// Deprecated class must be a top level to avoid visibility issue
//		"	/** @deprecated */\n" + 
//		"	class X_dep{}\n" + 
		"	private class X_priv{}\n" + 
		"	/** @deprecated */\n" + 
		"	int x_dep;\n" + 

		"	void foo_dep() {}\n" + 
		"	private void foo_priv() {}\n" + 
		"	}\n";
	private static String[] CLASSES_INVALID_COMMENT = {
		"X.java",
		REF_CLASS,
		"X_dep.java",
		DEP_CLASS,
		"Y.java",
		"public class Y {\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	public class X_pub {}\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	protected class X_prot {}\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	class X_pack {}\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	private class X_priv {}\n" + 
			"}\n" + 
			"\n"
	};
	private static String[] FIELDS_INVALID_COMMENT = {
		"X.java",
		REF_CLASS,
		"X_dep.java",
		DEP_CLASS,
		"Y.java",
		"public class Y {\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	public int x_pub;\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	protected int x_prot;\n" + 
			INVALID_CLASS_JAVADOC_REF + 
			"	int x_pack;\n" + 
			INVALID_CLASS_JAVADOC_REF+ 
			"	private int x_priv;\n" + 
			"}\n" + 
			"\n"
	};
	private static String[] METHODS_INVALID_COMMENT = {
		"X.java",
		REF_CLASS,
		"X_dep.java",
		DEP_CLASS,
		"Y.java",
		"public class Y {\n" + 
			INVALID_METHOD_JAVADOC_REF + 
			"	public void foo_pub(String str) throws IllegalArgumentException {}\n" +
			INVALID_METHOD_JAVADOC_REF + 
			"	protected void foo_pro(String str) throws IllegalArgumentException {}\n" + 
			INVALID_METHOD_JAVADOC_REF + 
			"	void foo_pack(String str) throws IllegalArgumentException {}\n" + 
			INVALID_METHOD_JAVADOC_REF + 
			"	private void foo_priv(String str) throws IllegalArgumentException {}\n" +
			"}\n" + 
			"\n"
	};
	private static String[] CONSTRUCTORS_INVALID_COMMENT = {
		"X.java",
		REF_CLASS,
		"X_dep.java",
		DEP_CLASS,
		"Y.java",
		"public class Y {\n" + 
			INVALID_METHOD_JAVADOC_REF + 
			"	public Y(int str) {}\n" +
			INVALID_METHOD_JAVADOC_REF + 
			"	protected Y(long str) {}\n" + 
			INVALID_METHOD_JAVADOC_REF + 
			"	Y(float str) {}\n" + 
			INVALID_METHOD_JAVADOC_REF + 
			"	private Y(double str) {}\n" +
			"}\n" + 
			"\n"
	};
	private static String[] MISSING_TAGS = {
		"X.java",
		"public class X {\n" + 
			"	// public\n" + 

			"	private int privateMethod(long l) { return 0;}\n" + 
			"}\n"
	};
	private static String[] MISSING_COMMENTS = {
		"X.java",
		"/** */\n" + 
			"public class X {\n" + 

			"}\n"
	};

	private static String[] CLASSES_ERRORS = {
		"1. ERROR in Y.java (at line 3)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"2. ERROR in Y.java (at line 4)\n" + 
			"	* @see X.X_priv\n" + 

			"Javadoc: The method unknown() is undefined for the type X\n" + 
			"----------\n",
		"14. ERROR in Y.java (at line 19)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"15. ERROR in Y.java (at line 20)\n" + 
			"	* @see X.X_priv\n" + 

			"Javadoc: The method unknown() is undefined for the type X\n" + 
			"----------\n",
		"27. ERROR in Y.java (at line 35)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"28. ERROR in Y.java (at line 36)\n" + 
			"	* @see X.X_priv\n" + 

			"Javadoc: The method unknown() is undefined for the type X\n" + 
			"----------\n",
		"40. ERROR in Y.java (at line 51)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"41. ERROR in Y.java (at line 52)\n" + 
			"	* @see X.X_priv\n" + 

			"----------\n"
	};

	private static String[] METHODS_ERRORS = {
		"1. ERROR in Y.java (at line 4)\n" + 
			"	* @param str\n" + 
			"	         ^^^\n" + 

			"Javadoc: Unknown cannot be resolved to a type\n" + 
			"----------\n" + 
			"5. ERROR in Y.java (at line 10)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"6. ERROR in Y.java (at line 11)\n" + 
			"	* @see X.X_priv\n" + 

			"Javadoc: Unknown cannot be resolved to a type\n" + 
			"----------\n" + 
			"22. ERROR in Y.java (at line 33)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"23. ERROR in Y.java (at line 34)\n" + 
			"	* @see X.X_priv\n" + 

			"Javadoc: Unknown cannot be resolved to a type\n" + 
			"----------\n" + 
			"39. ERROR in Y.java (at line 56)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"40. ERROR in Y.java (at line 57)\n" + 
			"	* @see X.X_priv\n" + 

			"Javadoc: Unknown cannot be resolved to a type\n" + 
			"----------\n" + 
			"56. ERROR in Y.java (at line 79)\n" + 
			"	* @see X_dep\n" + 
			"	       ^^^^^\n" + 
			"Javadoc: The type X_dep is deprecated\n" + 
			"----------\n" + 
			"57. ERROR in Y.java (at line 80)\n" + 
			"	* @see X.X_priv\n" + 

		if (reportInvalidJavadocTagsDeprecatedRef == null && reportInvalidJavadocTagsNotVisibleRef == null) {
			String result = "----------\n";
			for (int i=0; i<=visibility; i++) {
				result += CLASSES_ERRORS[i];
			}
			return result;
		}
		StringBuffer result = new StringBuffer("----------\n");
		for (int i=0, count=1; i<= visibility; i++) {
			StringTokenizer tokenizer = new StringTokenizer(CLASSES_ERRORS[i], "\n");
			while (tokenizer.hasMoreTokens()) {
				StringBuffer error = new StringBuffer();
				boolean add = true;

								add = line.indexOf("is deprecated") == -1;
							}
							if (add && CompilerOptions.DISABLED.equals(reportInvalidJavadocTagsNotVisibleRef)) {
								add = line.indexOf("is not visible") == -1 && line.indexOf("in generated documentation") == -1;
							}
						default:
							error.append(line);

		if (reportInvalidJavadocTagsDeprecatedRef == null && reportInvalidJavadocTagsNotVisibleRef == null) {
			String result = "----------\n";
			for (int i=0; i<=visibility; i++) {
				result += METHODS_ERRORS[i];
			}
			return result;
		}
		StringBuffer result = new StringBuffer("----------\n");
		for (int i=0, count=1; i<= visibility; i++) {
			StringTokenizer tokenizer = new StringTokenizer(METHODS_ERRORS[i], "\n");
			while (tokenizer.hasMoreTokens()) {
				StringBuffer error = new StringBuffer();
				boolean add = true;

								add = line.indexOf("is deprecated") == -1;
							}
							if (add && CompilerOptions.DISABLED.equals(reportInvalidJavadocTagsNotVisibleRef)) {
								add = line.indexOf("is not visible") == -1 && line.indexOf("in generated documentation") == -1;
							}
						default:
							error.append(line);

	 */
	public void testInvalidTagsClassNoSupport() {
		docCommentSupport = CompilerOptions.DISABLED;
		runConformTest(CLASSES_INVALID_COMMENT);
	}
	public void testInvalidTagsFieldNoSupport() {
		docCommentSupport = CompilerOptions.DISABLED;
		runConformTest(FIELDS_INVALID_COMMENT);
	}
	public void testInvalidTagsMethodNoSupport() {
		docCommentSupport = CompilerOptions.DISABLED;
		runConformTest(METHODS_INVALID_COMMENT);
	}
	public void testInvalidTagsConstructorNoSupport() {
		docCommentSupport = CompilerOptions.DISABLED;
		runConformTest(CONSTRUCTORS_INVALID_COMMENT);
	}

	/*

	 */
	// Test default invalid javadoc (means "ignore" with tags"disabled" and visibility "public")
	public void testInvalidTagsClassDefaults() {
		runConformTest(CLASSES_INVALID_COMMENT);
	}
	public void testInvalidTagsFieldDefaults() {
		runConformTest(FIELDS_INVALID_COMMENT);
	}
	public void testInvalidTagsMethodDefaults() {
		runConformTest(METHODS_INVALID_COMMENT);
	}
	public void testInvalidTagsConstructorDefaults() {
		runConformTest(CONSTRUCTORS_INVALID_COMMENT);
	}

	// Test invalid javadoc "error" + tags "disabled" and visibility "public"
	public void testInvalidTagsClassErrorNotags() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.DISABLED;
		runConformTest(CLASSES_INVALID_COMMENT);
	}
	public void testInvalidTagsFieldErrorNotags() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.DISABLED;
		runConformTest(FIELDS_INVALID_COMMENT);
	}
	public void testInvalidTagsMethodErrorNotags() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.DISABLED;
		runConformTest(METHODS_INVALID_COMMENT);
	}
	public void testInvalidTagsConstructorErrorNotags() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.DISABLED;
		runConformTest(CONSTRUCTORS_INVALID_COMMENT);
	}

	// Test invalid javadoc "error" + tags "enabled" and visibility "public"

		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsFieldErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsMethodErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsConstructorErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" and visibility "protected"

		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsFieldErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsMethodErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsConstructorErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" and visibility "default"

		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsFieldErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsMethodErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsConstructorErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" and visibility "private"
	public void testInvalidTagsClassErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsFieldErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsMethodErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsConstructorErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated references "disabled" and visibility "public"

		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefFieldErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefMethodErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefConstructorErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated references "disabled" visibility "protected"

		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefFieldErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefMethodErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefConstructorErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated references "disabled" and visibility "default"

		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefFieldErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefMethodErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefConstructorErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated references "disabled" and visibility "private"
	public void testInvalidTagsDeprecatedRefClassErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefFieldErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefMethodErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedRefConstructorErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid not visible references "disabled" and visibility "public"

		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefFieldErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefMethodErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefConstructorErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid not visible references "disabled" visibility "protected"

		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefFieldErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefMethodErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefConstructorErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid not visible references "disabled" and visibility "default"

		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefFieldErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefMethodErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefConstructorErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid not visible references "disabled" and visibility "private"
	public void testInvalidTagsNotVisibleRefClassErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefFieldErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefMethodErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsNotVisibleRefConstructorErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated or not visible references "disabled" and visibility "public"

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefFieldErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefMethodErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefConstructorErrorTagsPublic() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PUBLIC;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PUBLIC_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated or not visible references "disabled" visibility "protected"

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefFieldErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefMethodErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefConstructorErrorTagsProtected() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PROTECTED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PROTECTED_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated or not visible references "disabled" and visibility "default"

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefFieldErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefMethodErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefConstructorErrorTagsPackage() {
		reportInvalidJavadoc = CompilerOptions.ERROR;

		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.DEFAULT;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(DEFAULT_VISIBILITY));
	}

	// Test invalid javadoc "error" + tags "enabled" but invalid deprecated or not visible references "disabled" and visibility "private"

		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(CLASSES_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefFieldErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(FIELDS_INVALID_COMMENT, resultForInvalidTagsClassOrField(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefMethodErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(METHODS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}
	public void testInvalidTagsDeprecatedAndNotVisibleRefConstructorErrorTagsPrivate() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTagsDeprecatedRef = CompilerOptions.DISABLED;
		reportInvalidJavadocTagsNotVisibleRef = CompilerOptions.DISABLED;
		runNegativeTest(CONSTRUCTORS_INVALID_COMMENT, resultForInvalidTagsMethodOrConstructor(PRIVATE_VISIBILITY));
	}

	/*

	 */
	// Test default missing javadoc tags (means "ignore" with visibility "public" and overriding="enabled")
	public void testMissingTagsDefaults() {
		runConformTest(MISSING_TAGS);
	}

	// Test missing javadoc tags "error" + "public" visibility + "enabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PUBLIC;
		reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(PUBLIC_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "public" visibility + "disabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PUBLIC;
		reportMissingJavadocTagsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(PUBLIC_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "protected" visibility + "enabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PROTECTED;
		reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(PROTECTED_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "protected" visibility + "disabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PROTECTED;
		reportMissingJavadocTagsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(PROTECTED_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "default" visibility + "enabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.DEFAULT;
		reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(DEFAULT_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "default" visibility + "disabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.DEFAULT;
		reportMissingJavadocTagsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(DEFAULT_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "private" visibility + "enabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PRIVATE;
		reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(PRIVATE_VISIBILITY));
	}

	// Test missing javadoc tags "error" + "private" visibility + "disabled" overriding

		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PRIVATE;
		reportMissingJavadocTagsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_TAGS, resultForMissingTags(PRIVATE_VISIBILITY));
	}

	/*

	 */
	// Test default missing javadoc comments (means "ignore" with visibility "public" and overriding="enabled")
	public void testMissingCommentsDefaults() {
		runConformTest(MISSING_COMMENTS);
	}

	// Test missing javadoc comments "error" + "public" visibility + "enabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.PUBLIC;
		reportMissingJavadocCommentsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(PUBLIC_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "public" visibility + "disabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.PUBLIC;
		reportMissingJavadocCommentsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(PUBLIC_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "protected" visibility + "enabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.PROTECTED;
		reportMissingJavadocCommentsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(PROTECTED_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "protected" visibility + "disabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.PROTECTED;
		reportMissingJavadocCommentsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(PROTECTED_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "default" visibility + "enabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.DEFAULT;
		reportMissingJavadocCommentsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(DEFAULT_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "default" visibility + "disabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.DEFAULT;
		reportMissingJavadocCommentsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(DEFAULT_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "private" visibility + "enabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.PRIVATE;
		reportMissingJavadocCommentsOverriding = CompilerOptions.ENABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(PRIVATE_VISIBILITY));
	}

	// Test missing javadoc comments "error" + "private" visibility + "disabled" overriding

		reportMissingJavadocComments = CompilerOptions.ERROR;
		reportMissingJavadocCommentsVisibility = CompilerOptions.PRIVATE;
		reportMissingJavadocCommentsOverriding = CompilerOptions.DISABLED;
		runNegativeTest(MISSING_COMMENTS, resultForMissingComments(PRIVATE_VISIBILITY));
	}
	
	/*

	public void testInvalidTagsClassWithMissingTagsOption() {
		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PRIVATE;
		runConformTest(CLASSES_INVALID_COMMENT);
	}
	public void testInvalidTagsFieldWithMissingTagsOption() {
		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PRIVATE;
		runConformTest(FIELDS_INVALID_COMMENT);
	}
	public void testInvalidTagsMethodWithMissingTagsOption() {
		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PRIVATE;
		runConformTest(METHODS_INVALID_COMMENT);
	}
	public void testInvalidTagsConstructorWithMissingTagsOption() {
		reportMissingJavadocTags = CompilerOptions.ERROR;
		reportMissingJavadocTagsVisibility = CompilerOptions.PRIVATE;
		runConformTest(CONSTRUCTORS_INVALID_COMMENT);
	}
	public void testMissingTagsWithInvalidTagsOption() {
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ENABLED;
		reportInvalidJavadocTagsVisibility = CompilerOptions.PRIVATE;
		runConformTest(MISSING_TAGS);
	}

	/**

		this.docCommentSupport = CompilerOptions.DISABLED;
		reportInvalidJavadoc = CompilerOptions.ERROR;
		reportInvalidJavadocTags = CompilerOptions.ERROR;
		runConformTest(CLASSES_INVALID_COMMENT);
		runConformTest(FIELDS_INVALID_COMMENT);
		runConformTest(METHODS_INVALID_COMMENT);
		runConformTest(CONSTRUCTORS_INVALID_COMMENT);
	}

	// Test missing javadoc comments "error" with javadoc comment support disabled
	public void testMissingCommentsJavadocSupportDisabled() {
		this.docCommentSupport = CompilerOptions.DISABLED;
		reportMissingJavadocComments = CompilerOptions.ERROR;
		runConformReferenceTest(MISSING_COMMENTS);
	}

	// Test missing javadoc tags "error" with javadoc comment support disabled
	public void testMissingTagsJavadocSupportDisabled() {
		this.docCommentSupport = CompilerOptions.DISABLED;
		reportMissingJavadocTags = CompilerOptions.ERROR;
		runConformReferenceTest(MISSING_TAGS);
	}

}

Lines 20-25 Link Here

(-)src/org/eclipse/jdt/core/tests/compiler/regression/JavadocBugsTest.java (-261 / +254 lines)
20		20
21	String docCommentSupport = CompilerOptions.ENABLED;	21	String docCommentSupport = CompilerOptions.ENABLED;
22	String reportInvalidJavadoc = CompilerOptions.ERROR;	22	String reportInvalidJavadoc = CompilerOptions.ERROR;
		23	String reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
23	String reportMissingJavadocTags = CompilerOptions.ERROR;	24	String reportMissingJavadocTags = CompilerOptions.ERROR;
24	String reportMissingJavadocComments = null;	25	String reportMissingJavadocComments = null;
25	String reportDeprecation = CompilerOptions.ERROR;	26	String reportDeprecation = CompilerOptions.ERROR;
Lines 36-42 Link Here
36	// Use this static initializer to specify subset for tests	37	// Use this static initializer to specify subset for tests
37	// All specified tests which does not belong to the class are skipped...	38	// All specified tests which does not belong to the class are skipped...
38	static {	39	static {
39	// TESTS_PREFIX = "testBug83127";	40	// TESTS_PREFIX = "testBug96237";
40	// TESTS_NAMES = new String[] { "testBug68017javadocWarning2" };	41	// TESTS_NAMES = new String[] { "testBug68017javadocWarning2" };
41	// TESTS_NUMBERS = new int[] { 129241 };	42	// TESTS_NUMBERS = new int[] { 129241 };
42	// TESTS_RANGE = new int[] { 21, 50 };	43	// TESTS_RANGE = new int[] { 21, 50 };
Lines 49-54 Link Here
49	Map options = super.getCompilerOptions();	50	Map options = super.getCompilerOptions();
50	options.put(CompilerOptions.OPTION_DocCommentSupport, docCommentSupport);	51	options.put(CompilerOptions.OPTION_DocCommentSupport, docCommentSupport);
51	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);	52	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);
		53	if (!CompilerOptions.IGNORE.equals(reportInvalidJavadoc)) {
		54	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility, this.reportInvalidJavadocVisibility);
		55	}
52	if (this.reportJavadocDeprecation != null) {	56	if (this.reportJavadocDeprecation != null) {
53	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsDeprecatedRef, reportJavadocDeprecation);	57	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsDeprecatedRef, reportJavadocDeprecation);
54	}	58	}
Lines 77-82 Link Here
77	super.setUp();	81	super.setUp();
78	docCommentSupport = CompilerOptions.ENABLED;	82	docCommentSupport = CompilerOptions.ENABLED;
79	reportInvalidJavadoc = CompilerOptions.ERROR;	83	reportInvalidJavadoc = CompilerOptions.ERROR;
		84	reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
80	reportMissingJavadocTags = CompilerOptions.IGNORE;	85	reportMissingJavadocTags = CompilerOptions.IGNORE;
81	reportMissingJavadocComments = CompilerOptions.IGNORE;	86	reportMissingJavadocComments = CompilerOptions.IGNORE;
82	reportDeprecation = CompilerOptions.ERROR;	87	reportDeprecation = CompilerOptions.ERROR;
Lines 220-264 Link Here
220	public void testBug45592() {	225	public void testBug45592() {
221	runConformTest(	226	runConformTest(
222	new String[] {	227	new String[] {
223	"a/Y.java",	228	"a/Y.java",
224	"package a;\n" +	229	"package a;\n" +
225	"\n" +	230	"\n" +
226	"/** */\n" +	231	"/** */\n" +
227	"public class Y {\n" +	232	"public class Y {\n" +
228	" protected boolean bar(Object obj) {\n" +	233	" protected boolean bar(Object obj) {\n" +
229	" return obj == null;\n" +	234	" return obj == null;\n" +
230	" }\n" +	235	" }\n" +
231	"}\n",	236	"}\n",
232	"test/X.java",	237	"test/X.java",
233	"package test;\n" +	238	"package test;\n" +
234	"public class X {\n" +	239	"public class X {\n" +
235	" public static Boolean valueOf(boolean bool) {\n" +	240	" public static Boolean valueOf(boolean bool) {\n" +
236	" if (bool) {\n" +	241	" if (bool) {\n" +
237	" return Boolean.TRUE;\n" +	242	" return Boolean.TRUE;\n" +
238	" } else {\n" +	243	" } else {\n" +
239	" return Boolean.FALSE;\n" +	244	" return Boolean.FALSE;\n" +
240	" }\n" +	245	" }\n" +
241	" }\n" +	246	" }\n" +
242	"}\n",	247	"}\n",
243	"test/YY.java",	248	"test/YY.java",
244	"package test;\n" +	249	"package test;\n" +
245	"\n" +	250	"\n" +
246	"import a.Y;\n" +	251	"import a.Y;\n" +
247	"\n" +	252	"\n" +
248	"/** */\n" +	253	"/** */\n" +
249	"public class YY extends Y {\n" +	254	"public class YY extends Y {\n" +
250	" /**\n" +	255	" /**\n" +
251	" * Returns a Boolean.\n" +	256	" * Returns a Boolean.\n" +
252	" * @param key\n" +	257	" * @param key\n" +
253	" * @return A Boolean telling whether the key is null or not.\n" +	258	" * @return A Boolean telling whether the key is null or not.\n" +
254	" * @see #bar(Object)\n" +	259	" * @see #bar(Object)\n" +
255	" */\n" +	260	" */\n" +
256	" protected Boolean foo(Object key) {\n" +	261	" protected Boolean foo(Object key) {\n" +
257	" return X.valueOf(bar(key));\n" +	262	" return X.valueOf(bar(key));\n" +
258	" }\n" +	263	" }\n" +
259	"}\n"	264	"}\n"
260	}	265	}
261	);	266	);
262	}	267	}
263		268
264	/**	269	/**
Lines 690-701 Link Here
690	"}\n"	695	"}\n"
691	},	696	},
692	"----------\n" +	697	"----------\n" +
693	"1. WARNING in X.java (at line 2)\n" +	698	"1. ERROR in X.java (at line 5)\n" +
694	" public class X extends RuntimeException {\n" +
695	" ^\n" +
696	"The serializable class X does not declare a static final serialVersionUID field of type long\n" +
697	"----------\n" +
698	"2. ERROR in X.java (at line 5)\n" +
699	" public X(String message) {\n" +	699	" public X(String message) {\n" +
700	" ^^^^^^^\n" +	700	" ^^^^^^^\n" +
701	"Javadoc: Missing tag for parameter message\n" +	701	"Javadoc: Missing tag for parameter message\n" +
Lines 2420-2425 Link Here
2420	}	2420	}
2421		2421
2422	/**	2422	/**
		2423	* Bug 68726: [Javadoc] Target attribute in @see link triggers warning
		2424	* @see <a href="http://bugs.eclipse.org/bugs/show_bug.cgi?id=68726">68726</a>
		2425	*/
		2426	public void testBug68726conform1() {
		2427	runConformTest(
		2428	new String[] {
		2429	"X.java",
		2430	"public class X {\n" +
		2431	" /**\n" +
		2432	" * @see Object <a href=\"http://www.eclipse.org\" target=\"_top\">Eclipse</a>\n" +
		2433	" */\n" +
		2434	" void foo1() {}\n" +
		2435	" /*@see Object <a href=\"http://www.eclipse.org\" target=\"_top\" target1=\"_top1\" target2=\"_top2\">Eclipse</a>/\n" +
		2436	" void foo2() {}\n" +
		2437	"}\n"
		2438	}
		2439	);
		2440	}
		2441	public void testBug68726conform2() {
		2442	runConformTest(
		2443	new String[] {
		2444	"X.java",
		2445	"/**\n" +
		2446	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">IBM Home Page</a>\n" +
		2447	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
		2448	" * IBM Home Page</a>\n" +
		2449	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
		2450	" * IBM Home Page\n" +
		2451	" * </a>\n" +
		2452	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
		2453	" *\n" +
		2454	" * IBM\n" +
		2455	" *\n" +
		2456	" * Home Page\n" +
		2457	" *\n" +
		2458	" *\n" +
		2459	" * </a>\n" +
		2460	" * @see Object\n" +
		2461	" */\n" +
		2462	"public class X {\n" +
		2463	"}\n"
		2464	}
		2465	);
		2466	}
		2467	public void testBug68726negative1() {
		2468	runNegativeTest(
		2469	new String[] {
		2470	"X.java",
		2471	"public class X {\n" +
		2472	" /**\n" +
		2473	" * Invalid URL link references\n" +
		2474	" *\n" +
		2475	" * @see <a href=\"invalid\" target\n" +
		2476	" * @see <a href=\"invalid\" target=\n" +
		2477	" * @see <a href=\"invalid\" target=\"\n" +
		2478	" * @see <a href=\"invalid\" target=\"_top\n" +
		2479	" * @see <a href=\"invalid\" target=\"_top\"\n" +
		2480	" * @see <a href=\"invalid\" target=\"_top\">\n" +
		2481	" * @see <a href=\"invalid\" target=\"_top\">\n" +
		2482	" * @see <a href=\"invalid\" target=\"_top\">invalid\n" +
		2483	" * @see <a href=\"invalid\" target=\"_top\">invalid<\n" +
		2484	" * @see <a href=\"invalid\" target=\"_top\">invalid</\n" +
		2485	" * @see <a href=\"invalid\" target=\"_top\">invalid</a\n" +
		2486	" * @see <a href=\"invalid\" target=\"_top\">invalid</a> no text allowed after the href\n" +
2487	" */\n" +
2488	" void foo() {}\n" +
2489	"}\n"
2490	},
2491	"----------\n" +
2492	"1. ERROR in X.java (at line 5)\n" +
2493	" * @see <a href=\"invalid\" target\n" +
2494	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2495	"Javadoc: Malformed link reference\n" +
2496	"----------\n" +
2497	"2. ERROR in X.java (at line 6)\n" +
2498	" * @see <a href=\"invalid\" target=\n" +
2499	" ^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2500	"Javadoc: Malformed link reference\n" +
2501	"----------\n" +
2502	"3. ERROR in X.java (at line 7)\n" +
2503	" * @see <a href=\"invalid\" target=\"\n" +
2504	" ^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2505	"Javadoc: Malformed link reference\n" +
2506	"----------\n" +
2507	"4. ERROR in X.java (at line 8)\n" +
2508	" * @see <a href=\"invalid\" target=\"_top\n" +
2509	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2510	"Javadoc: Malformed link reference\n" +
2511	"----------\n" +
2512	"5. ERROR in X.java (at line 9)\n" +
2513	" * @see <a href=\"invalid\" target=\"_top\"\n" +
2514	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2515	"Javadoc: Malformed link reference\n" +
2516	"----------\n" +
2517	"6. ERROR in X.java (at line 10)\n" +
2518	" * @see <a href=\"invalid\" target=\"_top\">\n" +
2519	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2520	"Javadoc: Malformed link reference\n" +
2521	"----------\n" +
2522	"7. ERROR in X.java (at line 11)\n" +
2523	" * @see <a href=\"invalid\" target=\"_top\">\n" +
2524	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2525	"Javadoc: Malformed link reference\n" +
2526	"----------\n" +
2527	"8. ERROR in X.java (at line 12)\n" +
2528	" * @see <a href=\"invalid\" target=\"_top\">invalid\n" +
2529	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2530	"Javadoc: Malformed link reference\n" +
2531	"----------\n" +
2532	"9. ERROR in X.java (at line 13)\n" +
2533	" * @see <a href=\"invalid\" target=\"_top\">invalid<\n" +
2534	" ^\n" +
2535	"Javadoc: Malformed link reference\n" +
2536	"----------\n" +
2537	"10. ERROR in X.java (at line 14)\n" +
2538	" * @see <a href=\"invalid\" target=\"_top\">invalid</\n" +
2539	" ^^\n" +
2540	"Javadoc: Malformed link reference\n" +
2541	"----------\n" +
2542	"11. ERROR in X.java (at line 15)\n" +
2543	" * @see <a href=\"invalid\" target=\"_top\">invalid</a\n" +
2544	" ^^^\n" +
2545	"Javadoc: Malformed link reference\n" +
2546	"----------\n" +
2547	"12. ERROR in X.java (at line 16)\n" +
2548	" * @see <a href=\"invalid\" target=\"_top\">invalid</a> no text allowed after the href\n" +
2549	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2550	"Javadoc: Unexpected text\n" +
2551	"----------\n"
2552	);
2553	}
2554	public void testBug68726negative2() {
2555	runNegativeTest(
2556	new String[] {
2557	"X.java",
2558	"/**\n" +
2559	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">IBM Home Page\n" +
2560	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2561	" * IBM Home Page\n" +
2562	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2563	" * IBM Home Page<\n" +
2564	" * /a>\n" +
2565	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2566	" *\n" +
2567	" * IBM\n" +
2568	" *\n" +
2569	" * Home Page\n" +
2570	" *\n" +
2571	" *\n" +
2572	" * \n" +
2573	" * @see Unknown\n" +
2574	" */\n" +
2575	"public class X {\n" +
2576	"}\n"
2577	},
2578	"----------\n" +
2579	"1. ERROR in X.java (at line 2)\n" +
2580	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">IBM Home Page\n" +
2581	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2582	"Javadoc: Malformed link reference\n" +
2583	"----------\n" +
2584	"2. ERROR in X.java (at line 3)\n" +
2585	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2586	" * IBM Home Page\n" +
2587	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2588	"Javadoc: Malformed link reference\n" +
2589	"----------\n" +
2590	"3. ERROR in X.java (at line 6)\n" +
2591	" * IBM Home Page<\n" +
2592	" ^\n" +
2593	"Javadoc: Malformed link reference\n" +
2594	"----------\n" +
2595	"4. ERROR in X.java (at line 8)\n" +
2596	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2597	" *\n" +
2598	" * IBM\n" +
2599	" *\n" +
2600	" * Home Page\n" +
2601	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2602	"Javadoc: Malformed link reference\n" +
2603	"----------\n" +
2604	"5. ERROR in X.java (at line 16)\n" +
2605	" * @see Unknown\n" +
2606	" ^^^^^^^\n" +
2607	"Javadoc: Unknown cannot be resolved to a type\n" +
2608	"----------\n"
2609	);
2610	}
2611
2612	/**
2423	* Bug 69272: [Javadoc] Invalid malformed reference (missing separator)	2613	* Bug 69272: [Javadoc] Invalid malformed reference (missing separator)
2424	* @see <a href="http://bugs.eclipse.org/bugs/show_bug.cgi?id=69272">69272</a>	2614	* @see <a href="http://bugs.eclipse.org/bugs/show_bug.cgi?id=69272">69272</a>
2425	*/	2615	*/
Lines 2709-2904 Link Here
2709	}	2899	}
2710		2900
2711	/**	2901	/**
2712	* Bug 68726: [Javadoc] Target attribute in @see link triggers warning
2713	* @see <a href="http://bugs.eclipse.org/bugs/show_bug.cgi?id=68726">68726</a>
2714	*/
2715	public void testBug68726conform1() {
2716	runConformTest(
2717	new String[] {
2718	"X.java",
2719	"public class X {\n" +
2720	" /**\n" +
2721	" * @see Object <a href=\"http://www.eclipse.org\" target=\"_top\">Eclipse</a>\n" +
2722	" */\n" +
2723	" void foo1() {}\n" +
2724	" /*@see Object <a href=\"http://www.eclipse.org\" target=\"_top\" target1=\"_top1\" target2=\"_top2\">Eclipse</a>/\n" +
2725	" void foo2() {}\n" +
2726	"}\n"
2727	}
2728	);
2729	}
2730	public void testBug68726conform2() {
2731	runConformTest(
2732	new String[] {
2733	"X.java",
2734	"/**\n" +
2735	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">IBM Home Page</a>\n" +
2736	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2737	" * IBM Home Page</a>\n" +
2738	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2739	" * IBM Home Page\n" +
2740	" * </a>\n" +
2741	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2742	" *\n" +
2743	" * IBM\n" +
2744	" *\n" +
2745	" * Home Page\n" +
2746	" *\n" +
2747	" *\n" +
2748	" * </a>\n" +
2749	" * @see Object\n" +
2750	" */\n" +
2751	"public class X {\n" +
2752	"}\n"
2753	}
2754	);
2755	}
2756	public void testBug68726negative1() {
2757	runNegativeTest(
2758	new String[] {
2759	"X.java",
2760	"public class X {\n" +
2761	" /**\n" +
2762	" * Invalid URL link references\n" +
2763	" *\n" +
2764	" * @see <a href=\"invalid\" target\n" +
2765	" * @see <a href=\"invalid\" target=\n" +
2766	" * @see <a href=\"invalid\" target=\"\n" +
2767	" * @see <a href=\"invalid\" target=\"_top\n" +
2768	" * @see <a href=\"invalid\" target=\"_top\"\n" +
2769	" * @see <a href=\"invalid\" target=\"_top\">\n" +
2770	" * @see <a href=\"invalid\" target=\"_top\">\n" +
2771	" * @see <a href=\"invalid\" target=\"_top\">invalid\n" +
2772	" * @see <a href=\"invalid\" target=\"_top\">invalid<\n" +
2773	" * @see <a href=\"invalid\" target=\"_top\">invalid</\n" +
2774	" * @see <a href=\"invalid\" target=\"_top\">invalid</a\n" +
2775	" * @see <a href=\"invalid\" target=\"_top\">invalid</a> no text allowed after the href\n" +
2776	" */\n" +
2777	" void foo() {}\n" +
2778	"}\n"
2779	},
2780	"----------\n" +
2781	"1. ERROR in X.java (at line 5)\n" +
2782	" * @see <a href=\"invalid\" target\n" +
2783	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2784	"Javadoc: Malformed link reference\n" +
2785	"----------\n" +
2786	"2. ERROR in X.java (at line 6)\n" +
2787	" * @see <a href=\"invalid\" target=\n" +
2788	" ^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2789	"Javadoc: Malformed link reference\n" +
2790	"----------\n" +
2791	"3. ERROR in X.java (at line 7)\n" +
2792	" * @see <a href=\"invalid\" target=\"\n" +
2793	" ^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2794	"Javadoc: Malformed link reference\n" +
2795	"----------\n" +
2796	"4. ERROR in X.java (at line 8)\n" +
2797	" * @see <a href=\"invalid\" target=\"_top\n" +
2798	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2799	"Javadoc: Malformed link reference\n" +
2800	"----------\n" +
2801	"5. ERROR in X.java (at line 9)\n" +
2802	" * @see <a href=\"invalid\" target=\"_top\"\n" +
2803	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2804	"Javadoc: Malformed link reference\n" +
2805	"----------\n" +
2806	"6. ERROR in X.java (at line 10)\n" +
2807	" * @see <a href=\"invalid\" target=\"_top\">\n" +
2808	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2809	"Javadoc: Malformed link reference\n" +
2810	"----------\n" +
2811	"7. ERROR in X.java (at line 11)\n" +
2812	" * @see <a href=\"invalid\" target=\"_top\">\n" +
2813	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2814	"Javadoc: Malformed link reference\n" +
2815	"----------\n" +
2816	"8. ERROR in X.java (at line 12)\n" +
2817	" * @see <a href=\"invalid\" target=\"_top\">invalid\n" +
2818	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2819	"Javadoc: Malformed link reference\n" +
2820	"----------\n" +
2821	"9. ERROR in X.java (at line 13)\n" +
2822	" * @see <a href=\"invalid\" target=\"_top\">invalid<\n" +
2823	" ^\n" +
2824	"Javadoc: Malformed link reference\n" +
2825	"----------\n" +
2826	"10. ERROR in X.java (at line 14)\n" +
2827	" * @see <a href=\"invalid\" target=\"_top\">invalid</\n" +
2828	" ^^\n" +
2829	"Javadoc: Malformed link reference\n" +
2830	"----------\n" +
2831	"11. ERROR in X.java (at line 15)\n" +
2832	" * @see <a href=\"invalid\" target=\"_top\">invalid</a\n" +
2833	" ^^^\n" +
2834	"Javadoc: Malformed link reference\n" +
2835	"----------\n" +
2836	"12. ERROR in X.java (at line 16)\n" +
2837	" * @see <a href=\"invalid\" target=\"_top\">invalid</a> no text allowed after the href\n" +
2838	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2839	"Javadoc: Unexpected text\n" +
2840	"----------\n"
2841	);
2842	}
2843	public void testBug68726negative2() {
2844	runNegativeTest(
2845	new String[] {
2846	"X.java",
2847	"/**\n" +
2848	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">IBM Home Page\n" +
2849	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2850	" * IBM Home Page\n" +
2851	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2852	" * IBM Home Page<\n" +
2853	" * /a>\n" +
2854	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2855	" *\n" +
2856	" * IBM\n" +
2857	" *\n" +
2858	" * Home Page\n" +
2859	" *\n" +
2860	" *\n" +
2861	" * \n" +
2862	" * @see Unknown\n" +
2863	" */\n" +
2864	"public class X {\n" +
2865	"}\n"
2866	},
2867	"----------\n" +
2868	"1. ERROR in X.java (at line 2)\n" +
2869	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">IBM Home Page\n" +
2870	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2871	"Javadoc: Malformed link reference\n" +
2872	"----------\n" +
2873	"2. ERROR in X.java (at line 3)\n" +
2874	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2875	" * IBM Home Page\n" +
2876	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2877	"Javadoc: Malformed link reference\n" +
2878	"----------\n" +
2879	"3. ERROR in X.java (at line 6)\n" +
2880	" * IBM Home Page<\n" +
2881	" ^\n" +
2882	"Javadoc: Malformed link reference\n" +
2883	"----------\n" +
2884	"4. ERROR in X.java (at line 8)\n" +
2885	" * @see <a href=\"http:/www.ibm.com\" target=\"_top\">\n" +
2886	" *\n" +
2887	" * IBM\n" +
2888	" *\n" +
2889	" * Home Page\n" +
2890	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2891	"Javadoc: Malformed link reference\n" +
2892	"----------\n" +
2893	"5. ERROR in X.java (at line 16)\n" +
2894	" * @see Unknown\n" +
2895	" ^^^^^^^\n" +
2896	"Javadoc: Unknown cannot be resolved to a type\n" +
2897	"----------\n"
2898	);
2899	}
2900
2901	/**
2902	* Bug 70892: [1.5][Javadoc] Compiler should parse reference for inline tag @value	2902	* Bug 70892: [1.5][Javadoc] Compiler should parse reference for inline tag @value
2903	* @see <a href="http://bugs.eclipse.org/bugs/show_bug.cgi?id=70892">70892</a>	2903	* @see <a href="http://bugs.eclipse.org/bugs/show_bug.cgi?id=70892">70892</a>
2904	* These two tests should pass whatever the source level...	2904	* These two tests should pass whatever the source level...
Lines 3567-3596 Link Here
3567	/**	3567	/**
3568	* Bug 77602: [javadoc] "Only consider members as visible as" is does not work for syntax error	3568	* Bug 77602: [javadoc] "Only consider members as visible as" is does not work for syntax error
3569	*/	3569	*/
3570	public void testBug77602public() {	3570	public void testBug77602() {
3571	Map options = getCompilerOptions();
3572	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility, CompilerOptions.PUBLIC);
3573	runConformTest(
3574	new String[] {
3575	"X.java",
3576	"public class X {\n" +
3577	" /**\n" +
3578	" * @see\n" +
3579	" * @see UnknownClass\n" +
3580	" */\n" +
3581	" protected void foo() {\n" +
3582	" }\n" +
3583	"}\n"
3584	},
3585	"",
3586	null,
3587	true,
3588	null,
3589	options,
3590	null
3591	);
3592	}
3593	public void testBug77602private() {
3594	runNegativeTest(	3571	runNegativeTest(
3595	new String[] {	3572	new String[] {
3596	"X.java",	3573	"X.java",
Lines 3616-3621 Link Here
3616	"----------\n"	3593	"----------\n"
3617	);	3594	);
3618	}	3595	}
		3596	public void testBug77602_Public() {
		3597	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		3598	runConformTest(
		3599	new String[] {
		3600	"X.java",
		3601	"public class X {\n" +
		3602	" /**\n" +
		3603	" * @see\n" +
		3604	" * @see UnknownClass\n" +
		3605	" */\n" +
		3606	" protected void foo() {\n" +
		3607	" }\n" +
		3608	"}\n"
		3609	}
		3610	);
		3611	}
3619		3612
3620	/**	3613	/**
3621	* Bug 78091: [1.5][javadoc] Compiler should accept new 1.5 syntax for @param	3614	* Bug 78091: [1.5][javadoc] Compiler should accept new 1.5 syntax for @param
Lines 3751-3757 Link Here
3751	" class C { \n" +	3744	" class C { \n" +
3752	" /**\n" +	3745	" /**\n" +
3753	" * Link {@link #B(Exception)} OK\n" +	3746	" * Link {@link #B(Exception)} OK\n" +
3754	" * Link {@link #C(String)} OK\n" +	3747	" * Link {@link #B.C(String)} OK\n" +
3755	" * Link {@link #foo()} OK\n" +	3748	" * Link {@link #foo()} OK\n" +
3756	" * Link {@link #bar()} OK\n" +	3749	" * Link {@link #bar()} OK\n" +
3757	" */\n" +	3750	" */\n" +

Lines 20-25 Link Here

(-)src/org/eclipse/jdt/core/tests/compiler/regression/JavadocTest_1_4.java (-3 / +845 lines)
20		20
21	String docCommentSupport = CompilerOptions.ENABLED;	21	String docCommentSupport = CompilerOptions.ENABLED;
22	String reportInvalidJavadoc = CompilerOptions.ERROR;	22	String reportInvalidJavadoc = CompilerOptions.ERROR;
		23	String reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
23	String reportMissingJavadocTags = CompilerOptions.ERROR;	24	String reportMissingJavadocTags = CompilerOptions.ERROR;
24	String reportMissingJavadocComments = null;	25	String reportMissingJavadocComments = null;
25	String reportMissingJavadocCommentsVisibility = null;	26	String reportMissingJavadocCommentsVisibility = null;
Lines 50-55 Link Here
50	Map options = super.getCompilerOptions();	51	Map options = super.getCompilerOptions();
51	options.put(CompilerOptions.OPTION_DocCommentSupport, this.docCommentSupport);	52	options.put(CompilerOptions.OPTION_DocCommentSupport, this.docCommentSupport);
52	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);	53	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);
		54	if (!CompilerOptions.IGNORE.equals(reportInvalidJavadoc)) {
		55	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility, this.reportInvalidJavadocVisibility);
		56	}
53	if (reportMissingJavadocComments != null)	57	if (reportMissingJavadocComments != null)
54	options.put(CompilerOptions.OPTION_ReportMissingJavadocComments, reportMissingJavadocComments);	58	options.put(CompilerOptions.OPTION_ReportMissingJavadocComments, reportMissingJavadocComments);
55	else	59	else
Lines 72-80 Link Here
72	protected void setUp() throws Exception {	76	protected void setUp() throws Exception {
73	super.setUp();	77	super.setUp();
74	this.docCommentSupport = CompilerOptions.ENABLED;	78	this.docCommentSupport = CompilerOptions.ENABLED;
75	reportInvalidJavadoc = CompilerOptions.ERROR;	79	this.reportInvalidJavadoc = CompilerOptions.ERROR;
76	reportMissingJavadocTags = CompilerOptions.ERROR;	80	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
77	reportMissingJavadocComments = CompilerOptions.IGNORE;	81	this.reportMissingJavadocTags = CompilerOptions.ERROR;
		82	this.reportMissingJavadocComments = CompilerOptions.IGNORE;
78	}	83	}
79		84
80	/**	85	/**
Lines 2817-2822 Link Here
2817	}	2822	}
2818		2823
2819	/**	2824	/**
		2825	* Bug 96237: [javadoc] Inner types must be qualified
		2826	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=96237"
		2827	*/
		2828	public void testBug96237_Public01() {
		2829	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		2830	runConformTest(
		2831	new String[] {
		2832	"comment6/Valid.java",
		2833	"package comment6;\n" +
		2834	"public class Valid {\n" +
		2835	" /**\n" +
		2836	" * @see Valid.Inner\n" +
		2837	" */\n" +
		2838	" public class Inner { }\n" +
		2839	"}\n" +
		2840	"/**\n" +
		2841	" * See also {@link Valid.Inner}\n" +
		2842	" */\n" +
		2843	"class Sub2 extends Valid { }"
		2844	}
		2845	);
		2846	}
		2847	public void testBug96237_Public02() {
		2848	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		2849	runNegativeTest(
		2850	new String[] {
		2851	"comment6/Invalid.java",
		2852	"package comment6;\n" +
		2853	"public class Invalid {\n" +
		2854	" /**\n" +
		2855	" * @see Inner\n" +
		2856	" */\n" +
		2857	" public class Inner { }\n" +
		2858	"}\n" +
		2859	"/**\n" +
		2860	" * See also {@link Inner} \n" +
		2861	" */\n" +
		2862	"class Sub1 extends Invalid { }\n"
		2863	},
		2864	// comment6\Invalid.java:6: warning - Tag @see: reference not found: Inner
		2865	"----------\n" +
		2866	"1. ERROR in comment6\\Invalid.java (at line 4)\r\n" +
		2867	" * @see Inner\r\n" +
		2868	" ^^^^^\n" +
		2869	"Javadoc: Not visible reference\n" +
		2870	"----------\n"
		2871	);
		2872	}
		2873	public void testBug96237_Public03() {
		2874	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		2875	runNegativeTest(
		2876	new String[] {
		2877	"comment6a/def/Test.java",
		2878	"package comment6a.def;\n" +
		2879	"public class Test {\n" +
		2880	" /**\n" +
		2881	" * @see Inner\n" +
		2882	" */\n" +
		2883	" public class Inner { }\n" +
		2884	"}\n",
		2885	"comment6a/test/Invalid.java",
		2886	"package comment6a.test;\n" +
		2887	"import comment6a.def.Test;\n" +
		2888	"/**\n" +
2889	" * See also {@link Inner}\n" +
2890	" */\n" +
2891	"public class Invalid extends Test { \n" +
2892	"}",
2893	"comment6a/test/Invalid2.java",
2894	"package comment6a.test;\n" +
2895	"import comment6a.def.Test;\n" +
2896	"/**\n" +
2897	" * @see Test.Inner\n" +
2898	" */\n" +
2899	"public class Invalid2 extends Test { \n" +
2900	"}"
2901	},
2902	// comment6a\def\Test.java:6: warning - Tag @see: reference not found: Inner
2903	// comment6a\test\Invalid.java:8: warning - Tag @link: reference not found: Inner
2904	// comment6a\test\Invalid2.java:8: warning - Tag @see: reference not found: Test.Inner
2905	"----------\n" +
2906	"1. ERROR in comment6a\\def\\Test.java (at line 4)\n" +
2907	" * @see Inner\n" +
2908	" ^^^^^\n" +
2909	"Javadoc: Not visible reference\n" +
2910	"----------\n" +
2911	"----------\n" +
2912	"1. ERROR in comment6a\\test\\Invalid.java (at line 4)\n" +
2913	" * See also {@link Inner}\n" +
2914	" ^^^^^\n" +
2915	"Javadoc: Not visible reference\n" +
2916	"----------\n" +
2917	"----------\n" +
2918	"1. ERROR in comment6a\\test\\Invalid2.java (at line 4)\n" +
2919	" * @see Test.Inner\n" +
2920	" ^^^^^^^^^^\n" +
2921	"Javadoc: Not visible reference\n" +
2922	"----------\n"
2923	);
2924	}
2925	public void testBug96237_Public04() {
2926	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2927	runNegativeTest(
2928	new String[] {
2929	"comment6b/Invalid.java",
2930	"package comment6b;\n" +
2931	"\n" +
2932	"/**\n" +
2933	" * @see Inner\n" +
2934	" */\n" +
2935	"public class Invalid implements Test { \n" +
2936	"}",
2937	"comment6b/Test.java",
2938	"package comment6b;\n" +
2939	"public interface Test {\n" +
2940	" /**\n" +
2941	" * @see Inner\n" +
2942	" */\n" +
2943	" public class Inner { }\n" +
2944	"}\n",
2945	"comment6b/Valid.java",
2946	"package comment6b;\n" +
2947	"\n" +
2948	"/**\n" +
2949	" * @see Test.Inner\n" +
2950	" */\n" +
2951	"public class Valid implements Test { \n" +
2952	"}"
2953	},
2954	// comment6b\Test.java:6: warning - Tag @see: reference not found: Inner
2955	// comment6b\Invalid.java:6: warning - Tag @see: reference not found: Inner
2956	"----------\n" +
2957	"1. ERROR in comment6b\\Invalid.java (at line 4)\n" +
2958	" * @see Inner\n" +
2959	" ^^^^^\n" +
2960	"Javadoc: Not visible reference\n" +
2961	"----------\n" +
2962	"----------\n" +
2963	"1. ERROR in comment6b\\Test.java (at line 4)\n" +
2964	" * @see Inner\n" +
2965	" ^^^^^\n" +
2966	"Javadoc: Not visible reference\n" +
2967	"----------\n"
2968	);
2969	}
2970	public void testBug96237_Public05() {
2971	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2972	runNegativeTest(
2973	new String[] {
2974	"test/a/Test.java",
2975	"package test.a;\n" +
2976	"/**\n" +
2977	" * @see Inner\n" +
2978	" * @see Test.Inner\n" +
2979	" */\n" +
2980	"public class Test {\n" +
2981	" class Inner {}\n" +
2982	"}\n"
2983	},
2984	// test\a\Test.java:6: warning - Tag @see: reference not found: Inner
2985	// test\a\Test.java:6: warning - Tag @see: reference not found: Test.Inner
2986	"----------\n" +
2987	"1. ERROR in test\\a\\Test.java (at line 3)\n" +
2988	" * @see Inner\n" +
2989	" ^^^^^\n" +
2990	"Javadoc: Not visible reference\n" +
2991	"----------\n" +
2992	"2. ERROR in test\\a\\Test.java (at line 4)\n" +
2993	" * @see Test.Inner\n" +
2994	" ^^^^^^^^^^\n" +
2995	"Javadoc: Not visible reference\n" +
2996	"----------\n"
2997	);
2998	}
2999	public void testBug96237_Public06() {
3000	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
3001	runNegativeTest(
3002	new String[] {
3003	"test/b/Test.java",
3004	"package test.b;\n" +
3005	"/** \n" +
3006	" * @see Inner.Level2\n" +
3007	" * @see Test.Inner.Level2\n" +
3008	" */\n" +
3009	"public class Test {\n" +
3010	" /** \n" +
3011	" * @see Level2\n" +
3012	" * @see Test.Inner.Level2\n" +
3013	" */\n" +
3014	" public class Inner {\n" +
3015	" class Level2 {}\n" +
3016	" }\n" +
3017	"}\n"
3018	},
3019	// test\b\Test.java:6: warning - Tag @see: reference not found: Inner.Level2
3020	// test\b\Test.java:6: warning - Tag @see: reference not found: Test.Inner.Level2
3021	// test\b\Test.java:11: warning - Tag @see: reference not found: Level2
3022	// test\b\Test.java:11: warning - Tag @see: reference not found: Test.Inner.Level2
3023	"----------\n" +
3024	"1. ERROR in test\\b\\Test.java (at line 3)\n" +
3025	" * @see Inner.Level2\n" +
3026	" ^^^^^^^^^^^^\n" +
3027	"Javadoc: Not visible reference\n" +
3028	"----------\n" +
3029	"2. ERROR in test\\b\\Test.java (at line 4)\n" +
3030	" * @see Test.Inner.Level2\n" +
3031	" ^^^^^^^^^^^^^^^^^\n" +
3032	"Javadoc: Not visible reference\n" +
3033	"----------\n" +
3034	"3. ERROR in test\\b\\Test.java (at line 8)\n" +
3035	" * @see Level2\n" +
3036	" ^^^^^^\n" +
3037	"Javadoc: Not visible reference\n" +
3038	"----------\n" +
3039	"4. ERROR in test\\b\\Test.java (at line 9)\n" +
3040	" * @see Test.Inner.Level2\n" +
3041	" ^^^^^^^^^^^^^^^^^\n" +
3042	"Javadoc: Not visible reference\n" +
3043	"----------\n"
3044	);
3045	}
3046	public void testBug96237_Public07() {
3047	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
3048	runNegativeTest(
3049	new String[] {
3050	"test/c/Test.java",
3051	"package test.c;\n" +
3052	"/**\n" +
3053	" * @see Inner.Level2.Level3\n" +
3054	" * @see Test.Inner.Level2.Level3\n" +
3055	" */\n" +
3056	"public class Test {\n" +
3057	" public class Inner {\n" +
3058	" /**\n" +
3059	" * @see Level3\n" +
3060	" * @see Level2.Level3\n" +
3061	" * @see Inner.Level2.Level3\n" +
3062	" * @see Test.Inner.Level2.Level3\n" +
3063	" */\n" +
3064	" public class Level2 {\n" +
3065	" class Level3 {\n" +
3066	" }\n" +
3067	" }\n" +
3068	" }\n" +
3069	"}\n"
3070	},
3071	// test\c\Test.java:6: warning - Tag @see: reference not found: Inner.Level2.Level3
3072	// test\c\Test.java:6: warning - Tag @see: reference not found: Test.Inner.Level2.Level3
3073	// test\c\Test.java:14: warning - Tag @see: reference not found: Level3
3074	// test\c\Test.java:14: warning - Tag @see: reference not found: Level2.Level3
3075	// test\c\Test.java:14: warning - Tag @see: reference not found: Inner.Level2.Level3
3076	// test\c\Test.java:14: warning - Tag @see: reference not found: Test.Inner.Level2.Level3
3077	"----------\n" +
3078	"1. ERROR in test\\c\\Test.java (at line 3)\n" +
3079	" * @see Inner.Level2.Level3\n" +
3080	" ^^^^^^^^^^^^^^^^^^^\n" +
3081	"Javadoc: Not visible reference\n" +
3082	"----------\n" +
3083	"2. ERROR in test\\c\\Test.java (at line 4)\n" +
3084	" * @see Test.Inner.Level2.Level3\n" +
3085	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3086	"Javadoc: Not visible reference\n" +
3087	"----------\n" +
3088	"3. ERROR in test\\c\\Test.java (at line 9)\n" +
3089	" * @see Level3\n" +
3090	" ^^^^^^\n" +
3091	"Javadoc: Not visible reference\n" +
3092	"----------\n" +
3093	"4. ERROR in test\\c\\Test.java (at line 10)\n" +
3094	" * @see Level2.Level3\n" +
3095	" ^^^^^^^^^^^^^\n" +
3096	"Javadoc: Not visible reference\n" +
3097	"----------\n" +
3098	"5. ERROR in test\\c\\Test.java (at line 11)\n" +
3099	" * @see Inner.Level2.Level3\n" +
3100	" ^^^^^^^^^^^^^^^^^^^\n" +
3101	"Javadoc: Not visible reference\n" +
3102	"----------\n" +
3103	"6. ERROR in test\\c\\Test.java (at line 12)\n" +
3104	" * @see Test.Inner.Level2.Level3\n" +
3105	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3106	"Javadoc: Not visible reference\n" +
3107	"----------\n"
3108	);
3109	}
3110	public void testBug96237_Private01() {
3111	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3112	runConformTest(
3113	new String[] {
3114	"comment6/Valid.java",
3115	"package comment6;\n" +
3116	"public class Valid {\n" +
3117	" /**\n" +
3118	" * @see Valid.Inner\n" +
3119	" */\n" +
3120	" public class Inner { }\n" +
3121	"}\n" +
3122	"/**\n" +
3123	" * See also {@link Valid.Inner}\n" +
3124	" */\n" +
3125	"class Sub2 extends Valid { }"
3126	}
3127	);
3128	}
3129	public void testBug96237_Private02() {
3130	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3131	runNegativeTest(
3132	new String[] {
3133	"comment6/Invalid.java",
3134	"package comment6;\n" +
3135	"public class Invalid {\n" +
3136	" /**\n" +
3137	" * @see Inner\n" +
3138	" */\n" +
3139	" public class Inner { }\n" +
3140	"}\n" +
3141	"/**\n" +
3142	" * See also {@link Inner} \n" +
3143	" */\n" +
3144	"class Sub1 extends Invalid { }\n"
3145	},
3146	// comment6\Invalid.java:6: warning - Tag @see: reference not found: Inner
3147	// comment6\Invalid.java:11: warning - Tag @link: reference not found: Inner
3148	"----------\n" +
3149	"1. ERROR in comment6\\Invalid.java (at line 4)\n" +
3150	" * @see Inner\n" +
3151	" ^^^^^\n" +
3152	"Javadoc: Not visible reference\n" +
3153	"----------\n" +
3154	"2. ERROR in comment6\\Invalid.java (at line 9)\n" +
3155	" * See also {@link Inner} \n" +
3156	" ^^^^^\n" +
3157	"Javadoc: Not visible reference\n" +
3158	"----------\n"
3159	);
3160	}
3161	public void testBug96237_Private03() {
3162	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3163	runNegativeTest(
3164	new String[] {
3165	"comment6a/def/Test.java",
3166	"package comment6a.def;\n" +
3167	"public class Test {\n" +
3168	" /**\n" +
3169	" * @see Inner\n" +
3170	" */\n" +
3171	" public class Inner { }\n" +
3172	"}\n",
3173	"comment6a/test/Invalid.java",
3174	"package comment6a.test;\n" +
3175	"import comment6a.def.Test;\n" +
3176	"/**\n" +
3177	" * See also {@link Inner}\n" +
3178	" */\n" +
3179	"public class Invalid extends Test { \n" +
3180	"}",
3181	"comment6a/test/Invalid2.java",
3182	"package comment6a.test;\n" +
3183	"import comment6a.def.Test;\n" +
3184	"/**\n" +
3185	" * @see Test.Inner\n" +
3186	" */\n" +
3187	"public class Invalid2 extends Test { \n" +
3188	"}"
3189	},
3190	// comment6a\def\Test.java:6: warning - Tag @see: reference not found: Inner
3191	// comment6a\test\Invalid.java:8: warning - Tag @link: reference not found: Inner
3192	// comment6a\test\Invalid2.java:8: warning - Tag @see: reference not found: Test.Inner
3193	"----------\n" +
3194	"1. ERROR in comment6a\\def\\Test.java (at line 4)\n" +
3195	" * @see Inner\n" +
3196	" ^^^^^\n" +
3197	"Javadoc: Not visible reference\n" +
3198	"----------\n" +
3199	"----------\n" +
3200	"1. ERROR in comment6a\\test\\Invalid.java (at line 4)\n" +
3201	" * See also {@link Inner}\n" +
3202	" ^^^^^\n" +
3203	"Javadoc: Not visible reference\n" +
3204	"----------\n" +
3205	"----------\n" +
3206	"1. ERROR in comment6a\\test\\Invalid2.java (at line 4)\n" +
3207	" * @see Test.Inner\n" +
3208	" ^^^^^^^^^^\n" +
3209	"Javadoc: Not visible reference\n" +
3210	"----------\n"
3211	);
3212	}
3213	public void testBug96237_Private04() {
3214	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3215	runNegativeTest(
3216	new String[] {
3217	"comment6b/Invalid.java",
3218	"package comment6b;\n" +
3219	"\n" +
3220	"/**\n" +
3221	" * @see Inner\n" +
3222	" */\n" +
3223	"public class Invalid implements Test { \n" +
3224	"}",
3225	"comment6b/Test.java",
3226	"package comment6b;\n" +
3227	"public interface Test {\n" +
3228	" /**\n" +
3229	" * @see Inner\n" +
3230	" */\n" +
3231	" public class Inner { }\n" +
3232	"}\n",
3233	"comment6b/Valid.java",
3234	"package comment6b;\n" +
3235	"\n" +
3236	"/**\n" +
3237	" * @see Test.Inner\n" +
3238	" */\n" +
3239	"public class Valid implements Test { \n" +
3240	"}"
3241	},
3242	// comment6b\Invalid.java:6: warning - Tag @see: reference not found: Inner
3243	// comment6b\Test.java:6: warning - Tag @see: reference not found: Inner
3244	"----------\n" +
3245	"1. ERROR in comment6b\\Invalid.java (at line 4)\n" +
3246	" * @see Inner\n" +
3247	" ^^^^^\n" +
3248	"Javadoc: Not visible reference\n" +
3249	"----------\n" +
3250	"----------\n" +
3251	"1. ERROR in comment6b\\Test.java (at line 4)\n" +
3252	" * @see Inner\n" +
3253	" ^^^^^\n" +
3254	"Javadoc: Not visible reference\n" +
3255	"----------\n"
3256	);
3257	}
3258	public void testBug96237_Private05() {
3259	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3260	runConformTest(
3261	new String[] {
3262	"test/a/Test.java",
3263	"package test.a;\n" +
3264	"/**\n" +
3265	" * @see Inner\n" +
3266	" * @see Test.Inner\n" +
3267	" */\n" +
3268	"public class Test {\n" +
3269	" class Inner {}\n" +
3270	"}\n"
3271	}
3272	);
3273	}
3274	public void testBug96237_Private06() {
3275	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3276	runConformTest(
3277	new String[] {
3278	"test/b/Test.java",
3279	"package test.b;\n" +
3280	"/** \n" +
3281	" * @see Inner.Level2\n" +
3282	" * @see Test.Inner.Level2\n" +
3283	" */\n" +
3284	"public class Test {\n" +
3285	" /** \n" +
3286	" * @see Level2\n" +
3287	" * @see Test.Inner.Level2\n" +
3288	" */\n" +
3289	" public class Inner {\n" +
3290	" class Level2 {}\n" +
3291	" }\n" +
3292	"}\n"
3293	}
3294	);
3295	}
3296	public void testBug96237_Private07() {
3297	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3298	runConformTest(
3299	new String[] {
3300	"test/c/Test.java",
3301	"package test.c;\n" +
3302	"/**\n" +
3303	" * @see Inner.Level2.Level3\n" +
3304	" * @see Test.Inner.Level2.Level3\n" +
3305	" */\n" +
3306	"public class Test {\n" +
3307	" public class Inner {\n" +
3308	" /**\n" +
3309	" * @see Level3\n" +
3310	" * @see Level2.Level3\n" +
3311	" * @see Inner.Level2.Level3\n" +
3312	" * @see Test.Inner.Level2.Level3\n" +
3313	" */\n" +
3314	" public class Level2 {\n" +
3315	" class Level3 {\n" +
3316	" }\n" +
3317	" }\n" +
3318	" }\n" +
3319	"}\n"
3320	}
3321	);
3322	}
3323
3324	/**
2820	* Bug 101283: [1.5][javadoc] Javadoc validation raises missing implementation in compiler	3325	* Bug 101283: [1.5][javadoc] Javadoc validation raises missing implementation in compiler
2821	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=101283"	3326	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=101283"
2822	*/	3327	*/
Lines 3013-3018 Link Here
3013	}	3518	}
3014		3519
3015	/**	3520	/**
		3521	* Bug 103304: [Javadoc] Wrong reference proposal for inner classes.
		3522	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=103304"
		3523	*/
		3524	public void testBug103304a() {
		3525	runNegativeTest(
		3526	new String[] {
		3527	"boden/IAFAState.java",
		3528	"package boden;\n" +
		3529	"public interface IAFAState {\n" +
		3530	" public class ValidationException extends Exception {\n" +
		3531	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		3532	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		3533	" }\n" +
		3534	" public void method() {}\n" +
		3535	" }\n" +
		3536	" /**\n" +
		3537	" * Validates a formula for consistent bindings. Bindings are consistent, when at each point in time,\n" +
		3538	" * the set of povided variables can be guaranteed to be a superset of the set of required variables.\n" +
		3539	" * @throws ValidationException Thrown if a variable is unbound. \n" +
		3540	" * @see ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3541	" * @see IAFAState.ValidationException#method()\n" +
		3542	" * @see ValidationException\n" +
		3543	" * {@link ValidationException}\n" +
		3544	" */\n" +
		3545	" public void validate() throws ValidationException;\n" +
		3546	"}\n",
		3547	"boden/TestValid.java",
		3548	"package boden;\n" +
		3549	"import boden.IAFAState.ValidationException;\n" +
		3550	"/**\n" +
		3551	" * @see ValidationException\n" +
		3552	" * @see IAFAState.ValidationException\n" +
		3553	" */\n" +
		3554	"public class TestValid {\n" +
		3555	" /** \n" +
		3556	" * @see ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3557	" */\n" +
		3558	" IAFAState.ValidationException valid1;\n" +
		3559	" /**\n" +
		3560	" * @see IAFAState.ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3561	" */\n" +
		3562	" IAFAState.ValidationException valid2;\n" +
		3563	"}\n"
		3564	},
		3565	// boden\TestValid.java:8: warning - Tag @see: reference not found: ValidationException
		3566	// boden\TestValid.java:12: warning - Tag @see: reference not found: ValidationException#IAFAState.ValidationException(String, IAFAState)
		3567	"----------\n" +
		3568	"1. ERROR in boden\\TestValid.java (at line 4)\r\n" +
		3569	" * @see ValidationException\r\n" +
		3570	" ^^^^^^^^^^^^^^^^^^^\n" +
		3571	"Javadoc: Not visible reference\n" +
		3572	"----------\n"
		3573	);
		3574	}
		3575	public void testBug103304b() {
		3576	runNegativeTest(
		3577	new String[] {
		3578	"boden/IAFAState.java",
		3579	"package boden;\n" +
		3580	"public interface IAFAState {\n" +
		3581	" public class ValidationException extends Exception {\n" +
		3582	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		3583	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		3584	" }\n" +
3585	" public void method() {}\n" +
3586	" }\n" +
3587	"}\n",
3588	"boden/TestInvalid1.java",
3589	"package boden;\n" +
3590	"import boden.IAFAState.ValidationException;\n" +
3591	"public class TestInvalid1 {\n" +
3592	" /** \n" +
3593	" * @see ValidationException#ValidationException(String, IAFAState)\n" +
3594	" */ \n" +
3595	" IAFAState.ValidationException invalid;\n" +
3596	"}\n",
3597	"boden/TestInvalid2.java",
3598	"package boden;\n" +
3599	"import boden.IAFAState.ValidationException;\n" +
3600	"public class TestInvalid2 {\n" +
3601	" /**\n" +
3602	" * @see IAFAState.ValidationException#ValidationException(String, IAFAState)\n" +
3603	" */\n" +
3604	" IAFAState.ValidationException invalid;\n" +
3605	"}\n",
3606	"boden/TestInvalid3.java",
3607	"package boden;\n" +
3608	"import boden.IAFAState.ValidationException;\n" +
3609	"public class TestInvalid3 {\n" +
3610	" /**\n" +
3611	" * @see IAFAState.ValidationException#IAFA.State.ValidationException(String, IAFAState)\n" +
3612	" */\n" +
3613	" IAFAState.ValidationException invalid;\n" +
3614	"}\n",
3615	"boden/TestInvalid4.java",
3616	"package boden;\n" +
3617	"import boden.IAFAState.ValidationException;\n" +
3618	"public class TestInvalid4 {\n" +
3619	" /**\n" +
3620	" * @see IAFAState.ValidationException#IAFAState .ValidationException(String, IAFAState)\n" +
3621	" */\n" +
3622	" IAFAState.ValidationException invalid;\n" +
3623	"}\n"
3624	},
3625	// boden\TestInvalid1.java:7: warning - Tag @see: reference not found: ValidationException#ValidationException(String, IAFAState)
3626	// boden\TestInvalid2.java:6: warning - Tag @see: can't find ValidationException(String, IAFAState) in boden.IAFAState.ValidationException
3627	// boden\TestInvalid3.java:6: warning - Tag @see: can't find IAFA.State.ValidationException(String, IAFAState) in boden.IAFAState.ValidationException
3628	// boden\TestInvalid4.java:6: warning - Tag @see: can't find IAFAState in boden.IAFAState.ValidationException
3629	"----------\n" +
3630	"1. ERROR in boden\\TestInvalid1.java (at line 5)\n" +
3631	" * @see ValidationException#ValidationException(String, IAFAState)\n" +
3632	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3633	"Javadoc: Invalid qualification for member type constructor\n" +
3634	"----------\n" +
3635	"----------\n" +
3636	"1. ERROR in boden\\TestInvalid2.java (at line 5)\n" +
3637	" * @see IAFAState.ValidationException#ValidationException(String, IAFAState)\n" +
3638	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3639	"Javadoc: Invalid qualification for member type constructor\n" +
3640	"----------\n" +
3641	"----------\n" +
3642	"1. ERROR in boden\\TestInvalid3.java (at line 5)\n" +
3643	" * @see IAFAState.ValidationException#IAFA.State.ValidationException(String, IAFAState)\n" +
3644	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3645	"Javadoc: Invalid qualification for member type constructor\n" +
3646	"----------\n" +
3647	"----------\n" +
3648	"1. ERROR in boden\\TestInvalid4.java (at line 5)\n" +
3649	" * @see IAFAState.ValidationException#IAFAState .ValidationException(String, IAFAState)\n" +
3650	" ^^^^^^^^^\n" +
3651	"Javadoc: IAFAState cannot be resolved or is not a field\n" +
3652	"----------\n"
3653	);
3654	}
3655	public void testBug103304c() {
3656	runNegativeTest(
3657	new String[] {
3658	"test/Test.java",
3659	"package test;\n" +
3660	"public interface Test {\n" +
3661	" public class Level0 {\n" +
3662	" public Level0() {}\n" +
3663	" }\n" +
3664	" public interface Member {\n" +
3665	" public class Level1 {\n" +
3666	" public Level1() {}\n" +
3667	" }\n" +
3668	" }\n" +
3669	"}\n",
3670	"test/C.java",
3671	"package test;\n" +
3672	"public class C {\n" +
3673	" /**\n" +
3674	" * @see Test.Level0#Test.Level0()\n" +
3675	" */\n" +
3676	" Test.Level0 valid = new Test.Level0();\n" +
3677	" /**\n" +
3678	" * @see Test.Level0#Level0()\n" +
3679	" */\n" +
3680	" Test.Level0 invalid = new Test.Level0();\n" +
3681	"}\n"
3682	},
3683	// test\C.java:10: warning - Tag @see: can't find Level0() in test.Test.Level0
3684	"----------\n" +
3685	"1. ERROR in test\\C.java (at line 8)\n" +
3686	" * @see Test.Level0#Level0()\n" +
3687	" ^^^^^^^^\n" +
3688	"Javadoc: Invalid qualification for member type constructor\n" +
3689	"----------\n"
3690	);
3691	}
3692	public void testBug103304d() {
3693	runNegativeTest(
3694	new String[] {
3695	"test/Test.java",
3696	"package test;\n" +
3697	"public interface Test {\n" +
3698	" public class Level0 {\n" +
3699	" public Level0() {}\n" +
3700	" }\n" +
3701	" public interface Member {\n" +
3702	" public class Level1 {\n" +
3703	" public Level1() {}\n" +
3704	" }\n" +
3705	" }\n" +
3706	"}\n",
3707	"test/C2.java",
3708	"package test;\n" +
3709	"public class C2 {\n" +
3710	" /**\n" +
3711	" * @see Test.Member.Level1#Test.Member.Level1()\n" +
3712	" */\n" +
3713	" Test.Member.Level1 valid = new Test.Member.Level1();\n" +
3714	" /**\n" +
3715	" * @see Test.Member.Level1#Level1()\n" +
3716	" */\n" +
3717	" Test.Member.Level1 invalid = new Test.Member.Level1();\n" +
3718	" /**\n" +
3719	" * @see Test.Member.Level1#Test.Level1()\n" +
3720	" */\n" +
3721	" Test.Member.Level1 wrong = new Test.Member.Level1();\n" +
3722	"}\n"
3723	},
3724	// test\C2.java:10: warning - Tag @see: can't find Level1() in test.Test.Member.Level1
3725	// test\C2.java:14: warning - Tag @see: can't find Test.Level1() in test.Test.Member.Level1
3726	"----------\n" +
3727	"1. ERROR in test\\C2.java (at line 8)\n" +
3728	" * @see Test.Member.Level1#Level1()\n" +
3729	" ^^^^^^^^\n" +
3730	"Javadoc: Invalid qualification for member type constructor\n" +
3731	"----------\n" +
3732	"2. ERROR in test\\C2.java (at line 12)\n" +
3733	" * @see Test.Member.Level1#Test.Level1()\n" +
3734	" ^^^^^^^^^^^^^\n" +
3735	"Javadoc: Invalid qualification for member type constructor\n" +
3736	"----------\n"
3737	);
3738	}
3739	public void testBug103304e() {
3740	runConformTest(
3741	new String[] {
3742	"implicit/Valid.java",
3743	"package implicit;\n" +
3744	"public interface Valid {\n" +
3745	" public class Level0 {\n" +
3746	" /**\n" +
3747	" * @see #Valid.Level0() Valid\n" +
3748	" */\n" +
3749	" public Level0() {}\n" +
3750	" /**\n" +
3751	" * @see #Valid.Level0(String) Valid\n" +
3752	" */\n" +
3753	" public Level0(String str) {}\n" +
3754	" }\n" +
3755	" public interface Member {\n" +
3756	" public class Level1 {\n" +
3757	" /**\n" +
3758	" * @see #Valid.Member.Level1() Valid\n" +
3759	" */\n" +
3760	" public Level1() {}\n" +
3761	" /**\n" +
3762	" * @see #Valid.Member.Level1(int) Valid\n" +
3763	" */\n" +
3764	" public Level1(int x) {}\n" +
3765	" }\n" +
3766	" }\n" +
3767	"}\n"
3768	}
3769	);
3770	}
3771	public void testBug103304f() {
3772	runNegativeTest(
3773	new String[] {
3774	"implicit/Invalid.java",
3775	"package implicit;\n" +
3776	"public interface Invalid {\n" +
3777	" public class Level0 {\n" +
3778	" /**\n" +
3779	" * @see #Level0() Invalid\n" +
3780	" */\n" +
3781	" public Level0() {}\n" +
3782	" /**\n" +
3783	" * @see #Level0(String) Invalid\n" +
3784	" */\n" +
3785	" public Level0(String str) {}\n" +
3786	" }\n" +
3787	" public interface Member {\n" +
3788	" public class Level1 {\n" +
3789	" /**\n" +
3790	" * @see #Level1() Invalid\n" +
3791	" * @see #Member.Level1() Invalid\n" +
3792	" * @see #Invalid.Level1() Invalid\n" +
3793	" */\n" +
3794	" public Level1() {}\n" +
3795	" /**\n" +
3796	" * @see #Level1(int) Invalid\n" +
3797	" * @see #Invalid.Level1(int) Invalid\n" +
3798	" * @see #Member.Level1(int) Invalid\n" +
3799	" */\n" +
3800	" public Level1(int x) {}\n" +
3801	" }\n" +
3802	" }\n" +
3803	"}\n"
3804	},
3805	// implicit\Invalid.java:7: warning - Tag @see: can't find Level0() in implicit.Invalid.Level0
3806	// implicit\Invalid.java:11: warning - Tag @see: can't find Level0(String) in implicit.Invalid.Level0
3807	// implicit\Invalid.java:20: warning - Tag @see: can't find Level1() in implicit.Invalid.Member.Level1
3808	// implicit\Invalid.java:20: warning - Tag @see: can't find Member.Level1() in implicit.Invalid.Member.Level1
3809	// implicit\Invalid.java:20: warning - Tag @see: can't find Invalid.Level1() in implicit.Invalid.Member.Level1
3810	// implicit\Invalid.java:26: warning - Tag @see: can't find Level1(int) in implicit.Invalid.Member.Level1
3811	// implicit\Invalid.java:26: warning - Tag @see: can't find Invalid.Level1(int) in implicit.Invalid.Member.Level1
3812	// implicit\Invalid.java:26: warning - Tag @see: can't find Member.Level1(int) in implicit.Invalid.Member.Level1
3813	"----------\n" +
3814	"1. ERROR in implicit\\Invalid.java (at line 5)\n" +
3815	" * @see #Level0() Invalid\n" +
3816	" ^^^^^^^^\n" +
3817	"Javadoc: Invalid qualification for member type constructor\n" +
3818	"----------\n" +
3819	"2. ERROR in implicit\\Invalid.java (at line 9)\n" +
3820	" * @see #Level0(String) Invalid\n" +
3821	" ^^^^^^^^^^^^^^\n" +
3822	"Javadoc: Invalid qualification for member type constructor\n" +
3823	"----------\n" +
3824	"3. ERROR in implicit\\Invalid.java (at line 16)\n" +
3825	" * @see #Level1() Invalid\n" +
3826	" ^^^^^^^^\n" +
3827	"Javadoc: Invalid qualification for member type constructor\n" +
3828	"----------\n" +
3829	"4. ERROR in implicit\\Invalid.java (at line 17)\n" +
3830	" * @see #Member.Level1() Invalid\n" +
3831	" ^^^^^^^^^^^^^^^\n" +
3832	"Javadoc: Invalid qualification for member type constructor\n" +
3833	"----------\n" +
3834	"5. ERROR in implicit\\Invalid.java (at line 18)\n" +
3835	" * @see #Invalid.Level1() Invalid\n" +
3836	" ^^^^^^^^^^^^^^^^\n" +
3837	"Javadoc: Invalid qualification for member type constructor\n" +
3838	"----------\n" +
3839	"6. ERROR in implicit\\Invalid.java (at line 22)\n" +
3840	" * @see #Level1(int) Invalid\n" +
3841	" ^^^^^^^^^^^\n" +
3842	"Javadoc: Invalid qualification for member type constructor\n" +
3843	"----------\n" +
3844	"7. ERROR in implicit\\Invalid.java (at line 23)\n" +
3845	" * @see #Invalid.Level1(int) Invalid\n" +
3846	" ^^^^^^^^^^^^^^^^^^^\n" +
3847	"Javadoc: Invalid qualification for member type constructor\n" +
3848	"----------\n" +
3849	"8. ERROR in implicit\\Invalid.java (at line 24)\n" +
3850	" * @see #Member.Level1(int) Invalid\n" +
3851	" ^^^^^^^^^^^^^^^^^^\n" +
3852	"Javadoc: Invalid qualification for member type constructor\n" +
3853	"----------\n"
3854	);
3855	}
3856
3857	/**
3016	* Bug 112346: [javadoc] {@inheritedDoc} should be inactive for non-overridden method	3858	* Bug 112346: [javadoc] {@inheritedDoc} should be inactive for non-overridden method
3017	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=112346"	3859	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=112346"
3018	*/	3860	*/




		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"import test.copy.*;\n" +
				"public class X {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public void s_foo() {\n" +
				"	}\n" +
				"}\n"
			}
		);
	}

	public void test093() {

		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"public class X {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public void s_foo() {\n" +
				"	}\n" +
				"}\n"
			}
		);
	}

	public void test095() {




		options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsDeprecatedRef, CompilerOptions.ENABLED);
		options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsNotVisibleRef, CompilerOptions.ENABLED);
		options.put(CompilerOptions.OPTION_ReportMissingJavadocTagsVisibility, CompilerOptions.PRIVATE);
		options.put(CompilerOptions.OPTION_ReportMissingSerialVersion, CompilerOptions.IGNORE);
		return options;
	}
	

		referencedClasses =
			new String[] {
				"test/AbstractVisibility.java",
				"package test;\n" +
				"public abstract class AbstractVisibility {\n" +
				"	private class AvcPrivate {\n" +
				"		private int avf_private = 10;\n" +
				"		public int avf_public = avf_private;\n" +
				"		private int avm_private() {\n" +
				"			avf_private = (new AvcPrivate()).avf_private;\n" +
				"			return avf_private;\n" +
				"		}\n" +
				"		public int avm_public() {\n" +
				"			return avm_private();\n" +
				"		}\n" +
				"	}\n" +
				"	public class AvcPublic {\n" +
				"		private int avf_private = 10;\n" +
				"		public int avf_public = avf_private;\n" +
				"		private int avm_private() {\n" +
				"			avf_private = (new AvcPrivate()).avf_private;\n" +
				"			return avf_private;\n" +
				"		}\n" +
				"		public int avm_public() {\n" +
				"			return avm_private();\n" +
				"		}\n" +
				"	}\n" +
				"	private int avf_private = 100;\n" +
				"	public int avf_public = avf_private;\n" +
				"	\n" +
				"	private int avm_private() {\n" +
				"		avf_private = (new AvcPrivate()).avf_private;\n" +
				"		return avf_private;\n" +
				"	}\n" +
				"	public int avm_public() {\n" +
				"		return avm_private();\n" +
				"	}\n" +
				"}\n",
				"test/Visibility.java",
				"package test;\n" +
				"public class Visibility extends AbstractVisibility {\n" +
				"	private class VcPrivate {\n" +
				"		private int vf_private = 10;\n" +
				"		public int vf_public = vf_private;\n" +
				"		private int vm_private() {\n" +
				"			vf_private = (new VcPrivate()).vf_private;\n" +
				"			avf_private = vf_private;\n" +
				"			return vf_private+avf_private;\n" +
				"		}\n" +
				"		public int vm_public() {\n" +
				"			return vm_private();\n" +
				"		}\n" +
				"	};\n" +
				"	public class VcPublic {\n" +
				"		private int vf_private = 10;\n" +
				"		public int vf_public = vf_private;\n" +
				"		private int vm_private() {\n" +
				"			vf_private = (new VcPrivate()).vf_private;\n" +
				"			avf_private = vf_private;\n" +
				"			return vf_private+avf_private;\n" +
				"		}\n" +
				"		public int vm_public() {\n" +
				"			return vm_private();\n" +
				"		}\n" +
				"	};\n" +
				"	private int vf_private = 100;\n" +
				"	private int avf_private = 100;\n" +
				"	public int vf_public = vf_private;\n" +
				"	public int avf_public = vf_private;\n" +
				"	\n" +
				"	private int vm_private() {\n" +
				"		vf_private = (new VcPrivate()).vf_private;\n" +
				"		avf_private = vf_private;\n" +
				"		return vf_private+avf_private;\n" +
				"	}\n" +
				"	public int vm_public() {\n" +
				"		return vm_private();\n" +
				"	}\n" +
				"}\n",
				"test/copy/VisibilityPackage.java",
				"package test.copy;\n" +
				"class VisibilityPackage {\n" +
				"	private class VpPrivate {\n" +
				"		private int vf_private = 10;\n" +
				"		public int vf_public = vf_private;\n" +
				"		private int vm_private() {\n" +
				"			vf_private = (new VpPrivate()).vf_private;\n" +
				"			return vf_private;\n" +
				"		}\n" +
				"		public int vm_public() {\n" +
				"			return vm_private();\n" +
				"		}\n" +
				"	}\n" +
				"	public class VpPublic {\n" +
				"		private int vf_private = 10;\n" +
				"		public int vf_public = vf_private;\n" +
				"		private int vm_private() {\n" +
				"			vf_private = (new VpPrivate()).vf_private;\n" +
				"			return vf_private;\n" +
				"		}\n" +
				"		public int vm_public() {\n" +
				"			return vm_private();\n" +
				"		}\n" +
				"	}\n" +
				"	private int vf_private = 100;\n" +
				"	public int vf_public = vf_private;\n" +
				"	\n" +
				"	private int vm_private() {\n" +
				"		vf_private = (new VpPrivate()).vf_private;\n" +
				"		return vf_private;\n" +
				"	}\n" +
				"	public int vm_public() {\n" +
				"		return vm_private();\n" +
				"	}\n" +
				"}\n",
				"test/copy/VisibilityPublic.java",
				"package test.copy;\n" +
				"public class VisibilityPublic {\n" +
				"	private class VpPrivate {\n" +
				"		private int vf_private = 10;\n" +
				"		public int vf_public = vf_private;\n" +
				"		private int vm_private() {\n" +
				"			vf_private = (new VpPrivate()).vf_private;\n" +
				"			return vf_private;\n" +
				"		}\n" +
				"		public int vm_public() {\n" +
				"			return vm_private();\n" +
				"		}\n" +
				"	}\n" +
				"	public class VpPublic {\n" +
				"		private int vf_private = 10;\n" +
				"		public int vf_public = vf_private;\n" +
				"		private int vm_private() {\n" +
				"			vf_private = (new VpPrivate()).vf_private;\n" +
				"			return vf_private;\n" +
				"		}\n" +
				"		public int vm_public() {\n" +
				"			return vm_private();\n" +
				"		}\n" +
				"	}\n" +
				"	private int vf_private = 100;\n" +
				"	public int vf_public = vf_private;\n" +
				"	\n" +
				"	private int vm_private() {\n" +
				"		vf_private = (new VpPrivate()).vf_private;\n" +
				"		return vf_private;\n" +
				"	}\n" +
				"	public int vm_public() {\n" +
				"		return vm_private();\n" +
				"	}\n" +
				"}\n" };
	}
	/* (non-Javadoc)
	 * @see junit.framework.TestCase#setUp()




		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"import test.copy.*;\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"public class X {\n" +
				"	public void s_foo() {\n" +
				"	}\n" +
				"}\n"
			}
		);
	}

	public void test023() {

		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"public class X {\n" +
				"	public void s_foo() {\n" +
				"	}\n" +
				"}\n"
				}
		);
	}

	// @see Field references

Lines 20-25 Link Here

(-)src/org/eclipse/jdt/core/tests/compiler/regression/JavadocTest_1_3.java (-3 / +844 lines)
20		20
21	String docCommentSupport = CompilerOptions.ENABLED;	21	String docCommentSupport = CompilerOptions.ENABLED;
22	String reportInvalidJavadoc = CompilerOptions.ERROR;	22	String reportInvalidJavadoc = CompilerOptions.ERROR;
		23	String reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
23	String reportMissingJavadocTags = CompilerOptions.ERROR;	24	String reportMissingJavadocTags = CompilerOptions.ERROR;
24	String reportMissingJavadocComments = null;	25	String reportMissingJavadocComments = null;
25	String reportMissingJavadocCommentsVisibility = null;	26	String reportMissingJavadocCommentsVisibility = null;
Lines 49-54 Link Here
49	Map options = super.getCompilerOptions();	50	Map options = super.getCompilerOptions();
50	options.put(CompilerOptions.OPTION_DocCommentSupport, this.docCommentSupport);	51	options.put(CompilerOptions.OPTION_DocCommentSupport, this.docCommentSupport);
51	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);	52	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);
		53	if (!CompilerOptions.IGNORE.equals(reportInvalidJavadoc)) {
		54	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility, this.reportInvalidJavadocVisibility);
		55	}
52	if (reportMissingJavadocComments != null)	56	if (reportMissingJavadocComments != null)
53	options.put(CompilerOptions.OPTION_ReportMissingJavadocComments, reportMissingJavadocComments);	57	options.put(CompilerOptions.OPTION_ReportMissingJavadocComments, reportMissingJavadocComments);
54	else	58	else
Lines 71-79 Link Here
71	protected void setUp() throws Exception {	75	protected void setUp() throws Exception {
72	super.setUp();	76	super.setUp();
73	this.docCommentSupport = CompilerOptions.ENABLED;	77	this.docCommentSupport = CompilerOptions.ENABLED;
74	reportInvalidJavadoc = CompilerOptions.ERROR;	78	this.reportInvalidJavadoc = CompilerOptions.ERROR;
75	reportMissingJavadocTags = CompilerOptions.ERROR;	79	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
76	reportMissingJavadocComments = CompilerOptions.IGNORE;	80	this.reportMissingJavadocTags = CompilerOptions.ERROR;
		81	this.reportMissingJavadocComments = CompilerOptions.IGNORE;
77	}	82	}
78		83
79	/**	84	/**
Lines 2546-2551 Link Here
2546	""	2551	""
2547	);	2552	);
2548	}	2553	}
		2554	/**
		2555	* Bug 96237: [javadoc] Inner types must be qualified
		2556	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=96237"
		2557	*/
		2558	public void testBug96237_Public01() {
		2559	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		2560	runConformTest(
		2561	new String[] {
		2562	"comment6/Valid.java",
		2563	"package comment6;\n" +
		2564	"public class Valid {\n" +
		2565	" /**\n" +
		2566	" * @see Valid.Inner\n" +
		2567	" */\n" +
		2568	" public class Inner { }\n" +
		2569	"}\n" +
		2570	"/**\n" +
		2571	" * See also {@link Valid.Inner}\n" +
		2572	" */\n" +
		2573	"class Sub2 extends Valid { }"
		2574	}
		2575	);
		2576	}
		2577	public void testBug96237_Public02() {
		2578	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		2579	runNegativeTest(
		2580	new String[] {
		2581	"comment6/Invalid.java",
		2582	"package comment6;\n" +
		2583	"public class Invalid {\n" +
		2584	" /**\n" +
		2585	" * @see Inner\n" +
		2586	" */\n" +
		2587	" public class Inner { }\n" +
		2588	"}\n" +
		2589	"/**\n" +
		2590	" * See also {@link Inner} \n" +
		2591	" */\n" +
		2592	"class Sub1 extends Invalid { }\n"
		2593	},
		2594	// comment6\Invalid.java:6: warning - Tag @see: reference not found: Inner
		2595	"----------\n" +
		2596	"1. ERROR in comment6\\Invalid.java (at line 4)\r\n" +
		2597	" * @see Inner\r\n" +
		2598	" ^^^^^\n" +
		2599	"Javadoc: Not visible reference\n" +
		2600	"----------\n"
		2601	);
		2602	}
		2603	public void testBug96237_Public03() {
		2604	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		2605	runNegativeTest(
		2606	new String[] {
		2607	"comment6a/def/Test.java",
		2608	"package comment6a.def;\n" +
		2609	"public class Test {\n" +
		2610	" /**\n" +
		2611	" * @see Inner\n" +
		2612	" */\n" +
		2613	" public class Inner { }\n" +
		2614	"}\n",
		2615	"comment6a/test/Invalid.java",
		2616	"package comment6a.test;\n" +
		2617	"import comment6a.def.Test;\n" +
2618	"/**\n" +
2619	" * See also {@link Inner}\n" +
2620	" */\n" +
2621	"public class Invalid extends Test { \n" +
2622	"}",
2623	"comment6a/test/Invalid2.java",
2624	"package comment6a.test;\n" +
2625	"import comment6a.def.Test;\n" +
2626	"/**\n" +
2627	" * @see Test.Inner\n" +
2628	" */\n" +
2629	"public class Invalid2 extends Test { \n" +
2630	"}"
2631	},
2632	// comment6a\def\Test.java:6: warning - Tag @see: reference not found: Inner
2633	// comment6a\test\Invalid.java:8: warning - Tag @link: reference not found: Inner
2634	// comment6a\test\Invalid2.java:8: warning - Tag @see: reference not found: Test.Inner
2635	"----------\n" +
2636	"1. ERROR in comment6a\\def\\Test.java (at line 4)\n" +
2637	" * @see Inner\n" +
2638	" ^^^^^\n" +
2639	"Javadoc: Not visible reference\n" +
2640	"----------\n" +
2641	"----------\n" +
2642	"1. ERROR in comment6a\\test\\Invalid.java (at line 4)\n" +
2643	" * See also {@link Inner}\n" +
2644	" ^^^^^\n" +
2645	"Javadoc: Not visible reference\n" +
2646	"----------\n" +
2647	"----------\n" +
2648	"1. ERROR in comment6a\\test\\Invalid2.java (at line 4)\n" +
2649	" * @see Test.Inner\n" +
2650	" ^^^^^^^^^^\n" +
2651	"Javadoc: Not visible reference\n" +
2652	"----------\n"
2653	);
2654	}
2655	public void testBug96237_Public04() {
2656	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2657	runNegativeTest(
2658	new String[] {
2659	"comment6b/Invalid.java",
2660	"package comment6b;\n" +
2661	"\n" +
2662	"/**\n" +
2663	" * @see Inner\n" +
2664	" */\n" +
2665	"public class Invalid implements Test { \n" +
2666	"}",
2667	"comment6b/Test.java",
2668	"package comment6b;\n" +
2669	"public interface Test {\n" +
2670	" /**\n" +
2671	" * @see Inner\n" +
2672	" */\n" +
2673	" public class Inner { }\n" +
2674	"}\n",
2675	"comment6b/Valid.java",
2676	"package comment6b;\n" +
2677	"\n" +
2678	"/**\n" +
2679	" * @see Test.Inner\n" +
2680	" */\n" +
2681	"public class Valid implements Test { \n" +
2682	"}"
2683	},
2684	// comment6b\Test.java:6: warning - Tag @see: reference not found: Inner
2685	// comment6b\Invalid.java:6: warning - Tag @see: reference not found: Inner
2686	"----------\n" +
2687	"1. ERROR in comment6b\\Invalid.java (at line 4)\n" +
2688	" * @see Inner\n" +
2689	" ^^^^^\n" +
2690	"Javadoc: Not visible reference\n" +
2691	"----------\n" +
2692	"----------\n" +
2693	"1. ERROR in comment6b\\Test.java (at line 4)\n" +
2694	" * @see Inner\n" +
2695	" ^^^^^\n" +
2696	"Javadoc: Not visible reference\n" +
2697	"----------\n"
2698	);
2699	}
2700	public void testBug96237_Public05() {
2701	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2702	runNegativeTest(
2703	new String[] {
2704	"test/a/Test.java",
2705	"package test.a;\n" +
2706	"/**\n" +
2707	" * @see Inner\n" +
2708	" * @see Test.Inner\n" +
2709	" */\n" +
2710	"public class Test {\n" +
2711	" class Inner {}\n" +
2712	"}\n"
2713	},
2714	// test\a\Test.java:6: warning - Tag @see: reference not found: Inner
2715	// test\a\Test.java:6: warning - Tag @see: reference not found: Test.Inner
2716	"----------\n" +
2717	"1. ERROR in test\\a\\Test.java (at line 3)\n" +
2718	" * @see Inner\n" +
2719	" ^^^^^\n" +
2720	"Javadoc: Not visible reference\n" +
2721	"----------\n" +
2722	"2. ERROR in test\\a\\Test.java (at line 4)\n" +
2723	" * @see Test.Inner\n" +
2724	" ^^^^^^^^^^\n" +
2725	"Javadoc: Not visible reference\n" +
2726	"----------\n"
2727	);
2728	}
2729	public void testBug96237_Public06() {
2730	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2731	runNegativeTest(
2732	new String[] {
2733	"test/b/Test.java",
2734	"package test.b;\n" +
2735	"/** \n" +
2736	" * @see Inner.Level2\n" +
2737	" * @see Test.Inner.Level2\n" +
2738	" */\n" +
2739	"public class Test {\n" +
2740	" /** \n" +
2741	" * @see Level2\n" +
2742	" * @see Test.Inner.Level2\n" +
2743	" */\n" +
2744	" public class Inner {\n" +
2745	" class Level2 {}\n" +
2746	" }\n" +
2747	"}\n"
2748	},
2749	// test\b\Test.java:6: warning - Tag @see: reference not found: Inner.Level2
2750	// test\b\Test.java:6: warning - Tag @see: reference not found: Test.Inner.Level2
2751	// test\b\Test.java:11: warning - Tag @see: reference not found: Level2
2752	// test\b\Test.java:11: warning - Tag @see: reference not found: Test.Inner.Level2
2753	"----------\n" +
2754	"1. ERROR in test\\b\\Test.java (at line 3)\n" +
2755	" * @see Inner.Level2\n" +
2756	" ^^^^^^^^^^^^\n" +
2757	"Javadoc: Not visible reference\n" +
2758	"----------\n" +
2759	"2. ERROR in test\\b\\Test.java (at line 4)\n" +
2760	" * @see Test.Inner.Level2\n" +
2761	" ^^^^^^^^^^^^^^^^^\n" +
2762	"Javadoc: Not visible reference\n" +
2763	"----------\n" +
2764	"3. ERROR in test\\b\\Test.java (at line 8)\n" +
2765	" * @see Level2\n" +
2766	" ^^^^^^\n" +
2767	"Javadoc: Not visible reference\n" +
2768	"----------\n" +
2769	"4. ERROR in test\\b\\Test.java (at line 9)\n" +
2770	" * @see Test.Inner.Level2\n" +
2771	" ^^^^^^^^^^^^^^^^^\n" +
2772	"Javadoc: Not visible reference\n" +
2773	"----------\n"
2774	);
2775	}
2776	public void testBug96237_Public07() {
2777	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2778	runNegativeTest(
2779	new String[] {
2780	"test/c/Test.java",
2781	"package test.c;\n" +
2782	"/**\n" +
2783	" * @see Inner.Level2.Level3\n" +
2784	" * @see Test.Inner.Level2.Level3\n" +
2785	" */\n" +
2786	"public class Test {\n" +
2787	" public class Inner {\n" +
2788	" /**\n" +
2789	" * @see Level3\n" +
2790	" * @see Level2.Level3\n" +
2791	" * @see Inner.Level2.Level3\n" +
2792	" * @see Test.Inner.Level2.Level3\n" +
2793	" */\n" +
2794	" public class Level2 {\n" +
2795	" class Level3 {\n" +
2796	" }\n" +
2797	" }\n" +
2798	" }\n" +
2799	"}\n"
2800	},
2801	// test\c\Test.java:6: warning - Tag @see: reference not found: Inner.Level2.Level3
2802	// test\c\Test.java:6: warning - Tag @see: reference not found: Test.Inner.Level2.Level3
2803	// test\c\Test.java:14: warning - Tag @see: reference not found: Level3
2804	// test\c\Test.java:14: warning - Tag @see: reference not found: Level2.Level3
2805	// test\c\Test.java:14: warning - Tag @see: reference not found: Inner.Level2.Level3
2806	// test\c\Test.java:14: warning - Tag @see: reference not found: Test.Inner.Level2.Level3
2807	"----------\n" +
2808	"1. ERROR in test\\c\\Test.java (at line 3)\n" +
2809	" * @see Inner.Level2.Level3\n" +
2810	" ^^^^^^^^^^^^^^^^^^^\n" +
2811	"Javadoc: Not visible reference\n" +
2812	"----------\n" +
2813	"2. ERROR in test\\c\\Test.java (at line 4)\n" +
2814	" * @see Test.Inner.Level2.Level3\n" +
2815	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2816	"Javadoc: Not visible reference\n" +
2817	"----------\n" +
2818	"3. ERROR in test\\c\\Test.java (at line 9)\n" +
2819	" * @see Level3\n" +
2820	" ^^^^^^\n" +
2821	"Javadoc: Not visible reference\n" +
2822	"----------\n" +
2823	"4. ERROR in test\\c\\Test.java (at line 10)\n" +
2824	" * @see Level2.Level3\n" +
2825	" ^^^^^^^^^^^^^\n" +
2826	"Javadoc: Not visible reference\n" +
2827	"----------\n" +
2828	"5. ERROR in test\\c\\Test.java (at line 11)\n" +
2829	" * @see Inner.Level2.Level3\n" +
2830	" ^^^^^^^^^^^^^^^^^^^\n" +
2831	"Javadoc: Not visible reference\n" +
2832	"----------\n" +
2833	"6. ERROR in test\\c\\Test.java (at line 12)\n" +
2834	" * @see Test.Inner.Level2.Level3\n" +
2835	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2836	"Javadoc: Not visible reference\n" +
2837	"----------\n"
2838	);
2839	}
2840	public void testBug96237_Private01() {
2841	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2842	runConformTest(
2843	new String[] {
2844	"comment6/Valid.java",
2845	"package comment6;\n" +
2846	"public class Valid {\n" +
2847	" /**\n" +
2848	" * @see Valid.Inner\n" +
2849	" */\n" +
2850	" public class Inner { }\n" +
2851	"}\n" +
2852	"/**\n" +
2853	" * See also {@link Valid.Inner}\n" +
2854	" */\n" +
2855	"class Sub2 extends Valid { }"
2856	}
2857	);
2858	}
2859	public void testBug96237_Private02() {
2860	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2861	runNegativeTest(
2862	new String[] {
2863	"comment6/Invalid.java",
2864	"package comment6;\n" +
2865	"public class Invalid {\n" +
2866	" /**\n" +
2867	" * @see Inner\n" +
2868	" */\n" +
2869	" public class Inner { }\n" +
2870	"}\n" +
2871	"/**\n" +
2872	" * See also {@link Inner} \n" +
2873	" */\n" +
2874	"class Sub1 extends Invalid { }\n"
2875	},
2876	// comment6\Invalid.java:6: warning - Tag @see: reference not found: Inner
2877	// comment6\Invalid.java:11: warning - Tag @link: reference not found: Inner
2878	"----------\n" +
2879	"1. ERROR in comment6\\Invalid.java (at line 4)\n" +
2880	" * @see Inner\n" +
2881	" ^^^^^\n" +
2882	"Javadoc: Not visible reference\n" +
2883	"----------\n" +
2884	"2. ERROR in comment6\\Invalid.java (at line 9)\n" +
2885	" * See also {@link Inner} \n" +
2886	" ^^^^^\n" +
2887	"Javadoc: Not visible reference\n" +
2888	"----------\n"
2889	);
2890	}
2891	public void testBug96237_Private03() {
2892	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2893	runNegativeTest(
2894	new String[] {
2895	"comment6a/def/Test.java",
2896	"package comment6a.def;\n" +
2897	"public class Test {\n" +
2898	" /**\n" +
2899	" * @see Inner\n" +
2900	" */\n" +
2901	" public class Inner { }\n" +
2902	"}\n",
2903	"comment6a/test/Invalid.java",
2904	"package comment6a.test;\n" +
2905	"import comment6a.def.Test;\n" +
2906	"/**\n" +
2907	" * See also {@link Inner}\n" +
2908	" */\n" +
2909	"public class Invalid extends Test { \n" +
2910	"}",
2911	"comment6a/test/Invalid2.java",
2912	"package comment6a.test;\n" +
2913	"import comment6a.def.Test;\n" +
2914	"/**\n" +
2915	" * @see Test.Inner\n" +
2916	" */\n" +
2917	"public class Invalid2 extends Test { \n" +
2918	"}"
2919	},
2920	// comment6a\def\Test.java:6: warning - Tag @see: reference not found: Inner
2921	// comment6a\test\Invalid.java:8: warning - Tag @link: reference not found: Inner
2922	// comment6a\test\Invalid2.java:8: warning - Tag @see: reference not found: Test.Inner
2923	"----------\n" +
2924	"1. ERROR in comment6a\\def\\Test.java (at line 4)\n" +
2925	" * @see Inner\n" +
2926	" ^^^^^\n" +
2927	"Javadoc: Not visible reference\n" +
2928	"----------\n" +
2929	"----------\n" +
2930	"1. ERROR in comment6a\\test\\Invalid.java (at line 4)\n" +
2931	" * See also {@link Inner}\n" +
2932	" ^^^^^\n" +
2933	"Javadoc: Not visible reference\n" +
2934	"----------\n" +
2935	"----------\n" +
2936	"1. ERROR in comment6a\\test\\Invalid2.java (at line 4)\n" +
2937	" * @see Test.Inner\n" +
2938	" ^^^^^^^^^^\n" +
2939	"Javadoc: Not visible reference\n" +
2940	"----------\n"
2941	);
2942	}
2943	public void testBug96237_Private04() {
2944	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2945	runNegativeTest(
2946	new String[] {
2947	"comment6b/Invalid.java",
2948	"package comment6b;\n" +
2949	"\n" +
2950	"/**\n" +
2951	" * @see Inner\n" +
2952	" */\n" +
2953	"public class Invalid implements Test { \n" +
2954	"}",
2955	"comment6b/Test.java",
2956	"package comment6b;\n" +
2957	"public interface Test {\n" +
2958	" /**\n" +
2959	" * @see Inner\n" +
2960	" */\n" +
2961	" public class Inner { }\n" +
2962	"}\n",
2963	"comment6b/Valid.java",
2964	"package comment6b;\n" +
2965	"\n" +
2966	"/**\n" +
2967	" * @see Test.Inner\n" +
2968	" */\n" +
2969	"public class Valid implements Test { \n" +
2970	"}"
2971	},
2972	// comment6b\Invalid.java:6: warning - Tag @see: reference not found: Inner
2973	// comment6b\Test.java:6: warning - Tag @see: reference not found: Inner
2974	"----------\n" +
2975	"1. ERROR in comment6b\\Invalid.java (at line 4)\n" +
2976	" * @see Inner\n" +
2977	" ^^^^^\n" +
2978	"Javadoc: Not visible reference\n" +
2979	"----------\n" +
2980	"----------\n" +
2981	"1. ERROR in comment6b\\Test.java (at line 4)\n" +
2982	" * @see Inner\n" +
2983	" ^^^^^\n" +
2984	"Javadoc: Not visible reference\n" +
2985	"----------\n"
2986	);
2987	}
2988	public void testBug96237_Private05() {
2989	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2990	runConformTest(
2991	new String[] {
2992	"test/a/Test.java",
2993	"package test.a;\n" +
2994	"/**\n" +
2995	" * @see Inner\n" +
2996	" * @see Test.Inner\n" +
2997	" */\n" +
2998	"public class Test {\n" +
2999	" class Inner {}\n" +
3000	"}\n"
3001	}
3002	);
3003	}
3004	public void testBug96237_Private06() {
3005	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3006	runConformTest(
3007	new String[] {
3008	"test/b/Test.java",
3009	"package test.b;\n" +
3010	"/** \n" +
3011	" * @see Inner.Level2\n" +
3012	" * @see Test.Inner.Level2\n" +
3013	" */\n" +
3014	"public class Test {\n" +
3015	" /** \n" +
3016	" * @see Level2\n" +
3017	" * @see Test.Inner.Level2\n" +
3018	" */\n" +
3019	" public class Inner {\n" +
3020	" class Level2 {}\n" +
3021	" }\n" +
3022	"}\n"
3023	}
3024	);
3025	}
3026	public void testBug96237_Private07() {
3027	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
3028	runConformTest(
3029	new String[] {
3030	"test/c/Test.java",
3031	"package test.c;\n" +
3032	"/**\n" +
3033	" * @see Inner.Level2.Level3\n" +
3034	" * @see Test.Inner.Level2.Level3\n" +
3035	" */\n" +
3036	"public class Test {\n" +
3037	" public class Inner {\n" +
3038	" /**\n" +
3039	" * @see Level3\n" +
3040	" * @see Level2.Level3\n" +
3041	" * @see Inner.Level2.Level3\n" +
3042	" * @see Test.Inner.Level2.Level3\n" +
3043	" */\n" +
3044	" public class Level2 {\n" +
3045	" class Level3 {\n" +
3046	" }\n" +
3047	" }\n" +
3048	" }\n" +
3049	"}\n"
3050	}
3051	);
3052	}
2549		3053
2550	/**	3054	/**
2551	* Bug 95286: [1.5][javadoc] package-info.java incorrectly flags "Missing comment for public declaration"	3055	* Bug 95286: [1.5][javadoc] package-info.java incorrectly flags "Missing comment for public declaration"
Lines 3020-3025 Link Here
3020	}	3524	}
3021		3525
3022	/**	3526	/**
		3527	* Bug 103304: [Javadoc] Wrong reference proposal for inner classes.
		3528	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=103304"
		3529	*/
		3530	public void testBug103304a() {
		3531	runNegativeTest(
		3532	new String[] {
		3533	"boden/IAFAState.java",
		3534	"package boden;\n" +
		3535	"public interface IAFAState {\n" +
		3536	" public class ValidationException extends Exception {\n" +
		3537	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		3538	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		3539	" }\n" +
		3540	" public void method() {}\n" +
		3541	" }\n" +
		3542	" /**\n" +
		3543	" * Validates a formula for consistent bindings. Bindings are consistent, when at each point in time,\n" +
		3544	" * the set of povided variables can be guaranteed to be a superset of the set of required variables.\n" +
		3545	" * @throws ValidationException Thrown if a variable is unbound. \n" +
		3546	" * @see ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3547	" * @see IAFAState.ValidationException#method()\n" +
		3548	" * @see ValidationException\n" +
		3549	" * {@link ValidationException}\n" +
		3550	" */\n" +
		3551	" public void validate() throws ValidationException;\n" +
		3552	"}\n",
		3553	"boden/TestValid.java",
		3554	"package boden;\n" +
		3555	"import boden.IAFAState.ValidationException;\n" +
		3556	"/**\n" +
		3557	" * @see ValidationException\n" +
		3558	" * @see IAFAState.ValidationException\n" +
		3559	" */\n" +
		3560	"public class TestValid {\n" +
		3561	" /** \n" +
		3562	" * @see ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3563	" */\n" +
		3564	" IAFAState.ValidationException valid1;\n" +
		3565	" /**\n" +
		3566	" * @see IAFAState.ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3567	" */\n" +
		3568	" IAFAState.ValidationException valid2;\n" +
		3569	"}\n"
		3570	},
		3571	// boden\TestValid.java:8: warning - Tag @see: reference not found: ValidationException
		3572	// boden\TestValid.java:12: warning - Tag @see: reference not found: ValidationException#IAFAState.ValidationException(String, IAFAState)
		3573	"----------\n" +
		3574	"1. ERROR in boden\\TestValid.java (at line 4)\r\n" +
		3575	" * @see ValidationException\r\n" +
		3576	" ^^^^^^^^^^^^^^^^^^^\n" +
		3577	"Javadoc: Not visible reference\n" +
		3578	"----------\n"
		3579	);
		3580	}
		3581	public void testBug103304b() {
		3582	runNegativeTest(
		3583	new String[] {
		3584	"boden/IAFAState.java",
		3585	"package boden;\n" +
		3586	"public interface IAFAState {\n" +
		3587	" public class ValidationException extends Exception {\n" +
		3588	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		3589	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		3590	" }\n" +
3591	" public void method() {}\n" +
3592	" }\n" +
3593	"}\n",
3594	"boden/TestInvalid1.java",
3595	"package boden;\n" +
3596	"import boden.IAFAState.ValidationException;\n" +
3597	"public class TestInvalid1 {\n" +
3598	" /** \n" +
3599	" * @see ValidationException#ValidationException(String, IAFAState)\n" +
3600	" */ \n" +
3601	" IAFAState.ValidationException invalid;\n" +
3602	"}\n",
3603	"boden/TestInvalid2.java",
3604	"package boden;\n" +
3605	"import boden.IAFAState.ValidationException;\n" +
3606	"public class TestInvalid2 {\n" +
3607	" /**\n" +
3608	" * @see IAFAState.ValidationException#ValidationException(String, IAFAState)\n" +
3609	" */\n" +
3610	" IAFAState.ValidationException invalid;\n" +
3611	"}\n",
3612	"boden/TestInvalid3.java",
3613	"package boden;\n" +
3614	"import boden.IAFAState.ValidationException;\n" +
3615	"public class TestInvalid3 {\n" +
3616	" /**\n" +
3617	" * @see IAFAState.ValidationException#IAFA.State.ValidationException(String, IAFAState)\n" +
3618	" */\n" +
3619	" IAFAState.ValidationException invalid;\n" +
3620	"}\n",
3621	"boden/TestInvalid4.java",
3622	"package boden;\n" +
3623	"import boden.IAFAState.ValidationException;\n" +
3624	"public class TestInvalid4 {\n" +
3625	" /**\n" +
3626	" * @see IAFAState.ValidationException#IAFAState .ValidationException(String, IAFAState)\n" +
3627	" */\n" +
3628	" IAFAState.ValidationException invalid;\n" +
3629	"}\n"
3630	},
3631	// boden\TestInvalid1.java:7: warning - Tag @see: reference not found: ValidationException#ValidationException(String, IAFAState)
3632	// boden\TestInvalid2.java:6: warning - Tag @see: can't find ValidationException(String, IAFAState) in boden.IAFAState.ValidationException
3633	// boden\TestInvalid3.java:6: warning - Tag @see: can't find IAFA.State.ValidationException(String, IAFAState) in boden.IAFAState.ValidationException
3634	// boden\TestInvalid4.java:6: warning - Tag @see: can't find IAFAState in boden.IAFAState.ValidationException
3635	"----------\n" +
3636	"1. ERROR in boden\\TestInvalid1.java (at line 5)\n" +
3637	" * @see ValidationException#ValidationException(String, IAFAState)\n" +
3638	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3639	"Javadoc: Invalid qualification for member type constructor\n" +
3640	"----------\n" +
3641	"----------\n" +
3642	"1. ERROR in boden\\TestInvalid2.java (at line 5)\n" +
3643	" * @see IAFAState.ValidationException#ValidationException(String, IAFAState)\n" +
3644	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3645	"Javadoc: Invalid qualification for member type constructor\n" +
3646	"----------\n" +
3647	"----------\n" +
3648	"1. ERROR in boden\\TestInvalid3.java (at line 5)\n" +
3649	" * @see IAFAState.ValidationException#IAFA.State.ValidationException(String, IAFAState)\n" +
3650	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
3651	"Javadoc: Invalid qualification for member type constructor\n" +
3652	"----------\n" +
3653	"----------\n" +
3654	"1. ERROR in boden\\TestInvalid4.java (at line 5)\n" +
3655	" * @see IAFAState.ValidationException#IAFAState .ValidationException(String, IAFAState)\n" +
3656	" ^^^^^^^^^\n" +
3657	"Javadoc: IAFAState cannot be resolved or is not a field\n" +
3658	"----------\n"
3659	);
3660	}
3661	public void testBug103304c() {
3662	runNegativeTest(
3663	new String[] {
3664	"test/Test.java",
3665	"package test;\n" +
3666	"public interface Test {\n" +
3667	" public class Level0 {\n" +
3668	" public Level0() {}\n" +
3669	" }\n" +
3670	" public interface Member {\n" +
3671	" public class Level1 {\n" +
3672	" public Level1() {}\n" +
3673	" }\n" +
3674	" }\n" +
3675	"}\n",
3676	"test/C.java",
3677	"package test;\n" +
3678	"public class C {\n" +
3679	" /**\n" +
3680	" * @see Test.Level0#Test.Level0()\n" +
3681	" */\n" +
3682	" Test.Level0 valid = new Test.Level0();\n" +
3683	" /**\n" +
3684	" * @see Test.Level0#Level0()\n" +
3685	" */\n" +
3686	" Test.Level0 invalid = new Test.Level0();\n" +
3687	"}\n"
3688	},
3689	// test\C.java:10: warning - Tag @see: can't find Level0() in test.Test.Level0
3690	"----------\n" +
3691	"1. ERROR in test\\C.java (at line 8)\n" +
3692	" * @see Test.Level0#Level0()\n" +
3693	" ^^^^^^^^\n" +
3694	"Javadoc: Invalid qualification for member type constructor\n" +
3695	"----------\n"
3696	);
3697	}
3698	public void testBug103304d() {
3699	runNegativeTest(
3700	new String[] {
3701	"test/Test.java",
3702	"package test;\n" +
3703	"public interface Test {\n" +
3704	" public class Level0 {\n" +
3705	" public Level0() {}\n" +
3706	" }\n" +
3707	" public interface Member {\n" +
3708	" public class Level1 {\n" +
3709	" public Level1() {}\n" +
3710	" }\n" +
3711	" }\n" +
3712	"}\n",
3713	"test/C2.java",
3714	"package test;\n" +
3715	"public class C2 {\n" +
3716	" /**\n" +
3717	" * @see Test.Member.Level1#Test.Member.Level1()\n" +
3718	" */\n" +
3719	" Test.Member.Level1 valid = new Test.Member.Level1();\n" +
3720	" /**\n" +
3721	" * @see Test.Member.Level1#Level1()\n" +
3722	" */\n" +
3723	" Test.Member.Level1 invalid = new Test.Member.Level1();\n" +
3724	" /**\n" +
3725	" * @see Test.Member.Level1#Test.Level1()\n" +
3726	" */\n" +
3727	" Test.Member.Level1 wrong = new Test.Member.Level1();\n" +
3728	"}\n"
3729	},
3730	// test\C2.java:10: warning - Tag @see: can't find Level1() in test.Test.Member.Level1
3731	// test\C2.java:14: warning - Tag @see: can't find Test.Level1() in test.Test.Member.Level1
3732	"----------\n" +
3733	"1. ERROR in test\\C2.java (at line 8)\n" +
3734	" * @see Test.Member.Level1#Level1()\n" +
3735	" ^^^^^^^^\n" +
3736	"Javadoc: Invalid qualification for member type constructor\n" +
3737	"----------\n" +
3738	"2. ERROR in test\\C2.java (at line 12)\n" +
3739	" * @see Test.Member.Level1#Test.Level1()\n" +
3740	" ^^^^^^^^^^^^^\n" +
3741	"Javadoc: Invalid qualification for member type constructor\n" +
3742	"----------\n"
3743	);
3744	}
3745	public void testBug103304e() {
3746	runConformTest(
3747	new String[] {
3748	"implicit/Valid.java",
3749	"package implicit;\n" +
3750	"public interface Valid {\n" +
3751	" public class Level0 {\n" +
3752	" /**\n" +
3753	" * @see #Valid.Level0() Valid\n" +
3754	" */\n" +
3755	" public Level0() {}\n" +
3756	" /**\n" +
3757	" * @see #Valid.Level0(String) Valid\n" +
3758	" */\n" +
3759	" public Level0(String str) {}\n" +
3760	" }\n" +
3761	" public interface Member {\n" +
3762	" public class Level1 {\n" +
3763	" /**\n" +
3764	" * @see #Valid.Member.Level1() Valid\n" +
3765	" */\n" +
3766	" public Level1() {}\n" +
3767	" /**\n" +
3768	" * @see #Valid.Member.Level1(int) Valid\n" +
3769	" */\n" +
3770	" public Level1(int x) {}\n" +
3771	" }\n" +
3772	" }\n" +
3773	"}\n"
3774	}
3775	);
3776	}
3777	public void testBug103304f() {
3778	runNegativeTest(
3779	new String[] {
3780	"implicit/Invalid.java",
3781	"package implicit;\n" +
3782	"public interface Invalid {\n" +
3783	" public class Level0 {\n" +
3784	" /**\n" +
3785	" * @see #Level0() Invalid\n" +
3786	" */\n" +
3787	" public Level0() {}\n" +
3788	" /**\n" +
3789	" * @see #Level0(String) Invalid\n" +
3790	" */\n" +
3791	" public Level0(String str) {}\n" +
3792	" }\n" +
3793	" public interface Member {\n" +
3794	" public class Level1 {\n" +
3795	" /**\n" +
3796	" * @see #Level1() Invalid\n" +
3797	" * @see #Member.Level1() Invalid\n" +
3798	" * @see #Invalid.Level1() Invalid\n" +
3799	" */\n" +
3800	" public Level1() {}\n" +
3801	" /**\n" +
3802	" * @see #Level1(int) Invalid\n" +
3803	" * @see #Invalid.Level1(int) Invalid\n" +
3804	" * @see #Member.Level1(int) Invalid\n" +
3805	" */\n" +
3806	" public Level1(int x) {}\n" +
3807	" }\n" +
3808	" }\n" +
3809	"}\n"
3810	},
3811	// implicit\Invalid.java:7: warning - Tag @see: can't find Level0() in implicit.Invalid.Level0
3812	// implicit\Invalid.java:11: warning - Tag @see: can't find Level0(String) in implicit.Invalid.Level0
3813	// implicit\Invalid.java:20: warning - Tag @see: can't find Level1() in implicit.Invalid.Member.Level1
3814	// implicit\Invalid.java:20: warning - Tag @see: can't find Member.Level1() in implicit.Invalid.Member.Level1
3815	// implicit\Invalid.java:20: warning - Tag @see: can't find Invalid.Level1() in implicit.Invalid.Member.Level1
3816	// implicit\Invalid.java:26: warning - Tag @see: can't find Level1(int) in implicit.Invalid.Member.Level1
3817	// implicit\Invalid.java:26: warning - Tag @see: can't find Invalid.Level1(int) in implicit.Invalid.Member.Level1
3818	// implicit\Invalid.java:26: warning - Tag @see: can't find Member.Level1(int) in implicit.Invalid.Member.Level1
3819	"----------\n" +
3820	"1. ERROR in implicit\\Invalid.java (at line 5)\n" +
3821	" * @see #Level0() Invalid\n" +
3822	" ^^^^^^^^\n" +
3823	"Javadoc: Invalid qualification for member type constructor\n" +
3824	"----------\n" +
3825	"2. ERROR in implicit\\Invalid.java (at line 9)\n" +
3826	" * @see #Level0(String) Invalid\n" +
3827	" ^^^^^^^^^^^^^^\n" +
3828	"Javadoc: Invalid qualification for member type constructor\n" +
3829	"----------\n" +
3830	"3. ERROR in implicit\\Invalid.java (at line 16)\n" +
3831	" * @see #Level1() Invalid\n" +
3832	" ^^^^^^^^\n" +
3833	"Javadoc: Invalid qualification for member type constructor\n" +
3834	"----------\n" +
3835	"4. ERROR in implicit\\Invalid.java (at line 17)\n" +
3836	" * @see #Member.Level1() Invalid\n" +
3837	" ^^^^^^^^^^^^^^^\n" +
3838	"Javadoc: Invalid qualification for member type constructor\n" +
3839	"----------\n" +
3840	"5. ERROR in implicit\\Invalid.java (at line 18)\n" +
3841	" * @see #Invalid.Level1() Invalid\n" +
3842	" ^^^^^^^^^^^^^^^^\n" +
3843	"Javadoc: Invalid qualification for member type constructor\n" +
3844	"----------\n" +
3845	"6. ERROR in implicit\\Invalid.java (at line 22)\n" +
3846	" * @see #Level1(int) Invalid\n" +
3847	" ^^^^^^^^^^^\n" +
3848	"Javadoc: Invalid qualification for member type constructor\n" +
3849	"----------\n" +
3850	"7. ERROR in implicit\\Invalid.java (at line 23)\n" +
3851	" * @see #Invalid.Level1(int) Invalid\n" +
3852	" ^^^^^^^^^^^^^^^^^^^\n" +
3853	"Javadoc: Invalid qualification for member type constructor\n" +
3854	"----------\n" +
3855	"8. ERROR in implicit\\Invalid.java (at line 24)\n" +
3856	" * @see #Member.Level1(int) Invalid\n" +
3857	" ^^^^^^^^^^^^^^^^^^\n" +
3858	"Javadoc: Invalid qualification for member type constructor\n" +
3859	"----------\n"
3860	);
3861	}
3862
3863	/**
3023	* Bug 112346: [javadoc] {@inheritedDoc} should be inactive for non-overridden method	3864	* Bug 112346: [javadoc] {@inheritedDoc} should be inactive for non-overridden method
3024	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=112346"	3865	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=112346"
3025	*/	3866	*/

Lines 20-25 Link Here

(-)src/org/eclipse/jdt/core/tests/compiler/regression/JavadocTest_1_5.java (-4 / +804 lines)
20		20
21	String docCommentSupport = CompilerOptions.ENABLED;	21	String docCommentSupport = CompilerOptions.ENABLED;
22	String reportInvalidJavadoc = CompilerOptions.ERROR;	22	String reportInvalidJavadoc = CompilerOptions.ERROR;
		23	String reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
23	String reportMissingJavadocTags = CompilerOptions.ERROR;	24	String reportMissingJavadocTags = CompilerOptions.ERROR;
24	String reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;	25	String reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;
25	String reportMissingJavadocComments = null;	26	String reportMissingJavadocComments = null;
Lines 49-54 Link Here
49	Map options = super.getCompilerOptions();	50	Map options = super.getCompilerOptions();
50	options.put(CompilerOptions.OPTION_DocCommentSupport, this.docCommentSupport);	51	options.put(CompilerOptions.OPTION_DocCommentSupport, this.docCommentSupport);
51	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);	52	options.put(CompilerOptions.OPTION_ReportInvalidJavadoc, reportInvalidJavadoc);
		53	if (!CompilerOptions.IGNORE.equals(reportInvalidJavadoc)) {
		54	options.put(CompilerOptions.OPTION_ReportInvalidJavadocTagsVisibility, this.reportInvalidJavadocVisibility);
		55	}
52	if (reportMissingJavadocComments != null)	56	if (reportMissingJavadocComments != null)
53	options.put(CompilerOptions.OPTION_ReportMissingJavadocComments, reportMissingJavadocComments);	57	options.put(CompilerOptions.OPTION_ReportMissingJavadocComments, reportMissingJavadocComments);
54	else	58	else
Lines 75-84 Link Here
75	protected void setUp() throws Exception {	79	protected void setUp() throws Exception {
76	super.setUp();	80	super.setUp();
77	this.docCommentSupport = CompilerOptions.ENABLED;	81	this.docCommentSupport = CompilerOptions.ENABLED;
78	reportInvalidJavadoc = CompilerOptions.ERROR;	82	this.reportInvalidJavadoc = CompilerOptions.ERROR;
79	reportMissingJavadocTags = CompilerOptions.ERROR;	83	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
80	reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;	84	this.reportMissingJavadocTags = CompilerOptions.ERROR;
81	reportMissingJavadocComments = CompilerOptions.IGNORE;	85	this.reportMissingJavadocTagsOverriding = CompilerOptions.ENABLED;
		86	this.reportMissingJavadocComments = CompilerOptions.IGNORE;
82	}	87	}
83		88
84	/**	89	/**
Lines 1899-1904 Link Here
1899	}	1904	}
1900		1905
1901	/**	1906	/**
		1907	* Bug 96237: [javadoc] Inner types must be qualified
		1908	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=96237"
		1909	*/
		1910	public void testBug96237_Public01() {
		1911	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		1912	runConformTest(
		1913	new String[] {
		1914	"comment6/Valid.java",
		1915	"package comment6;\n" +
		1916	"public class Valid {\n" +
		1917	" /**\n" +
		1918	" * @see Valid.Inner\n" +
		1919	" */\n" +
		1920	" public class Inner { }\n" +
		1921	"}\n" +
		1922	"/**\n" +
		1923	" * See also {@link Valid.Inner}\n" +
		1924	" */\n" +
		1925	"class Sub2 extends Valid { }"
		1926	}
		1927	);
		1928	}
		1929	public void testBug96237_Public02() {
		1930	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		1931	runConformTest(
		1932	new String[] {
		1933	"comment6/Invalid.java",
		1934	"package comment6;\n" +
		1935	"public class Invalid {\n" +
		1936	" /**\n" +
		1937	" * @see Inner\n" +
		1938	" */\n" +
		1939	" public class Inner { }\n" +
		1940	"}\n" +
		1941	"/**\n" +
		1942	" * See also {@link Inner} \n" +
		1943	" */\n" +
		1944	"class Sub1 extends Invalid { }\n"
		1945	}
		1946	);
		1947	}
		1948	public void testBug96237_Public03() {
		1949	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
		1950	runNegativeTest(
		1951	new String[] {
		1952	"comment6a/def/Test.java",
		1953	"package comment6a.def;\n" +
		1954	"public class Test {\n" +
		1955	" /**\n" +
		1956	" * @see Inner\n" +
		1957	" */\n" +
		1958	" public class Inner { }\n" +
		1959	"}\n",
		1960	"comment6a/test/Invalid.java",
		1961	"package comment6a.test;\n" +
		1962	"import comment6a.def.Test;\n" +
		1963	"/**\n" +
		1964	" * See also {@link Inner}\n" +
		1965	" */\n" +
		1966	"public class Invalid extends Test { \n" +
		1967	"}",
		1968	"comment6a/test/Invalid2.java",
		1969	"package comment6a.test;\n" +
		1970	"import comment6a.def.Test;\n" +
1971	"/**\n" +
1972	" * @see Test.Inner\n" +
1973	" */\n" +
1974	"public class Invalid2 extends Test { \n" +
1975	"}"
1976	},
1977	// comment6a\test\Invalid.java:8: warning - Tag @link: reference not found: Inner
1978	// comment6a\test\Invalid2.java:8: warning - Tag @see: reference not found: Test.Inner
1979	"----------\n" +
1980	"1. ERROR in comment6a\\test\\Invalid.java (at line 4)\n" +
1981	" * See also {@link Inner}\n" +
1982	" ^^^^^\n" +
1983	"Javadoc: Not visible reference\n" +
1984	"----------\n" +
1985	"----------\n" +
1986	"1. ERROR in comment6a\\test\\Invalid2.java (at line 4)\n" +
1987	" * @see Test.Inner\n" +
1988	" ^^^^^^^^^^\n" +
1989	"Javadoc: Not visible reference\n" +
1990	"----------\n"
1991	);
1992	}
1993	public void testBug96237_Public04() {
1994	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
1995	runNegativeTest(
1996	new String[] {
1997	"comment6b/Invalid.java",
1998	"package comment6b;\n" +
1999	"\n" +
2000	"/**\n" +
2001	" * @see Inner\n" +
2002	" */\n" +
2003	"public class Invalid implements Test { \n" +
2004	"}",
2005	"comment6b/Test.java",
2006	"package comment6b;\n" +
2007	"public interface Test {\n" +
2008	" /**\n" +
2009	" * @see Inner\n" +
2010	" */\n" +
2011	" public class Inner { }\n" +
2012	"}\n",
2013	"comment6b/Valid.java",
2014	"package comment6b;\n" +
2015	"\n" +
2016	"/**\n" +
2017	" * @see Test.Inner\n" +
2018	" */\n" +
2019	"public class Valid implements Test { \n" +
2020	"}"
2021	},
2022	// comment6b\Invalid.java:6: warning - Tag @see: reference not found: Inner
2023	"----------\n" +
2024	"1. ERROR in comment6b\\Invalid.java (at line 4)\n" +
2025	" * @see Inner\n" +
2026	" ^^^^^\n" +
2027	"Javadoc: Not visible reference\n" +
2028	"----------\n"
2029	);
2030	}
2031	public void testBug96237_Public05() {
2032	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2033	runNegativeTest(
2034	new String[] {
2035	"test/a/Test.java",
2036	"package test.a;\n" +
2037	"/**\n" +
2038	" * @see Inner\n" +
2039	" * @see Test.Inner\n" +
2040	" */\n" +
2041	"public class Test {\n" +
2042	" class Inner {}\n" +
2043	"}\n"
2044	},
2045	// test\a\Test.java:6: warning - Tag @see: reference not found: Inner
2046	// test\a\Test.java:6: warning - Tag @see: reference not found: Test.Inner
2047	"----------\n" +
2048	"1. ERROR in test\\a\\Test.java (at line 3)\n" +
2049	" * @see Inner\n" +
2050	" ^^^^^\n" +
2051	"Javadoc: Not visible reference\n" +
2052	"----------\n" +
2053	"2. ERROR in test\\a\\Test.java (at line 4)\n" +
2054	" * @see Test.Inner\n" +
2055	" ^^^^^^^^^^\n" +
2056	"Javadoc: Not visible reference\n" +
2057	"----------\n"
2058	);
2059	}
2060	public void testBug96237_Public06() {
2061	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2062	runNegativeTest(
2063	new String[] {
2064	"test/b/Test.java",
2065	"package test.b;\n" +
2066	"/** \n" +
2067	" * @see Inner.Level2\n" +
2068	" * @see Test.Inner.Level2\n" +
2069	" */\n" +
2070	"public class Test {\n" +
2071	" /** \n" +
2072	" * @see Level2\n" +
2073	" * @see Test.Inner.Level2\n" +
2074	" */\n" +
2075	" public class Inner {\n" +
2076	" class Level2 {}\n" +
2077	" }\n" +
2078	"}\n"
2079	},
2080	// test\b\Test.java:6: warning - Tag @see: reference not found: Inner.Level2
2081	// test\b\Test.java:6: warning - Tag @see: reference not found: Test.Inner.Level2
2082	// test\b\Test.java:11: warning - Tag @see: reference not found: Level2
2083	// test\b\Test.java:11: warning - Tag @see: reference not found: Test.Inner.Level2
2084	"----------\n" +
2085	"1. ERROR in test\\b\\Test.java (at line 3)\n" +
2086	" * @see Inner.Level2\n" +
2087	" ^^^^^^^^^^^^\n" +
2088	"Javadoc: Not visible reference\n" +
2089	"----------\n" +
2090	"2. ERROR in test\\b\\Test.java (at line 4)\n" +
2091	" * @see Test.Inner.Level2\n" +
2092	" ^^^^^^^^^^^^^^^^^\n" +
2093	"Javadoc: Not visible reference\n" +
2094	"----------\n" +
2095	"3. ERROR in test\\b\\Test.java (at line 8)\n" +
2096	" * @see Level2\n" +
2097	" ^^^^^^\n" +
2098	"Javadoc: Not visible reference\n" +
2099	"----------\n" +
2100	"4. ERROR in test\\b\\Test.java (at line 9)\n" +
2101	" * @see Test.Inner.Level2\n" +
2102	" ^^^^^^^^^^^^^^^^^\n" +
2103	"Javadoc: Not visible reference\n" +
2104	"----------\n"
2105	);
2106	}
2107	public void testBug96237_Public07() {
2108	this.reportInvalidJavadocVisibility = CompilerOptions.PUBLIC;
2109	runNegativeTest(
2110	new String[] {
2111	"test/c/Test.java",
2112	"package test.c;\n" +
2113	"/**\n" +
2114	" * @see Inner.Level2.Level3\n" +
2115	" * @see Test.Inner.Level2.Level3\n" +
2116	" */\n" +
2117	"public class Test {\n" +
2118	" public class Inner {\n" +
2119	" /**\n" +
2120	" * @see Level3\n" +
2121	" * @see Level2.Level3\n" +
2122	" * @see Inner.Level2.Level3\n" +
2123	" * @see Test.Inner.Level2.Level3\n" +
2124	" */\n" +
2125	" public class Level2 {\n" +
2126	" class Level3 {\n" +
2127	" }\n" +
2128	" }\n" +
2129	" }\n" +
2130	"}\n"
2131	},
2132	// test\c\Test.java:6: warning - Tag @see: reference not found: Inner.Level2.Level3
2133	// test\c\Test.java:6: warning - Tag @see: reference not found: Test.Inner.Level2.Level3
2134	// test\c\Test.java:14: warning - Tag @see: reference not found: Level3
2135	// test\c\Test.java:14: warning - Tag @see: reference not found: Level2.Level3
2136	// test\c\Test.java:14: warning - Tag @see: reference not found: Inner.Level2.Level3
2137	// test\c\Test.java:14: warning - Tag @see: reference not found: Test.Inner.Level2.Level3
2138	"----------\n" +
2139	"1. ERROR in test\\c\\Test.java (at line 3)\n" +
2140	" * @see Inner.Level2.Level3\n" +
2141	" ^^^^^^^^^^^^^^^^^^^\n" +
2142	"Javadoc: Not visible reference\n" +
2143	"----------\n" +
2144	"2. ERROR in test\\c\\Test.java (at line 4)\n" +
2145	" * @see Test.Inner.Level2.Level3\n" +
2146	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2147	"Javadoc: Not visible reference\n" +
2148	"----------\n" +
2149	"3. ERROR in test\\c\\Test.java (at line 9)\n" +
2150	" * @see Level3\n" +
2151	" ^^^^^^\n" +
2152	"Javadoc: Not visible reference\n" +
2153	"----------\n" +
2154	"4. ERROR in test\\c\\Test.java (at line 10)\n" +
2155	" * @see Level2.Level3\n" +
2156	" ^^^^^^^^^^^^^\n" +
2157	"Javadoc: Not visible reference\n" +
2158	"----------\n" +
2159	"5. ERROR in test\\c\\Test.java (at line 11)\n" +
2160	" * @see Inner.Level2.Level3\n" +
2161	" ^^^^^^^^^^^^^^^^^^^\n" +
2162	"Javadoc: Not visible reference\n" +
2163	"----------\n" +
2164	"6. ERROR in test\\c\\Test.java (at line 12)\n" +
2165	" * @see Test.Inner.Level2.Level3\n" +
2166	" ^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2167	"Javadoc: Not visible reference\n" +
2168	"----------\n"
2169	);
2170	}
2171	public void testBug96237_Private01() {
2172	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2173	runConformTest(
2174	new String[] {
2175	"comment6/Valid.java",
2176	"package comment6;\n" +
2177	"public class Valid {\n" +
2178	" /**\n" +
2179	" * @see Valid.Inner\n" +
2180	" */\n" +
2181	" public class Inner { }\n" +
2182	"}\n" +
2183	"/**\n" +
2184	" * See also {@link Valid.Inner}\n" +
2185	" */\n" +
2186	"class Sub2 extends Valid { }"
2187	}
2188	);
2189	}
2190	public void testBug96237_Private02() {
2191	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2192	runNegativeTest(
2193	new String[] {
2194	"comment6/Invalid.java",
2195	"package comment6;\n" +
2196	"public class Invalid {\n" +
2197	" /**\n" +
2198	" * @see Inner\n" +
2199	" */\n" +
2200	" public class Inner { }\n" +
2201	"}\n" +
2202	"/**\n" +
2203	" * See also {@link Inner} \n" +
2204	" */\n" +
2205	"class Sub1 extends Invalid { }\n"
2206	},
2207	// comment6\Invalid.java:11: warning - Tag @link: reference not found: Inner
2208	"----------\n" +
2209	"1. ERROR in comment6\\Invalid.java (at line 9)\n" +
2210	" * See also {@link Inner} \n" +
2211	" ^^^^^\n" +
2212	"Javadoc: Not visible reference\n" +
2213	"----------\n"
2214	);
2215	}
2216	public void testBug96237_Private03() {
2217	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2218	runNegativeTest(
2219	new String[] {
2220	"comment6a/def/Test.java",
2221	"package comment6a.def;\n" +
2222	"public class Test {\n" +
2223	" /**\n" +
2224	" * @see Inner\n" +
2225	" */\n" +
2226	" public class Inner { }\n" +
2227	"}\n",
2228	"comment6a/test/Invalid.java",
2229	"package comment6a.test;\n" +
2230	"import comment6a.def.Test;\n" +
2231	"/**\n" +
2232	" * See also {@link Inner}\n" +
2233	" */\n" +
2234	"public class Invalid extends Test { \n" +
2235	"}",
2236	"comment6a/test/Invalid2.java",
2237	"package comment6a.test;\n" +
2238	"import comment6a.def.Test;\n" +
2239	"/**\n" +
2240	" * @see Test.Inner\n" +
2241	" */\n" +
2242	"public class Invalid2 extends Test { \n" +
2243	"}"
2244	},
2245	// comment6a\test\Invalid.java:8: warning - Tag @link: reference not found: Inner
2246	// comment6a\test\Invalid2.java:8: warning - Tag @see: reference not found: Test.Inner
2247	"----------\n" +
2248	"1. ERROR in comment6a\\test\\Invalid.java (at line 4)\n" +
2249	" * See also {@link Inner}\n" +
2250	" ^^^^^\n" +
2251	"Javadoc: Not visible reference\n" +
2252	"----------\n" +
2253	"----------\n" +
2254	"1. ERROR in comment6a\\test\\Invalid2.java (at line 4)\n" +
2255	" * @see Test.Inner\n" +
2256	" ^^^^^^^^^^\n" +
2257	"Javadoc: Not visible reference\n" +
2258	"----------\n"
2259	);
2260	}
2261	public void testBug96237_Private04() {
2262	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2263	runNegativeTest(
2264	new String[] {
2265	"comment6b/Invalid.java",
2266	"package comment6b;\n" +
2267	"\n" +
2268	"/**\n" +
2269	" * @see Inner\n" +
2270	" */\n" +
2271	"public class Invalid implements Test { \n" +
2272	"}",
2273	"comment6b/Test.java",
2274	"package comment6b;\n" +
2275	"public interface Test {\n" +
2276	" /**\n" +
2277	" * @see Inner\n" +
2278	" */\n" +
2279	" public class Inner { }\n" +
2280	"}\n",
2281	"comment6b/Valid.java",
2282	"package comment6b;\n" +
2283	"\n" +
2284	"/**\n" +
2285	" * @see Test.Inner\n" +
2286	" */\n" +
2287	"public class Valid implements Test { \n" +
2288	"}"
2289	},
2290	// comment6b\Invalid.java:6: warning - Tag @see: reference not found: Inner
2291	"----------\n" +
2292	"1. ERROR in comment6b\\Invalid.java (at line 4)\n" +
2293	" * @see Inner\n" +
2294	" ^^^^^\n" +
2295	"Javadoc: Not visible reference\n" +
2296	"----------\n"
2297	);
2298	}
2299	public void testBug96237_Private05() {
2300	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2301	runConformTest(
2302	new String[] {
2303	"test/a/Test.java",
2304	"package test.a;\n" +
2305	"/**\n" +
2306	" * @see Inner\n" +
2307	" * @see Test.Inner\n" +
2308	" */\n" +
2309	"public class Test {\n" +
2310	" class Inner {}\n" +
2311	"}\n"
2312	}
2313	);
2314	}
2315	public void testBug96237_Private06() {
2316	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2317	runConformTest(
2318	new String[] {
2319	"test/b/Test.java",
2320	"package test.b;\n" +
2321	"/** \n" +
2322	" * @see Inner.Level2\n" +
2323	" * @see Test.Inner.Level2\n" +
2324	" */\n" +
2325	"public class Test {\n" +
2326	" /** \n" +
2327	" * @see Level2\n" +
2328	" * @see Test.Inner.Level2\n" +
2329	" */\n" +
2330	" public class Inner {\n" +
2331	" class Level2 {}\n" +
2332	" }\n" +
2333	"}\n"
2334	}
2335	);
2336	}
2337	public void testBug96237_Private07() {
2338	this.reportInvalidJavadocVisibility = CompilerOptions.PRIVATE;
2339	runConformTest(
2340	new String[] {
2341	"test/c/Test.java",
2342	"package test.c;\n" +
2343	"/**\n" +
2344	" * @see Inner.Level2.Level3\n" +
2345	" * @see Test.Inner.Level2.Level3\n" +
2346	" */\n" +
2347	"public class Test {\n" +
2348	" public class Inner {\n" +
2349	" /**\n" +
2350	" * @see Level3\n" +
2351	" * @see Level2.Level3\n" +
2352	" * @see Inner.Level2.Level3\n" +
2353	" * @see Test.Inner.Level2.Level3\n" +
2354	" */\n" +
2355	" public class Level2 {\n" +
2356	" class Level3 {\n" +
2357	" }\n" +
2358	" }\n" +
2359	" }\n" +
2360	"}\n"
2361	}
2362	);
2363	}
2364
2365	/**
1902	* Bug 101283: [1.5][javadoc] Javadoc validation raises missing implementation in compiler	2366	* Bug 101283: [1.5][javadoc] Javadoc validation raises missing implementation in compiler
1903	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=101283"	2367	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=101283"
1904	*/	2368	*/
Lines 2150-2155 Link Here
2150	}	2614	}
2151		2615
2152	/**	2616	/**
		2617	* Bug 103304: [Javadoc] Wrong reference proposal for inner classes.
		2618	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=103304"
		2619	*/
		2620	public void testBug103304a() {
		2621	runNegativeTest(
		2622	new String[] {
		2623	"boden/IAFAState.java",
		2624	"package boden;\n" +
		2625	"public interface IAFAState {\n" +
		2626	" public class ValidationException extends Exception {\n" +
		2627	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		2628	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		2629	" }\n" +
		2630	" public void method() {}\n" +
		2631	" }\n" +
		2632	" /**\n" +
		2633	" * Validates a formula for consistent bindings. Bindings are consistent, when at each point in time,\n" +
		2634	" * the set of povided variables can be guaranteed to be a superset of the set of required variables.\n" +
		2635	" * @throws ValidationException Thrown if a variable is unbound. \n" +
		2636	" * @see ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		2637	" * @see IAFAState.ValidationException#method()\n" +
		2638	" * @see ValidationException\n" +
		2639	" * {@link ValidationException}\n" +
		2640	" */\n" +
		2641	" public void validate() throws ValidationException;\n" +
		2642	"}\n",
		2643	"boden/TestValid.java",
		2644	"package boden;\n" +
		2645	"import boden.IAFAState.ValidationException;\n" +
		2646	"/**\n" +
		2647	" * @see ValidationException\n" +
		2648	" * @see IAFAState.ValidationException\n" +
		2649	" */\n" +
		2650	"public class TestValid {\n" +
		2651	" /** \n" +
		2652	" * @see ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		2653	" */\n" +
		2654	" IAFAState.ValidationException valid1;\n" +
		2655	" /**\n" +
		2656	" * @see IAFAState.ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		2657	" */\n" +
		2658	" IAFAState.ValidationException valid2;\n" +
		2659	"}\n"
		2660	},
		2661	// no warning
		2662	"----------\n" +
		2663	"1. ERROR in boden\\TestValid.java (at line 4)\r\n" +
		2664	" * @see ValidationException\r\n" +
		2665	" ^^^^^^^^^^^^^^^^^^^\n" +
		2666	"Javadoc: Not visible reference\n" +
		2667	"----------\n"
		2668	);
		2669	}
		2670	public void testBug103304b() {
		2671	runNegativeTest(
		2672	new String[] {
		2673	"boden/IAFAState.java",
		2674	"package boden;\n" +
		2675	"public interface IAFAState {\n" +
		2676	" public class ValidationException extends Exception {\n" +
		2677	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		2678	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		2679	" }\n" +
		2680	" public void method() {}\n" +
2681	" }\n" +
2682	"}\n",
2683	"boden/TestInvalid1.java",
2684	"package boden;\n" +
2685	"import boden.IAFAState.ValidationException;\n" +
2686	"public class TestInvalid1 {\n" +
2687	" /** \n" +
2688	" * @see ValidationException#ValidationException(String, IAFAState)\n" +
2689	" */ \n" +
2690	" IAFAState.ValidationException invalid;\n" +
2691	"}\n",
2692	"boden/TestInvalid2.java",
2693	"package boden;\n" +
2694	"import boden.IAFAState.ValidationException;\n" +
2695	"public class TestInvalid2 {\n" +
2696	" /**\n" +
2697	" * @see IAFAState.ValidationException#ValidationException(String, IAFAState)\n" +
2698	" */\n" +
2699	" IAFAState.ValidationException invalid;\n" +
2700	"}\n",
2701	"boden/TestInvalid3.java",
2702	"package boden;\n" +
2703	"import boden.IAFAState.ValidationException;\n" +
2704	"public class TestInvalid3 {\n" +
2705	" /**\n" +
2706	" * @see IAFAState.ValidationException#IAFA.State.ValidationException(String, IAFAState)\n" +
2707	" */\n" +
2708	" IAFAState.ValidationException invalid;\n" +
2709	"}\n",
2710	"boden/TestInvalid4.java",
2711	"package boden;\n" +
2712	"import boden.IAFAState.ValidationException;\n" +
2713	"public class TestInvalid4 {\n" +
2714	" /**\n" +
2715	" * @see IAFAState.ValidationException#IAFAState .ValidationException(String, IAFAState)\n" +
2716	" */\n" +
2717	" IAFAState.ValidationException invalid;\n" +
2718	"}\n"
2719	},
2720	// boden\TestInvalid1.java:7: warning - Tag @see: reference not found: ValidationException#ValidationException(String, IAFAState)
2721	// boden\TestInvalid2.java:6: warning - Tag @see: can't find ValidationException(String, IAFAState) in boden.IAFAState.ValidationException
2722	// boden\TestInvalid3.java:6: warning - Tag @see: can't find IAFA.State.ValidationException(String, IAFAState) in boden.IAFAState.ValidationException
2723	// boden\TestInvalid4.java:6: warning - Tag @see: can't find IAFAState in boden.IAFAState.ValidationException
2724	"----------\n" +
2725	"1. ERROR in boden\\TestInvalid1.java (at line 5)\n" +
2726	" * @see ValidationException#ValidationException(String, IAFAState)\n" +
2727	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2728	"Javadoc: Invalid qualification for member type constructor\n" +
2729	"----------\n" +
2730	"----------\n" +
2731	"1. ERROR in boden\\TestInvalid2.java (at line 5)\n" +
2732	" * @see IAFAState.ValidationException#ValidationException(String, IAFAState)\n" +
2733	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2734	"Javadoc: Invalid qualification for member type constructor\n" +
2735	"----------\n" +
2736	"----------\n" +
2737	"1. ERROR in boden\\TestInvalid3.java (at line 5)\n" +
2738	" * @see IAFAState.ValidationException#IAFA.State.ValidationException(String, IAFAState)\n" +
2739	" ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n" +
2740	"Javadoc: Invalid qualification for member type constructor\n" +
2741	"----------\n" +
2742	"----------\n" +
2743	"1. ERROR in boden\\TestInvalid4.java (at line 5)\n" +
2744	" * @see IAFAState.ValidationException#IAFAState .ValidationException(String, IAFAState)\n" +
2745	" ^^^^^^^^^\n" +
2746	"Javadoc: IAFAState cannot be resolved or is not a field\n" +
2747	"----------\n"
2748	);
2749	}
2750	public void testBug103304c() {
2751	runNegativeTest(
2752	new String[] {
2753	"test/Test.java",
2754	"package test;\n" +
2755	"public interface Test {\n" +
2756	" public class Level0 {\n" +
2757	" public Level0() {}\n" +
2758	" }\n" +
2759	" public interface Member {\n" +
2760	" public class Level1 {\n" +
2761	" public Level1() {}\n" +
2762	" }\n" +
2763	" }\n" +
2764	"}\n",
2765	"test/C.java",
2766	"package test;\n" +
2767	"public class C {\n" +
2768	" /**\n" +
2769	" * @see Test.Level0#Test.Level0()\n" +
2770	" */\n" +
2771	" Test.Level0 valid = new Test.Level0();\n" +
2772	" /**\n" +
2773	" * @see Test.Level0#Level0()\n" +
2774	" */\n" +
2775	" Test.Level0 invalid = new Test.Level0();\n" +
2776	"}\n"
2777	},
2778	// test\C.java:10: warning - Tag @see: can't find Level0() in test.Test.Level0
2779	"----------\n" +
2780	"1. ERROR in test\\C.java (at line 8)\n" +
2781	" * @see Test.Level0#Level0()\n" +
2782	" ^^^^^^^^\n" +
2783	"Javadoc: Invalid qualification for member type constructor\n" +
2784	"----------\n"
2785	);
2786	}
2787	public void testBug103304d() {
2788	runNegativeTest(
2789	new String[] {
2790	"test/Test.java",
2791	"package test;\n" +
2792	"public interface Test {\n" +
2793	" public class Level0 {\n" +
2794	" public Level0() {}\n" +
2795	" }\n" +
2796	" public interface Member {\n" +
2797	" public class Level1 {\n" +
2798	" public Level1() {}\n" +
2799	" }\n" +
2800	" }\n" +
2801	"}\n",
2802	"test/C2.java",
2803	"package test;\n" +
2804	"public class C2 {\n" +
2805	" /**\n" +
2806	" * @see Test.Member.Level1#Test.Member.Level1()\n" +
2807	" */\n" +
2808	" Test.Member.Level1 valid = new Test.Member.Level1();\n" +
2809	" /**\n" +
2810	" * @see Test.Member.Level1#Level1()\n" +
2811	" */\n" +
2812	" Test.Member.Level1 invalid = new Test.Member.Level1();\n" +
2813	" /**\n" +
2814	" * @see Test.Member.Level1#Test.Level1()\n" +
2815	" */\n" +
2816	" Test.Member.Level1 wrong = new Test.Member.Level1();\n" +
2817	"}\n"
2818	},
2819	// test\C2.java:10: warning - Tag @see: can't find Level1() in test.Test.Member.Level1
2820	// test\C2.java:14: warning - Tag @see: can't find Test.Level1() in test.Test.Member.Level1
2821	"----------\n" +
2822	"1. ERROR in test\\C2.java (at line 8)\n" +
2823	" * @see Test.Member.Level1#Level1()\n" +
2824	" ^^^^^^^^\n" +
2825	"Javadoc: Invalid qualification for member type constructor\n" +
2826	"----------\n" +
2827	"2. ERROR in test\\C2.java (at line 12)\n" +
2828	" * @see Test.Member.Level1#Test.Level1()\n" +
2829	" ^^^^^^^^^^^^^\n" +
2830	"Javadoc: Invalid qualification for member type constructor\n" +
2831	"----------\n"
2832	);
2833	}
2834	public void testBug103304e() {
2835	runConformTest(
2836	new String[] {
2837	"implicit/Valid.java",
2838	"package implicit;\n" +
2839	"public interface Valid {\n" +
2840	" public class Level0 {\n" +
2841	" /**\n" +
2842	" * @see #Valid.Level0() Valid\n" +
2843	" */\n" +
2844	" public Level0() {}\n" +
2845	" /**\n" +
2846	" * @see #Valid.Level0(String) Valid\n" +
2847	" */\n" +
2848	" public Level0(String str) {}\n" +
2849	" }\n" +
2850	" public interface Member {\n" +
2851	" public class Level1 {\n" +
2852	" /**\n" +
2853	" * @see #Valid.Member.Level1() Valid\n" +
2854	" */\n" +
2855	" public Level1() {}\n" +
2856	" /**\n" +
2857	" * @see #Valid.Member.Level1(int) Valid\n" +
2858	" */\n" +
2859	" public Level1(int x) {}\n" +
2860	" }\n" +
2861	" }\n" +
2862	"}\n"
2863	}
2864	);
2865	}
2866	public void testBug103304f() {
2867	runNegativeTest(
2868	new String[] {
2869	"implicit/Invalid.java",
2870	"package implicit;\n" +
2871	"public interface Invalid {\n" +
2872	" public class Level0 {\n" +
2873	" /**\n" +
2874	" * @see #Level0() Invalid\n" +
2875	" */\n" +
2876	" public Level0() {}\n" +
2877	" /**\n" +
2878	" * @see #Level0(String) Invalid\n" +
2879	" */\n" +
2880	" public Level0(String str) {}\n" +
2881	" }\n" +
2882	" public interface Member {\n" +
2883	" public class Level1 {\n" +
2884	" /**\n" +
2885	" * @see #Level1() Invalid\n" +
2886	" * @see #Member.Level1() Invalid\n" +
2887	" * @see #Invalid.Level1() Invalid\n" +
2888	" */\n" +
2889	" public Level1() {}\n" +
2890	" /**\n" +
2891	" * @see #Level1(int) Invalid\n" +
2892	" * @see #Invalid.Level1(int) Invalid\n" +
2893	" * @see #Member.Level1(int) Invalid\n" +
2894	" */\n" +
2895	" public Level1(int x) {}\n" +
2896	" }\n" +
2897	" }\n" +
2898	"}\n"
2899	},
2900	// implicit\Invalid.java:7: warning - Tag @see: can't find Level0() in implicit.Invalid.Level0
2901	// implicit\Invalid.java:11: warning - Tag @see: can't find Level0(String) in implicit.Invalid.Level0
2902	// implicit\Invalid.java:20: warning - Tag @see: can't find Level1() in implicit.Invalid.Member.Level1
2903	// implicit\Invalid.java:20: warning - Tag @see: can't find Member.Level1() in implicit.Invalid.Member.Level1
2904	// implicit\Invalid.java:20: warning - Tag @see: can't find Invalid.Level1() in implicit.Invalid.Member.Level1
2905	// implicit\Invalid.java:26: warning - Tag @see: can't find Level1(int) in implicit.Invalid.Member.Level1
2906	// implicit\Invalid.java:26: warning - Tag @see: can't find Invalid.Level1(int) in implicit.Invalid.Member.Level1
2907	// implicit\Invalid.java:26: warning - Tag @see: can't find Member.Level1(int) in implicit.Invalid.Member.Level1
2908	"----------\n" +
2909	"1. ERROR in implicit\\Invalid.java (at line 5)\n" +
2910	" * @see #Level0() Invalid\n" +
2911	" ^^^^^^^^\n" +
2912	"Javadoc: Invalid qualification for member type constructor\n" +
2913	"----------\n" +
2914	"2. ERROR in implicit\\Invalid.java (at line 9)\n" +
2915	" * @see #Level0(String) Invalid\n" +
2916	" ^^^^^^^^^^^^^^\n" +
2917	"Javadoc: Invalid qualification for member type constructor\n" +
2918	"----------\n" +
2919	"3. ERROR in implicit\\Invalid.java (at line 16)\n" +
2920	" * @see #Level1() Invalid\n" +
2921	" ^^^^^^^^\n" +
2922	"Javadoc: Invalid qualification for member type constructor\n" +
2923	"----------\n" +
2924	"4. ERROR in implicit\\Invalid.java (at line 17)\n" +
2925	" * @see #Member.Level1() Invalid\n" +
2926	" ^^^^^^^^^^^^^^^\n" +
2927	"Javadoc: Invalid qualification for member type constructor\n" +
2928	"----------\n" +
2929	"5. ERROR in implicit\\Invalid.java (at line 18)\n" +
2930	" * @see #Invalid.Level1() Invalid\n" +
2931	" ^^^^^^^^^^^^^^^^\n" +
2932	"Javadoc: Invalid qualification for member type constructor\n" +
2933	"----------\n" +
2934	"6. ERROR in implicit\\Invalid.java (at line 22)\n" +
2935	" * @see #Level1(int) Invalid\n" +
2936	" ^^^^^^^^^^^\n" +
2937	"Javadoc: Invalid qualification for member type constructor\n" +
2938	"----------\n" +
2939	"7. ERROR in implicit\\Invalid.java (at line 23)\n" +
2940	" * @see #Invalid.Level1(int) Invalid\n" +
2941	" ^^^^^^^^^^^^^^^^^^^\n" +
2942	"Javadoc: Invalid qualification for member type constructor\n" +
2943	"----------\n" +
2944	"8. ERROR in implicit\\Invalid.java (at line 24)\n" +
2945	" * @see #Member.Level1(int) Invalid\n" +
2946	" ^^^^^^^^^^^^^^^^^^\n" +
2947	"Javadoc: Invalid qualification for member type constructor\n" +
2948	"----------\n"
2949	);
2950	}
2951
2952	/**
2153	* Bug 112346: [javadoc] {@inheritedDoc} should be inactive for non-overridden method	2953	* Bug 112346: [javadoc] {@inheritedDoc} should be inactive for non-overridden method
2154	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=112346"	2954	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=112346"
2155	*/	2955	*/




		this.runConformReferenceTest(
			new String[] {
				"test/IX.java",
				"package test;\n" +
				"import test.copy.*;\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"public interface IX {\n" +
				"	public void foo();\n" +
				"}\n"
			}
		);
	}

	public void test023() {

		this.runConformReferenceTest(
			new String[] {
				"test/IX.java",
				"package test;\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"public interface IX {\n" +
				"	public void foo();\n" +
				"}\n"
			}
		);
	}

	// @see Field references

		this.runConformReferenceTest(
			new String[] {
				"test/IX.java",
				"package test;\n" +
				"import test.copy.*;\n" +
				"public interface IX {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public void foo();\n" +
				"}\n"
				}
			);
	}

	public void test098() {

		this.runConformReferenceTest(
			new String[] {
				"test/IX.java",
				"package test;\n" +
				"public interface IX {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public void foo();\n" +
				"}\n"
			}
		);
	}

	// @see tag: field references




		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"import test.copy.*;\n" +
				"public class X {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public int x;\n" +
				"}\n"
			}
		);
	}

	public void test023() {

		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"public class X {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public int x;\n" +
				"}\n"
				}
			);
	}

	// @see Field references




		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"import test.copy.*;\n" +
				"public class X {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public X() {\n" +
				"	}\n" +
				"}\n"
			}
		);
	}

	public void test023() {

		this.runConformReferenceTest(
			new String[] {
				"test/X.java",
				"package test;\n" +
				"public class X {\n" +
				"	/**\n" +
				"	 * Valid external classes references \n" +
				"	 *\n" +
				"	 * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
				// Inner classes are not visible in generated documentation
				//"	 * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
				"	 */\n" +
				"	public X() {\n" +
				"	}\n" +
				"}\n"
			}
		);
	}

	// @see Field references




		// Run test cases subset
		COPY_DIR = false;
		System.err.println("WARNING: only subset of tests will be executed!!!");
		suite.addTest(new ASTConverterJavadocTest("testBug103304"));
		return suite;
	}


					}
					// Verify member name position
					Name name = methodRef.getName();
					int nameLength = name.getLength();
					text = new String(source, start, nameLength);
					if (!text.equals(name.toString())) { // may have qualified constructor reference for inner classes
						if (methodRef.getQualifier().isQualifiedName()) {
							text = new String(source, start, methodRef.getQualifier().getLength());
							assumeEquals(prefix+"Misplaced method ref name at <"+start+">: ", text, methodRef.getQualifier().toString());
							while (source[start] != '.' || Character.isWhitespace(source[start])) {
								start++; // purge non-stored characters
							}
							start++;
						} else {
							while (source[start] != '.' || Character.isWhitespace(source[start])) {
								start++; // purge non-stored characters
							}
							start++;
							text = new String(source, start, nameLength);
							assumeEquals(prefix+"Misplaced method ref name at <"+start+">: ", text, name.toString());
						}
					}
					verifyNamePositions(start, name, source);
					start += nameLength;
					// Verify arguments starting open parenthesis
					while (source[start] == '*' || Character.isWhitespace(source[start])) {
						start++; // purge non-stored characters

	}

	/**
	 * @bug 103304: [Javadoc] Wrong reference proposal for inner classes.
	 * @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=103304"
	 */
	public void testBug103304() throws JavaModelException {
		this.packageBinding = false; // do NOT verify that qualification only can be package name
		workingCopies = new ICompilationUnit[1];
		workingCopies[0] = getWorkingCopy("/Converter15/src/javadoc/b103304/Test.java",
			"package javadoc.b103304;\n" + 
			"interface IAFAState {\n" + 
			"    public class ValidationException extends Exception {\n" + 
			"        public ValidationException(String variableName, IAFAState subformula) {\n" + 
			"            super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" + 
			"        }\n" + 
			"    }\n" + 
			"}\n" +
			"public class Test {\n" + 
			"	/**\n" + 
			"	 * @see IAFAState.ValidationException#IAFAState.ValidationException(String, IAFAState)\n" + 
			"	 */\n" + 
			"	IAFAState.ValidationException valid;\n" + 
			"}\n"
		);
		CompilationUnit compilUnit = (CompilationUnit) runConversion(workingCopies[0], true);
		verifyWorkingCopiesComments();
		if (docCommentSupport.equals(JavaCore.ENABLED)) {
			// Verify comment type
			Iterator unitComments = compilUnit.getCommentList().iterator();
			while (unitComments.hasNext()) {
				Comment comment = (Comment) unitComments.next();
				assertEquals("Comment should be javadoc", comment.getNodeType(), ASTNode.JAVADOC);
				Javadoc javadoc = (Javadoc) comment;

				// Verify that there's always a method reference in tags
				List tags = javadoc.tags();
				int size = tags.size();
				for (int i=0; i<size; i++) {
					TagElement tag = (TagElement) javadoc.tags().get(i);				
					assertEquals("Invalid number of fragment for see reference: "+tag, 1, tag.fragments().size());
					ASTNode node = (ASTNode) tag.fragments().get(0);
					assertEquals("Invalid kind of name reference for tag element: "+tag, ASTNode.METHOD_REF, node.getNodeType());
				}
			}
		}
	}

	/**
	 * Bug 106581: [javadoc] null type binding for parameter in javadoc
	 * @see "https://bugs.eclipse.org/bugs/show_bug.cgi?id=106581"
	 */




		this.sourceStart = sourceStart;
		this.sourceEnd = sourceEnd;
	}
	/**
	 * Returns whether a type can be seen at a given visibility level or not.
	 *
	 * @param visibility Level of visiblity allowed to see references
	 * @param binding Type to be seen
	 * @return true if the type can be seen, false otherwise
	 */
	boolean canBeSeen(int visibility, ReferenceBinding binding) {
		int modifiers = binding.modifiers;
		if (!binding.isValidBinding()) {
			ProblemReferenceBinding problemBinding = (ProblemReferenceBinding) binding;
			if (problemBinding.closestMatch != null) {
				modifiers = problemBinding.closestMatch.modifiers;
			}
		}
		if (modifiers < 0) return true;
		switch (modifiers & ExtraCompilerModifiers.AccVisibilityMASK) {
			case ClassFileConstants.AccPublic :
				return true;
			case ClassFileConstants.AccProtected:
				return (visibility != ClassFileConstants.AccPublic);
			case ClassFileConstants.AccDefault:
				return (visibility == ClassFileConstants.AccDefault || visibility == ClassFileConstants.AccPrivate);
			case ClassFileConstants.AccPrivate:
				return (visibility == ClassFileConstants.AccPrivate);
		}
		return true;
	}

	/*
	 * @see org.eclipse.jdt.internal.compiler.ast.ASTNode#print(int, java.lang.StringBuffer)
	 */

		}

		// Verify field references
		boolean source15 = scope.compilerOptions().sourceLevel >= ClassFileConstants.JDK1_5;
		int scopeModifiers = -1;
		if (reference instanceof JavadocFieldReference) {
			JavadocFieldReference fieldRef = (JavadocFieldReference) reference;
			int modifiers = fieldRef.binding==null ? -1 : fieldRef.binding.modifiers;
			
			// Verify if this is a method reference
			// see bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=51911
			if (fieldRef.methodBinding != null) {
				// cannot refer to method for @value tag
				if (fieldRef.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) {
					if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
					scope.problemReporter().javadocInvalidValueReference(fieldRef.sourceStart, fieldRef.sourceEnd, scopeModifiers);
				}
				else if (fieldRef.receiverType != null) {
					fieldRef.superAccess = scope.enclosingSourceType().isCompatibleWith(fieldRef.receiverType);

			}

			// Verify whether field ref should be static or not (for @value tags)
			else if (source15 && fieldRef.binding != null && fieldRef.binding.isValidBinding()) {
				if (fieldRef.tagValue == JavadocTagConstants.TAG_VALUE_VALUE && !fieldRef.binding.isStatic()) {
					if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
					scope.problemReporter().javadocInvalidValueReference(fieldRef.sourceStart, fieldRef.sourceEnd, scopeModifiers);
				}
			}

			// That's it for field references
			return;
		}

		// Verify type references
		if ((reference instanceof JavadocSingleTypeReference || reference instanceof JavadocQualifiedTypeReference) && reference.resolvedType instanceof ReferenceBinding) {
			ReferenceBinding resolvedType = (ReferenceBinding) reference.resolvedType;
			if (reference.resolvedType.isValidBinding()) {

				// member types
				if (resolvedType.isMemberType()) {
					ReferenceBinding topLevelType = resolvedType;
					int depth = 0;
					while (topLevelType.enclosingType() != null) {
						topLevelType = topLevelType.enclosingType();
						depth++;
					}
					ClassScope topLevelScope = scope.classScope();
					// when scope is not on compilation unit type, then inner class may not be visible...
					if (topLevelScope.parent.kind != Scope.COMPILATION_UNIT_SCOPE ||
						!CharOperation.equals(topLevelType.sourceName, topLevelScope.referenceContext.name)) {
						topLevelScope = topLevelScope.outerMostClassScope();
						if (reference instanceof JavadocSingleTypeReference) {
							// inner class single reference can only be done in same unit
							if ((!source15 && depth == 1) || topLevelType != topLevelScope.referenceContext.binding) {
								if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
								scope.problemReporter().javadocNotVisibleReference(reference.sourceStart, reference.sourceEnd, scopeModifiers);
							}
						} else {
							// inner class qualified reference can only be done in same package
							if (topLevelType.getPackage() != topLevelScope.referenceContext.binding.getPackage()) {
								if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
								scope.problemReporter().javadocNotVisibleReference(reference.sourceStart, reference.sourceEnd, scopeModifiers);
							}
						}
					}
				}

				// reference must have enough visibility to be used
				if (!canBeSeen(scope.problemReporter().options.reportInvalidJavadocTagsVisibility, resolvedType)) {
					if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
					scope.problemReporter().javadocNotVisibleReference(reference.sourceStart, reference.sourceEnd, scopeModifiers);
				}
			}
		}

		// Verify that message reference are not used for @value tags
		if (reference instanceof JavadocMessageSend) {
			JavadocMessageSend msgSend = (JavadocMessageSend) reference;

			// tag value
			if (source15 && msgSend.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) { // cannot refer to method for @value tag
				if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
				scope.problemReporter().javadocInvalidValueReference(msgSend.sourceStart, msgSend.sourceEnd, scopeModifiers);
			}

		}

		// Verify that constructor reference are not used for @value tags
		else if (reference instanceof JavadocAllocationExpression) {
			JavadocAllocationExpression alloc = (JavadocAllocationExpression) reference;

			// tag value
			if (source15 && alloc.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) { // cannot refer to method for @value tag
				if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
				scope.problemReporter().javadocInvalidValueReference(alloc.sourceStart, alloc.sourceEnd, scopeModifiers);
			}

		}
		
		// Verify that there's no type variable reference




 *******************************************************************************/
package org.eclipse.jdt.internal.compiler.ast;

import org.eclipse.jdt.core.compiler.CharOperation;
import org.eclipse.jdt.internal.compiler.impl.Constant;
import org.eclipse.jdt.internal.compiler.lookup.*;

public class JavadocAllocationExpression extends AllocationExpression {

	public int tagSourceStart, tagSourceEnd;
	public int tagValue, memberStart;
	public boolean superAccess = false;
	public char[][] qualification;

	public JavadocAllocationExpression(int start, int end) {
		this.sourceStart = start;
		this.sourceEnd = end;

		this((int) (pos >>> 32), (int) pos);
	}

	TypeBinding internalResolveType(Scope scope) {
	
		// Propagate the type checking to the arguments, and check if the constructor is defined.
		this.constant = Constant.NotAConstant;

					}
				}
			}
		} else if (this.resolvedType.isMemberType()) { // inner class constructor reference must be qualified
			int length = qualification.length;
			ReferenceBinding enclosingTypeBinding = allocationType;
			if (type instanceof JavadocQualifiedTypeReference && (((JavadocQualifiedTypeReference)type).tokens.length != length)) {
				scope.problemReporter().javadocInvalidConstructorQualification(this.memberStart+1, this.sourceEnd); //, scope.getDeclarationModifiers());
			} else {
				int idx = length;
				while (idx > 0 && CharOperation.equals(qualification[--idx], enclosingTypeBinding.sourceName) && (enclosingTypeBinding = enclosingTypeBinding.enclosingType()) != null) {
					// verify that each qualification token matches enclosing types
				}
				if (idx > 0 || enclosingTypeBinding != null) {
					scope.problemReporter().javadocInvalidConstructorQualification(this.memberStart+1, this.sourceEnd); //, scope.getDeclarationModifiers());
				}
			}
		}
		if (isMethodUseDeprecated(this.binding, scope, true)) {
			scope.problemReporter().javadocDeprecatedMethod(this.binding, this, scope.getDeclarationModifiers());




	/*
	 * Javadoc comments
	 */
	/** 
	 * Problem signaled on a reference to an java element which has a too low visibility level.
	 * @since 3.3
	 */
	int JavadocNotVisibleReference = Javadoc + Internal + 465;
	/** 
	 * Problem signaled on an unqualified inner class constructor reference.
	 * @since 3.3
	 */
	int JavadocInvalidConstructorQualification = Javadoc + Internal + 466;
	/** @since 3.2 */
	int JavadocMissingIdentifier = Javadoc + Internal + 467;
	/** @since 3.2 */




	 * Otherwise return null as we do not need this reference.
	 */
	protected Object createMethodReference(Object receiver, List arguments) throws InvalidInputException {
		int memberPtr = this.identifierLengthStack[0] - 1;	// may be > 0 for inner class constructor reference
		int start = (int) (this.identifierPositionStack[memberPtr] >>> 32);
		int end = (int) this.identifierPositionStack[memberPtr];
		if (start <= this.selectionStart && this.selectionEnd <= end) {
			selectedNode = (ASTNode) super.createMethodReference(receiver, arguments);
			this.abort = true;

					if (start <= this.selectionStart && this.selectionEnd <= end) {
						int pos = i + 1;
						char[][] tokens = new char[pos][];
						int ptr = this.identifierPtr - size;
						System.arraycopy(this.identifierStack, ptr, tokens, 0, pos);
						long[] positions = new long[pos];
						System.arraycopy(this.identifierPositionStack, ptr, positions, 0, pos);
						selectedNode = new JavadocQualifiedTypeReference(tokens, positions, this.tagSourceStart, this.tagSourceEnd);
						this.abort = true; // we got selected node => cancel parse
						if (SelectionEngine.DEBUG) {

Lines 362-367 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/problem/messages.properties (+2 lines)
362	460 = Empty block should be documented	362	460 = Empty block should be documented
363		363
364	### DOC	364	### DOC
		365	465 = Not visible reference
		366	466 = Invalid qualification for member type constructor
365	467 = Missing identifier	367	467 = Missing identifier
366	468 = Cannot make a static reference to the non-static type variable {0}	368	468 = Cannot make a static reference to the non-static type variable {0}
367	469 = Invalid param tag type parameter name	369	469 = Invalid param tag type parameter name




			case IProblem.JavadocDuplicateParamName:
			case IProblem.JavadocMissingParamName:
			case IProblem.JavadocMissingIdentifier:
			case IProblem.JavadocInvalidConstructorQualification:
			case IProblem.JavadocInvalidThrowsClassName:
			case IProblem.JavadocDuplicateThrowsClassName:
			case IProblem.JavadocMissingThrowsClassName:

			case IProblem.JavadocUsingDeprecatedConstructor:
			case IProblem.JavadocUsingDeprecatedMethod:
			case IProblem.JavadocUsingDeprecatedType:
			case IProblem.JavadocNotVisibleReference:
				return CompilerOptions.InvalidJavadoc;
	
			case IProblem.JavadocMissingParamTag:

		case IProblem.JavadocDuplicateParamName:
		case IProblem.JavadocMissingParamName:
		case IProblem.JavadocMissingIdentifier:
		case IProblem.JavadocInvalidConstructorQualification:
		case IProblem.JavadocInvalidThrowsClassName:
		case IProblem.JavadocDuplicateThrowsClassName:
		case IProblem.JavadocMissingThrowsClassName:

		case IProblem.JavadocNotVisibleConstructor:
		case IProblem.JavadocNotVisibleMethod:
		case IProblem.JavadocNotVisibleType:
		case IProblem.JavadocNotVisibleReference:
			if (!(this.options.reportInvalidJavadocTags && this.options.reportInvalidJavadocTagsNotVisibleRef)) {
				return ProblemSeverities.Ignore;			
			}

	int severity = computeSeverity(IProblem.JavadocUsingDeprecatedType);
	if (severity == ProblemSeverities.Ignore) return;
	if (javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, modifiers)) {
		if (type.isMemberType() && type instanceof ReferenceBinding && !javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, ((ReferenceBinding)type).modifiers)) {
			this.handle(IProblem.JavadocNotVisibleReference, NoArgument, NoArgument, location.sourceStart, location.sourceEnd);
		} else {
			this.handle(
				IProblem.JavadocUsingDeprecatedType,
				new String[] {new String(type.readableName())},
				new String[] {new String(type.shortReadableName())},
				severity,
				location.sourceStart,
				location.sourceEnd);
		}
	}
}
public void javadocDuplicatedParamTag(char[] token, int sourceStart, int sourceEnd, int modifiers) {

		statement.sourceStart,
		statement.sourceEnd);
}
public void javadocInvalidConstructorQualification(int sourceStart, int sourceEnd){
	this.handle(IProblem.JavadocInvalidConstructorQualification, NoArgument, NoArgument, sourceStart, sourceEnd);
}
/*
 * Similar implementation than invalidField(FieldReference...)
 * Note that following problem id cannot occur for Javadoc:

			typeRef.sourceEnd);
	}
}
public void javadocNotVisibleReference(int sourceStart, int sourceEnd, int modifiers) {
	if (javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, modifiers))
		this.handle(IProblem.JavadocNotVisibleReference, NoArgument, NoArgument, sourceStart, sourceEnd);
}
public void javadocUndeclaredParamTagName(char[] token, int sourceStart, int sourceEnd, int modifiers) {
	int severity = computeSeverity(IProblem.JavadocInvalidParamName);
	if (severity == ProblemSeverities.Ignore) return;




			TypeReference typeRef = (TypeReference) receiver;
			// Decide whether we have a constructor or not
			boolean isConstructor = false;
			int length = this.identifierLengthStack[0];	// may be > 0 for inner class constructor reference
			if (typeRef == null) {
				char[] name = this.sourceParser.compilationUnit.getMainTypeName();
				TypeDeclaration typeDecl = getParsedTypeDeclaration();
				if (typeDecl != null) {
					name = typeDecl.name;
				}
				isConstructor = CharOperation.equals(this.identifierStack[length-1], name);
				typeRef = new JavadocImplicitTypeReference(name, this.memberStart);
			} else {
				char[] name = null;
				if (typeRef instanceof JavadocSingleTypeReference) {
					char[] name = ((JavadocSingleTypeReference)typeRef).token;
					isConstructor = CharOperation.equals(this.identifierStack[length-1], name);
				} else if (typeRef instanceof JavadocQualifiedTypeReference) {
					char[][] tokens = ((JavadocQualifiedTypeReference)typeRef).tokens;
					int last = tokens.length-1;
					isConstructor = CharOperation.equals(this.identifierStack[length-1], tokens[last]);
					if (isConstructor) {
						boolean valid = true;
						if (valid) {
							for (int i=0; i<length-1 && valid; i++) {
								valid = CharOperation.equals(this.identifierStack[i], tokens[i]);
							}
						}
						if (!valid) {
							if (this.reportProblems) {
								this.sourceParser.problemReporter().javadocInvalidConstructorQualification(this.memberStart+1, this.scanner.getCurrentTokenEndPosition());
							}
							return null;
						}
					}
				} else {
					throw new InvalidInputException();
				}
				isConstructor = CharOperation.equals(this.identifierStack[0], name);
			}
			// Create node
			if (arguments == null) {
				if (isConstructor) {
					JavadocAllocationExpression allocation = new JavadocAllocationExpression(this.identifierPositionStack[length-1]);
					allocation.type = typeRef;
					allocation.tagValue = this.tagValue;
					allocation.sourceEnd = this.scanner.getCurrentTokenEndPosition();
					if (length == 1) {
						allocation.qualification = new char[][] { this.identifierStack[0] };
					} else {
						System.arraycopy(this.identifierStack, 0, allocation.qualification = new char[length][], 0, length);
					}
					allocation.memberStart = this.memberStart;
					return allocation;
				} else {
					JavadocMessageSend msg = new JavadocMessageSend(this.identifierStack[length-1], this.identifierPositionStack[length-1]);
					msg.receiver = typeRef;
					msg.tagValue = this.tagValue;
					msg.sourceEnd = this.scanner.getCurrentTokenEndPosition();

				JavadocArgumentExpression[] expressions = new JavadocArgumentExpression[arguments.size()];
				arguments.toArray(expressions);
				if (isConstructor) {
					JavadocAllocationExpression allocation = new JavadocAllocationExpression(this.identifierPositionStack[length-1]);
					allocation.arguments = expressions;
					allocation.type = typeRef;
					allocation.tagValue = this.tagValue;
					allocation.sourceEnd = this.scanner.getCurrentTokenEndPosition();
					if (length == 1) {
						allocation.qualification = new char[][] { this.identifierStack[0] };
					} else {
						System.arraycopy(this.identifierStack, 0, allocation.qualification = new char[length][], 0, length);
					}
					allocation.memberStart = this.memberStart;
					return allocation;
				} else {
					JavadocMessageSend msg = new JavadocMessageSend(this.identifierStack[length-1], this.identifierPositionStack[length-1], expressions);
					msg.receiver = typeRef;
					msg.tagValue = this.tagValue;
					msg.sourceEnd = this.scanner.getCurrentTokenEndPosition();

	 */
	protected Object createTypeReference(int primitiveToken) {
		TypeReference typeRef = null;
		int size = this.identifierLengthStack[this.identifierLengthPtr];
		if (size == 1) { // Single Type ref
			typeRef = new JavadocSingleTypeReference(
						this.identifierStack[this.identifierPtr],

			System.arraycopy(this.identifierPositionStack, this.identifierPtr - size + 1, positions, 0, size);
			typeRef = new JavadocQualifiedTypeReference(tokens, positions, this.tagSourceStart, this.tagSourceEnd);
		}
		this.identifierPtr -= size;
		return typeRef;
	}


	 * Fill associated comment fields with ast nodes information stored in stack.
	 */
	protected void updateDocComment() {

		// Set inherited positions
		this.docComment.inheritedPositions = this.inheritedPositions;


Lines 641-648 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/parser/AbstractCommentParser.java (-2 / +6 lines)
641		641
642	// Get member identifier	642	// Get member identifier
643	if (readToken() == TerminalTokens.TokenNameIdentifier) {	643	if (readToken() == TerminalTokens.TokenNameIdentifier) {
644	consumeToken();	644	if (this.scanner.currentCharacter == '.') { // member name may be qualified (inner class constructor reference)
645	pushIdentifier(true, false);	645	parseQualifiedName(true);
		646	} else {
		647	consumeToken();
		648	pushIdentifier(true, false);
		649	}
646	// Look for next token to know whether it's a field or method reference	650	// Look for next token to know whether it's a field or method reference
647	int previousPosition = this.index;	651	int previousPosition = this.index;
648	if (readToken() == TerminalTokens.TokenNameLPAREN) {	652	if (readToken() == TerminalTokens.TokenNameLPAREN) {

Lines 23-28 Link Here

(-)codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionOnJavadocAllocationExpression.java (+1 lines)
23	this.tagValue = allocation.tagValue;	23	this.tagValue = allocation.tagValue;
24	this.sourceEnd = allocation.sourceEnd;	24	this.sourceEnd = allocation.sourceEnd;
25	this.separatorPosition = position;	25	this.separatorPosition = position;
		26	this.qualification = allocation.qualification;
26	}	27	}
27		28
28	public CompletionOnJavadocAllocationExpression(JavadocAllocationExpression allocation, int position, int flags) {	29	public CompletionOnJavadocAllocationExpression(JavadocAllocationExpression allocation, int position, int flags) {




	 * Otherwise return null as we do not need this reference.
	 */
	protected Object createMethodReference(Object receiver, List arguments) throws InvalidInputException {
		int memberPtr = this.identifierLengthStack[0] - 1; // may be > 0 for inner class constructor reference
		int refStart = (int) (this.identifierPositionStack[memberPtr] >>> 32);
		int refEnd = (int) this.identifierPositionStack[memberPtr];
		boolean inCompletion = (refStart <= (this.cursorLocation+1) && this.cursorLocation <= refEnd) // completion cursor is between first and last stacked identifiers
			|| ((refStart == (refEnd+1) && refEnd == this.cursorLocation)) // or it's a completion on empty token
			|| (this.memberStart == this.cursorLocation); // or it's a completion just after the member separator with an identifier after the cursor

				this.completionNode = new CompletionOnJavadocQualifiedTypeReference(tokens, this.identifierStack[this.identifierPtr], positions, this.tagSourceStart, this.tagSourceEnd);
			}
		}
		this.identifierPtr -= nbIdentifiers;

		if (CompletionEngine.DEBUG) {
			System.out.println("	completion partial qualified type="+completionNode); //$NON-NLS-1$

			// special case of completion just before the dot.
			return createTypeReference(primitiveToken);
		}
		int idLength = this.identifierLengthStack[this.identifierLengthPtr];
		char[][] tokens = new char[idLength][];
		int startPtr = this.identifierPtr-idLength+1;
		System.arraycopy(this.identifierStack, startPtr, tokens, 0, idLength);

		System.arraycopy(this.identifierPositionStack, startPtr, positions, 0, idLength);
		positions[idLength] = (((long)this.tokenPreviousPosition)<<32) + this.tokenPreviousPosition;
		this.completionNode = new CompletionOnJavadocQualifiedTypeReference(tokens, CharOperation.NO_CHAR, positions, this.tagSourceStart, this.tagSourceEnd);
		this.identifierPtr -= idLength;

		if (CompletionEngine.DEBUG) {
			System.out.println("	completion partial qualified type="+completionNode); //$NON-NLS-1$




		buffer.append(super.toString());
		return buffer.toString();
	}

	/* (non-Javadoc)
	 * @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#createArgumentReference(char[], java.lang.Object, int)
	 */

			// Create method ref
			MethodRef methodRef = this.ast.newMethodRef();
			SimpleName methodName = new SimpleName(this.ast);
			int memberPtr = this.identifierLengthStack[0] - 1;
			methodName.internalSetIdentifier(new String(this.identifierStack[memberPtr]));
			methodRef.setName(methodName);
			int start = (int) (this.identifierPositionStack[memberPtr] >>> 32);
			int end = (int) this.identifierPositionStack[memberPtr];
			methodName.setSourceRange(start, end - start + 1);
			// Set qualifier
//			int end = methodName.getStartPosition()+methodName.getLength()-1;
			if (receiver == null) {
				start = this.memberStart;
				methodRef.setSourceRange(start, end - start + 1);

				while (parameters.hasNext()) {
					MethodRefParameter param = (MethodRefParameter) parameters.next();
					methodRef.parameters().add(param);
//					end = param.getStartPosition()+param.getLength()-1;
				}
			}
//			methodRef.setSourceRange(start, end-start+1);
			methodRef.setSourceRange(start, this.scanner.getCurrentTokenEndPosition()-start+1);
			return methodRef;
		}

	 * @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#createTypeReference()
	 */
	protected Object createTypeReference(int primitiveToken) {
		int size = this.identifierLengthStack[this.identifierLengthPtr];
		String[] identifiers = new String[size];
		int pos = this.identifierPtr - size + 1;
		for (int i = 0; i < size; i++) {

			int end = (int) this.identifierPositionStack[pos];
			typeRef.setSourceRange(start, end-start+1);
		}
		this.identifierPtr -= size;
		return typeRef;
	}

	/*
	 * Parse @return tag declaration
	 */
	protected boolean parseReturn() {
		createTag();
		return true;
	}

	/* (non-Javadoc)
	 * @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#parseIdentifierTag(boolean)
	 */

		return false;
	}

	/*
	 * Parse @return tag declaration
	 */
	protected boolean parseReturn() {
		createTag();
		return true;
	}

	/* (non-Javadoc)
	 * @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#parseTag(int)
	 */

Return to bug 96237

Lines 2018-2035 Link Here

(-)src/org/eclipse/jdt/core/tests/compiler/regression/JavadocTestForMethod.java (-23 / +29 lines)
2018	this.runConformReferenceTest(	2018	this.runConformReferenceTest(
2019	new String[] {	2019	new String[] {
2020	"test/X.java",	2020	"test/X.java",
2021	"package test;\n"	2021	"package test;\n" +
2022	+ "import test.copy.*;\n"	2022	"import test.copy.*;\n" +
2023	+ "public class X {\n"	2023	"public class X {\n" +
2024	+ " /**\n"	2024	" /**\n" +
2025	+ " * Valid external classes references \n"	2025	" * Valid external classes references \n" +
2026	+ " *\n"	2026	" *\n" +
2027	+ " * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n"	2027	" * @see VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
2028	+ " * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n"	2028	// Inner classes are not visible in generated documentation
2029	+ " */\n"	2029	//" * @see VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
2030	+ " public void s_foo() {\n"	2030	" */\n" +
2031	+ " }\n"	2031	" public void s_foo() {\n" +
2032	+ "}\n" });	2032	" }\n" +
		2033	"}\n"
		2034	}
		2035	);
2033	}	2036	}
2034		2037
2035	public void test093() {	2038	public void test093() {
Lines 2077-2093 Link Here
2077	this.runConformReferenceTest(	2080	this.runConformReferenceTest(
2078	new String[] {	2081	new String[] {
2079	"test/X.java",	2082	"test/X.java",
2080	"package test;\n"	2083	"package test;\n" +
2081	+ "public class X {\n"	2084	"public class X {\n" +
2082	+ " /**\n"	2085	" /**\n" +
2083	+ " * Valid external classes references \n"	2086	" * Valid external classes references \n" +
2084	+ " *\n"	2087	" *\n" +
2085	+ " * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n"	2088	" * @see test.copy.VisibilityPublic Valid ref: visible class through import => no warning on import\n" +
2086	+ " * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n"	2089	// Inner classes are not visible in generated documentation
2087	+ " */\n"	2090	//" * @see test.copy.VisibilityPublic.VpPublic Valid ref: visible inner class in visible class \n" +
2088	+ " public void s_foo() {\n"	2091	" */\n" +
2089	+ " }\n"	2092	" public void s_foo() {\n" +
2090	+ "}\n" });	2093	" }\n" +
		2094	"}\n"
		2095	}
		2096	);
2091	}	2097	}
2092		2098
2093	public void test095() {	2099	public void test095() {

Lines 114-120 Link Here

(-)src/org/eclipse/jdt/core/tests/dom/ASTConverterJavadocTest.java (-4 / +67 lines)
114	// Run test cases subset	114	// Run test cases subset
115	COPY_DIR = false;	115	COPY_DIR = false;
116	System.err.println("WARNING: only subset of tests will be executed!!!");	116	System.err.println("WARNING: only subset of tests will be executed!!!");
117	suite.addTest(new ASTConverterJavadocTest("testBug130752"));	117	suite.addTest(new ASTConverterJavadocTest("testBug103304"));
118	return suite;	118	return suite;
119	}	119	}
120		120
Lines 989-998 Link Here
989	}	989	}
990	// Verify member name position	990	// Verify member name position
991	Name name = methodRef.getName();	991	Name name = methodRef.getName();
992	text = new String(source, start, name.getLength());	992	int nameLength = name.getLength();
993	assumeEquals(prefix+"Misplaced method ref name at <"+start+">: ", text, name.toString());	993	text = new String(source, start, nameLength);
		994	if (!text.equals(name.toString())) { // may have qualified constructor reference for inner classes
		995	if (methodRef.getQualifier().isQualifiedName()) {
		996	text = new String(source, start, methodRef.getQualifier().getLength());
		997	assumeEquals(prefix+"Misplaced method ref name at <"+start+">: ", text, methodRef.getQualifier().toString());
		998	while (source[start] != '.' \|\| Character.isWhitespace(source[start])) {
		999	start++; // purge non-stored characters
		1000	}
		1001	start++;
		1002	} else {
		1003	while (source[start] != '.' \|\| Character.isWhitespace(source[start])) {
		1004	start++; // purge non-stored characters
		1005	}
		1006	start++;
		1007	text = new String(source, start, nameLength);
		1008	assumeEquals(prefix+"Misplaced method ref name at <"+start+">: ", text, name.toString());
		1009	}
		1010	}
994	verifyNamePositions(start, name, source);	1011	verifyNamePositions(start, name, source);
995	start += name.getLength();	1012	start += nameLength;
996	// Verify arguments starting open parenthesis	1013	// Verify arguments starting open parenthesis
997	while (source[start] == '*' \|\| Character.isWhitespace(source[start])) {	1014	while (source[start] == '*' \|\| Character.isWhitespace(source[start])) {
998	start++; // purge non-stored characters	1015	start++; // purge non-stored characters
Lines 2989-2994 Link Here
2989	}	3006	}
2990		3007
2991	/**	3008	/**
		3009	* @bug 103304: [Javadoc] Wrong reference proposal for inner classes.
		3010	* @see "http://bugs.eclipse.org/bugs/show_bug.cgi?id=103304"
		3011	*/
		3012	public void testBug103304() throws JavaModelException {
		3013	this.packageBinding = false; // do NOT verify that qualification only can be package name
		3014	workingCopies = new ICompilationUnit[1];
		3015	workingCopies[0] = getWorkingCopy("/Converter15/src/javadoc/b103304/Test.java",
		3016	"package javadoc.b103304;\n" +
		3017	"interface IAFAState {\n" +
		3018	" public class ValidationException extends Exception {\n" +
		3019	" public ValidationException(String variableName, IAFAState subformula) {\n" +
		3020	" super(\"Variable \'\"+variableName+\"\' may be unbound in \'\"+subformula+\"\'\");\n" +
		3021	" }\n" +
		3022	" }\n" +
		3023	"}\n" +
		3024	"public class Test {\n" +
		3025	" /**\n" +
		3026	" * @see IAFAState.ValidationException#IAFAState.ValidationException(String, IAFAState)\n" +
		3027	" */\n" +
		3028	" IAFAState.ValidationException valid;\n" +
		3029	"}\n"
		3030	);
		3031	CompilationUnit compilUnit = (CompilationUnit) runConversion(workingCopies[0], true);
		3032	verifyWorkingCopiesComments();
		3033	if (docCommentSupport.equals(JavaCore.ENABLED)) {
		3034	// Verify comment type
		3035	Iterator unitComments = compilUnit.getCommentList().iterator();
		3036	while (unitComments.hasNext()) {
		3037	Comment comment = (Comment) unitComments.next();
		3038	assertEquals("Comment should be javadoc", comment.getNodeType(), ASTNode.JAVADOC);
		3039	Javadoc javadoc = (Javadoc) comment;
		3040
		3041	// Verify that there's always a method reference in tags
		3042	List tags = javadoc.tags();
		3043	int size = tags.size();
		3044	for (int i=0; i<size; i++) {
		3045	TagElement tag = (TagElement) javadoc.tags().get(i);
		3046	assertEquals("Invalid number of fragment for see reference: "+tag, 1, tag.fragments().size());
		3047	ASTNode node = (ASTNode) tag.fragments().get(0);
		3048	assertEquals("Invalid kind of name reference for tag element: "+tag, ASTNode.METHOD_REF, node.getNodeType());
		3049	}
		3050	}
		3051	}
		3052	}
		3053
		3054	/**
2992	* Bug 106581: [javadoc] null type binding for parameter in javadoc	3055	* Bug 106581: [javadoc] null type binding for parameter in javadoc
2993	* @see "https://bugs.eclipse.org/bugs/show_bug.cgi?id=106581"	3056	* @see "https://bugs.eclipse.org/bugs/show_bug.cgi?id=106581"
2994	*/	3057	*/

Lines 34-40 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/ast/Javadoc.java (-17 / +92 lines)
34	this.sourceStart = sourceStart;	34	this.sourceStart = sourceStart;
35	this.sourceEnd = sourceEnd;	35	this.sourceEnd = sourceEnd;
36	}	36	}
37		37	/**
		38	* Returns whether a type can be seen at a given visibility level or not.
		39	*
		40	* @param visibility Level of visiblity allowed to see references
		41	* @param binding Type to be seen
		42	* @return true if the type can be seen, false otherwise
		43	*/
		44	boolean canBeSeen(int visibility, ReferenceBinding binding) {
		45	int modifiers = binding.modifiers;
		46	if (!binding.isValidBinding()) {
		47	ProblemReferenceBinding problemBinding = (ProblemReferenceBinding) binding;
		48	if (problemBinding.closestMatch != null) {
		49	modifiers = problemBinding.closestMatch.modifiers;
		50	}
		51	}
		52	if (modifiers < 0) return true;
		53	switch (modifiers & ExtraCompilerModifiers.AccVisibilityMASK) {
		54	case ClassFileConstants.AccPublic :
		55	return true;
		56	case ClassFileConstants.AccProtected:
		57	return (visibility != ClassFileConstants.AccPublic);
		58	case ClassFileConstants.AccDefault:
		59	return (visibility == ClassFileConstants.AccDefault \|\| visibility == ClassFileConstants.AccPrivate);
		60	case ClassFileConstants.AccPrivate:
		61	return (visibility == ClassFileConstants.AccPrivate);
		62	}
		63	return true;
		64	}
		65
38	/*	66	/*
39	* @see org.eclipse.jdt.internal.compiler.ast.ASTNode#print(int, java.lang.StringBuffer)	67	* @see org.eclipse.jdt.internal.compiler.ast.ASTNode#print(int, java.lang.StringBuffer)
40	*/	68	*/
Lines 233-249 Link Here
233	}	261	}
234		262
235	// Verify field references	263	// Verify field references
236	boolean verifyValues = scope.compilerOptions().sourceLevel >= ClassFileConstants.JDK1_5;	264	boolean source15 = scope.compilerOptions().sourceLevel >= ClassFileConstants.JDK1_5;
		265	int scopeModifiers = -1;
237	if (reference instanceof JavadocFieldReference) {	266	if (reference instanceof JavadocFieldReference) {
238	JavadocFieldReference fieldRef = (JavadocFieldReference) reference;	267	JavadocFieldReference fieldRef = (JavadocFieldReference) reference;
239	int modifiers = fieldRef.binding==null ? -1 : fieldRef.binding.modifiers;
240		268
241	// Verify if this is a method reference	269	// Verify if this is a method reference
242	// see bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=51911	270	// see bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=51911
243	if (fieldRef.methodBinding != null) {	271	if (fieldRef.methodBinding != null) {
244	// cannot refer to method for @value tag	272	// cannot refer to method for @value tag
245	if (fieldRef.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) {	273	if (fieldRef.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) {
246	scope.problemReporter().javadocInvalidValueReference(fieldRef.sourceStart, fieldRef.sourceEnd, modifiers);	274	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		275	scope.problemReporter().javadocInvalidValueReference(fieldRef.sourceStart, fieldRef.sourceEnd, scopeModifiers);
247	}	276	}
248	else if (fieldRef.receiverType != null) {	277	else if (fieldRef.receiverType != null) {
249	fieldRef.superAccess = scope.enclosingSourceType().isCompatibleWith(fieldRef.receiverType);	278	fieldRef.superAccess = scope.enclosingSourceType().isCompatibleWith(fieldRef.receiverType);
Lines 252-286 Link Here
252	}	281	}
253		282
254	// Verify whether field ref should be static or not (for @value tags)	283	// Verify whether field ref should be static or not (for @value tags)
255	else if (verifyValues && fieldRef.binding != null && fieldRef.binding.isValidBinding()) {	284	else if (source15 && fieldRef.binding != null && fieldRef.binding.isValidBinding()) {
256	if (fieldRef.tagValue == JavadocTagConstants.TAG_VALUE_VALUE && !fieldRef.binding.isStatic()) {	285	if (fieldRef.tagValue == JavadocTagConstants.TAG_VALUE_VALUE && !fieldRef.binding.isStatic()) {
257	scope.problemReporter().javadocInvalidValueReference(fieldRef.sourceStart, fieldRef.sourceEnd, modifiers);	286	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		287	scope.problemReporter().javadocInvalidValueReference(fieldRef.sourceStart, fieldRef.sourceEnd, scopeModifiers);
258	}	288	}
259	}	289	}
260		290
261	// That's it for field references	291	// That's it for field references
262	return;	292	return;
263	}	293	}
264		294
265	// If not 1.5 level, verification is finished	295	// Verify type references
266	if (!verifyValues) return;	296	if ((reference instanceof JavadocSingleTypeReference \|\| reference instanceof JavadocQualifiedTypeReference) && reference.resolvedType instanceof ReferenceBinding) {
		297	ReferenceBinding resolvedType = (ReferenceBinding) reference.resolvedType;
		298	if (reference.resolvedType.isValidBinding()) {
		299
		300	// member types
		301	if (resolvedType.isMemberType()) {
		302	ReferenceBinding topLevelType = resolvedType;
		303	int depth = 0;
		304	while (topLevelType.enclosingType() != null) {
		305	topLevelType = topLevelType.enclosingType();
		306	depth++;
		307	}
		308	ClassScope topLevelScope = scope.classScope();
		309	// when scope is not on compilation unit type, then inner class may not be visible...
		310	if (topLevelScope.parent.kind != Scope.COMPILATION_UNIT_SCOPE \|\|
		311	!CharOperation.equals(topLevelType.sourceName, topLevelScope.referenceContext.name)) {
		312	topLevelScope = topLevelScope.outerMostClassScope();
		313	if (reference instanceof JavadocSingleTypeReference) {
		314	// inner class single reference can only be done in same unit
		315	if ((!source15 && depth == 1) \|\| topLevelType != topLevelScope.referenceContext.binding) {
		316	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		317	scope.problemReporter().javadocNotVisibleReference(reference.sourceStart, reference.sourceEnd, scopeModifiers);
		318	}
		319	} else {
		320	// inner class qualified reference can only be done in same package
		321	if (topLevelType.getPackage() != topLevelScope.referenceContext.binding.getPackage()) {
		322	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		323	scope.problemReporter().javadocNotVisibleReference(reference.sourceStart, reference.sourceEnd, scopeModifiers);
		324	}
		325	}
		326	}
		327	}
		328
		329	// reference must have enough visibility to be used
		330	if (!canBeSeen(scope.problemReporter().options.reportInvalidJavadocTagsVisibility, resolvedType)) {
		331	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		332	scope.problemReporter().javadocNotVisibleReference(reference.sourceStart, reference.sourceEnd, scopeModifiers);
		333	}
		334	}
		335	}
267		336
268	// Verify that message reference are not used for @value tags	337	// Verify that message reference are not used for @value tags
269	else if (reference instanceof JavadocMessageSend) {	338	if (reference instanceof JavadocMessageSend) {
270	JavadocMessageSend msgSend = (JavadocMessageSend) reference;	339	JavadocMessageSend msgSend = (JavadocMessageSend) reference;
271	int modifiers = msgSend.binding==null ? -1 : msgSend.binding.modifiers;	340
272	if (msgSend.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) { // cannot refer to method for @value tag	341	// tag value
273	scope.problemReporter().javadocInvalidValueReference(msgSend.sourceStart, msgSend.sourceEnd, modifiers);	342	if (source15 && msgSend.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) { // cannot refer to method for @value tag
		343	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		344	scope.problemReporter().javadocInvalidValueReference(msgSend.sourceStart, msgSend.sourceEnd, scopeModifiers);
274	}	345	}
		346
275	}	347	}
276		348
277	// Verify that constructorreference are not used for @value tags	349	// Verify that constructor reference are not used for @value tags
278	else if (reference instanceof JavadocAllocationExpression) {	350	else if (reference instanceof JavadocAllocationExpression) {
279	JavadocAllocationExpression alloc = (JavadocAllocationExpression) reference;	351	JavadocAllocationExpression alloc = (JavadocAllocationExpression) reference;
280	int modifiers = alloc.binding==null ? -1 : alloc.binding.modifiers;	352
281	if (alloc.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) { // cannot refer to method for @value tag	353	// tag value
282	scope.problemReporter().javadocInvalidValueReference(alloc.sourceStart, alloc.sourceEnd, modifiers);	354	if (source15 && alloc.tagValue == JavadocTagConstants.TAG_VALUE_VALUE) { // cannot refer to method for @value tag
		355	if (scopeModifiers == -1) scopeModifiers = scope.getDeclarationModifiers();
		356	scope.problemReporter().javadocInvalidValueReference(alloc.sourceStart, alloc.sourceEnd, scopeModifiers);
283	}	357	}
		358
284	}	359	}
285		360
286	// Verify that there's no type variable reference	361	// Verify that there's no type variable reference

Lines 10-24 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/ast/JavadocAllocationExpression.java (-3 / +19 lines)
10	*******************************************************************************/	10	*******************************************************************************/
11	package org.eclipse.jdt.internal.compiler.ast;	11	package org.eclipse.jdt.internal.compiler.ast;
12		12
		13	import org.eclipse.jdt.core.compiler.CharOperation;
13	import org.eclipse.jdt.internal.compiler.impl.Constant;	14	import org.eclipse.jdt.internal.compiler.impl.Constant;
14	import org.eclipse.jdt.internal.compiler.lookup.*;	15	import org.eclipse.jdt.internal.compiler.lookup.*;
15		16
16	public class JavadocAllocationExpression extends AllocationExpression {	17	public class JavadocAllocationExpression extends AllocationExpression {
17		18
18	public int tagSourceStart, tagSourceEnd;	19	public int tagSourceStart, tagSourceEnd;
19	public int tagValue;	20	public int tagValue, memberStart;
20	public boolean superAccess = false;	21	public boolean superAccess = false;
21		22	public char[][] qualification;
		23
22	public JavadocAllocationExpression(int start, int end) {	24	public JavadocAllocationExpression(int start, int end) {
23	this.sourceStart = start;	25	this.sourceStart = start;
24	this.sourceEnd = end;	26	this.sourceEnd = end;
Lines 28-34 Link Here
28	this((int) (pos >>> 32), (int) pos);	30	this((int) (pos >>> 32), (int) pos);
29	}	31	}
30		32
31	private TypeBinding internalResolveType(Scope scope) {	33	TypeBinding internalResolveType(Scope scope) {
32		34
33	// Propagate the type checking to the arguments, and check if the constructor is defined.	35	// Propagate the type checking to the arguments, and check if the constructor is defined.
34	this.constant = Constant.NotAConstant;	36	this.constant = Constant.NotAConstant;
Lines 120-125 Link Here
120	}	122	}
121	}	123	}
122	}	124	}
		125	} else if (this.resolvedType.isMemberType()) { // inner class constructor reference must be qualified
		126	int length = qualification.length;
		127	ReferenceBinding enclosingTypeBinding = allocationType;
		128	if (type instanceof JavadocQualifiedTypeReference && (((JavadocQualifiedTypeReference)type).tokens.length != length)) {
		129	scope.problemReporter().javadocInvalidConstructorQualification(this.memberStart+1, this.sourceEnd); //, scope.getDeclarationModifiers());
		130	} else {
		131	int idx = length;
		132	while (idx > 0 && CharOperation.equals(qualification[--idx], enclosingTypeBinding.sourceName) && (enclosingTypeBinding = enclosingTypeBinding.enclosingType()) != null) {
		133	// verify that each qualification token matches enclosing types
		134	}
		135	if (idx > 0 \|\| enclosingTypeBinding != null) {
		136	scope.problemReporter().javadocInvalidConstructorQualification(this.memberStart+1, this.sourceEnd); //, scope.getDeclarationModifiers());
		137	}
		138	}
123	}	139	}
124	if (isMethodUseDeprecated(this.binding, scope, true)) {	140	if (isMethodUseDeprecated(this.binding, scope, true)) {
125	scope.problemReporter().javadocDeprecatedMethod(this.binding, this, scope.getDeclarationModifiers());	141	scope.problemReporter().javadocDeprecatedMethod(this.binding, this, scope.getDeclarationModifiers());

Lines 753-758 Link Here

(-)compiler/org/eclipse/jdt/core/compiler/IProblem.java (+10 lines)
753	/*	753	/*
754	* Javadoc comments	754	* Javadoc comments
755	*/	755	*/
		756	/**
		757	* Problem signaled on a reference to an java element which has a too low visibility level.
		758	* @since 3.3
		759	*/
		760	int JavadocNotVisibleReference = Javadoc + Internal + 465;
		761	/**
		762	* Problem signaled on an unqualified inner class constructor reference.
		763	* @since 3.3
		764	*/
		765	int JavadocInvalidConstructorQualification = Javadoc + Internal + 466;
756	/** @since 3.2 */	766	/** @since 3.2 */
757	int JavadocMissingIdentifier = Javadoc + Internal + 467;	767	int JavadocMissingIdentifier = Javadoc + Internal + 467;
758	/** @since 3.2 */	768	/** @since 3.2 */

Lines 101-108 Link Here

(-)codeassist/org/eclipse/jdt/internal/codeassist/select/SelectionJavadocParser.java (-4 / +6 lines)
101	* Otherwise return null as we do not need this reference.	101	* Otherwise return null as we do not need this reference.
102	*/	102	*/
103	protected Object createMethodReference(Object receiver, List arguments) throws InvalidInputException {	103	protected Object createMethodReference(Object receiver, List arguments) throws InvalidInputException {
104	int start = (int) (this.identifierPositionStack[0] >>> 32);	104	int memberPtr = this.identifierLengthStack[0] - 1; // may be > 0 for inner class constructor reference
105	int end = (int) this.identifierPositionStack[0];	105	int start = (int) (this.identifierPositionStack[memberPtr] >>> 32);
		106	int end = (int) this.identifierPositionStack[memberPtr];
106	if (start <= this.selectionStart && this.selectionEnd <= end) {	107	if (start <= this.selectionStart && this.selectionEnd <= end) {
107	selectedNode = (ASTNode) super.createMethodReference(receiver, arguments);	108	selectedNode = (ASTNode) super.createMethodReference(receiver, arguments);
108	this.abort = true;	109	this.abort = true;
Lines 134-142 Link Here
134	if (start <= this.selectionStart && this.selectionEnd <= end) {	135	if (start <= this.selectionStart && this.selectionEnd <= end) {
135	int pos = i + 1;	136	int pos = i + 1;
136	char[][] tokens = new char[pos][];	137	char[][] tokens = new char[pos][];
137	System.arraycopy(this.identifierStack, this.identifierPtr+1, tokens, 0, pos);	138	int ptr = this.identifierPtr - size;
		139	System.arraycopy(this.identifierStack, ptr, tokens, 0, pos);
138	long[] positions = new long[pos];	140	long[] positions = new long[pos];
139	System.arraycopy(this.identifierPositionStack, this.identifierPtr + 1, positions, 0, pos);	141	System.arraycopy(this.identifierPositionStack, ptr, positions, 0, pos);
140	selectedNode = new JavadocQualifiedTypeReference(tokens, positions, this.tagSourceStart, this.tagSourceEnd);	142	selectedNode = new JavadocQualifiedTypeReference(tokens, positions, this.tagSourceStart, this.tagSourceEnd);
141	this.abort = true; // we got selected node => cancel parse	143	this.abort = true; // we got selected node => cancel parse
142	if (SelectionEngine.DEBUG) {	144	if (SelectionEngine.DEBUG) {

Lines 215-220 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/problem/ProblemReporter.java (-7 / +22 lines)
215	case IProblem.JavadocDuplicateParamName:	215	case IProblem.JavadocDuplicateParamName:
216	case IProblem.JavadocMissingParamName:	216	case IProblem.JavadocMissingParamName:
217	case IProblem.JavadocMissingIdentifier:	217	case IProblem.JavadocMissingIdentifier:
		218	case IProblem.JavadocInvalidConstructorQualification:
218	case IProblem.JavadocInvalidThrowsClassName:	219	case IProblem.JavadocInvalidThrowsClassName:
219	case IProblem.JavadocDuplicateThrowsClassName:	220	case IProblem.JavadocDuplicateThrowsClassName:
220	case IProblem.JavadocMissingThrowsClassName:	221	case IProblem.JavadocMissingThrowsClassName:
Lines 255-260 Link Here
255	case IProblem.JavadocUsingDeprecatedConstructor:	256	case IProblem.JavadocUsingDeprecatedConstructor:
256	case IProblem.JavadocUsingDeprecatedMethod:	257	case IProblem.JavadocUsingDeprecatedMethod:
257	case IProblem.JavadocUsingDeprecatedType:	258	case IProblem.JavadocUsingDeprecatedType:
		259	case IProblem.JavadocNotVisibleReference:
258	return CompilerOptions.InvalidJavadoc;	260	return CompilerOptions.InvalidJavadoc;
259		261
260	case IProblem.JavadocMissingParamTag:	262	case IProblem.JavadocMissingParamTag:
Lines 988-993 Link Here
988	case IProblem.JavadocDuplicateParamName:	990	case IProblem.JavadocDuplicateParamName:
989	case IProblem.JavadocMissingParamName:	991	case IProblem.JavadocMissingParamName:
990	case IProblem.JavadocMissingIdentifier:	992	case IProblem.JavadocMissingIdentifier:
		993	case IProblem.JavadocInvalidConstructorQualification:
991	case IProblem.JavadocInvalidThrowsClassName:	994	case IProblem.JavadocInvalidThrowsClassName:
992	case IProblem.JavadocDuplicateThrowsClassName:	995	case IProblem.JavadocDuplicateThrowsClassName:
993	case IProblem.JavadocMissingThrowsClassName:	996	case IProblem.JavadocMissingThrowsClassName:
Lines 1042-1047 Link Here
1042	case IProblem.JavadocNotVisibleConstructor:	1045	case IProblem.JavadocNotVisibleConstructor:
1043	case IProblem.JavadocNotVisibleMethod:	1046	case IProblem.JavadocNotVisibleMethod:
1044	case IProblem.JavadocNotVisibleType:	1047	case IProblem.JavadocNotVisibleType:
		1048	case IProblem.JavadocNotVisibleReference:
1045	if (!(this.options.reportInvalidJavadocTags && this.options.reportInvalidJavadocTagsNotVisibleRef)) {	1049	if (!(this.options.reportInvalidJavadocTags && this.options.reportInvalidJavadocTagsNotVisibleRef)) {
1046	return ProblemSeverities.Ignore;	1050	return ProblemSeverities.Ignore;
1047	}	1051	}
Lines 3578-3590 Link Here
3578	int severity = computeSeverity(IProblem.JavadocUsingDeprecatedType);	3582	int severity = computeSeverity(IProblem.JavadocUsingDeprecatedType);
3579	if (severity == ProblemSeverities.Ignore) return;	3583	if (severity == ProblemSeverities.Ignore) return;
3580	if (javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, modifiers)) {	3584	if (javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, modifiers)) {
3581	this.handle(	3585	if (type.isMemberType() && type instanceof ReferenceBinding && !javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, ((ReferenceBinding)type).modifiers)) {
3582	IProblem.JavadocUsingDeprecatedType,	3586	this.handle(IProblem.JavadocNotVisibleReference, NoArgument, NoArgument, location.sourceStart, location.sourceEnd);
3583	new String[] {new String(type.readableName())},	3587	} else {
3584	new String[] {new String(type.shortReadableName())},	3588	this.handle(
3585	severity,	3589	IProblem.JavadocUsingDeprecatedType,
3586	location.sourceStart,	3590	new String[] {new String(type.readableName())},
3587	location.sourceEnd);	3591	new String[] {new String(type.shortReadableName())},
		3592	severity,
		3593	location.sourceStart,
		3594	location.sourceEnd);
		3595	}
3588	}	3596	}
3589	}	3597	}
3590	public void javadocDuplicatedParamTag(char[] token, int sourceStart, int sourceEnd, int modifiers) {	3598	public void javadocDuplicatedParamTag(char[] token, int sourceStart, int sourceEnd, int modifiers) {
Lines 3807-3812 Link Here
3807	statement.sourceStart,	3815	statement.sourceStart,
3808	statement.sourceEnd);	3816	statement.sourceEnd);
3809	}	3817	}
		3818	public void javadocInvalidConstructorQualification(int sourceStart, int sourceEnd){
		3819	this.handle(IProblem.JavadocInvalidConstructorQualification, NoArgument, NoArgument, sourceStart, sourceEnd);
		3820	}
3810	/*	3821	/*
3811	* Similar implementation than invalidField(FieldReference...)	3822	* Similar implementation than invalidField(FieldReference...)
3812	* Note that following problem id cannot occur for Javadoc:	3823	* Note that following problem id cannot occur for Javadoc:
Lines 4208-4213 Link Here
4208	typeRef.sourceEnd);	4219	typeRef.sourceEnd);
4209	}	4220	}
4210	}	4221	}
		4222	public void javadocNotVisibleReference(int sourceStart, int sourceEnd, int modifiers) {
		4223	if (javadocVisibility(this.options.reportInvalidJavadocTagsVisibility, modifiers))
		4224	this.handle(IProblem.JavadocNotVisibleReference, NoArgument, NoArgument, sourceStart, sourceEnd);
		4225	}
4211	public void javadocUndeclaredParamTagName(char[] token, int sourceStart, int sourceEnd, int modifiers) {	4226	public void javadocUndeclaredParamTagName(char[] token, int sourceStart, int sourceEnd, int modifiers) {
4212	int severity = computeSeverity(IProblem.JavadocInvalidParamName);	4227	int severity = computeSeverity(IProblem.JavadocInvalidParamName);
4213	if (severity == ProblemSeverities.Ignore) return;	4228	if (severity == ProblemSeverities.Ignore) return;

Lines 180-215 Link Here

(-)compiler/org/eclipse/jdt/internal/compiler/parser/JavadocParser.java (-21 / +47 lines)
180	TypeReference typeRef = (TypeReference) receiver;	180	TypeReference typeRef = (TypeReference) receiver;
181	// Decide whether we have a constructor or not	181	// Decide whether we have a constructor or not
182	boolean isConstructor = false;	182	boolean isConstructor = false;
		183	int length = this.identifierLengthStack[0]; // may be > 0 for inner class constructor reference
183	if (typeRef == null) {	184	if (typeRef == null) {
184	char[] name = this.sourceParser.compilationUnit.getMainTypeName();	185	char[] name = this.sourceParser.compilationUnit.getMainTypeName();
185	TypeDeclaration typeDecl = getParsedTypeDeclaration();	186	TypeDeclaration typeDecl = getParsedTypeDeclaration();
186	if (typeDecl != null) {	187	if (typeDecl != null) {
187	name = typeDecl.name;	188	name = typeDecl.name;
188	}	189	}
189	isConstructor = CharOperation.equals(this.identifierStack[0], name);	190	isConstructor = CharOperation.equals(this.identifierStack[length-1], name);
190	typeRef = new JavadocImplicitTypeReference(name, this.memberStart);	191	typeRef = new JavadocImplicitTypeReference(name, this.memberStart);
191	} else {	192	} else {
192	char[] name = null;
193	if (typeRef instanceof JavadocSingleTypeReference) {	193	if (typeRef instanceof JavadocSingleTypeReference) {
194	name = ((JavadocSingleTypeReference)typeRef).token;	194	char[] name = ((JavadocSingleTypeReference)typeRef).token;
		195	isConstructor = CharOperation.equals(this.identifierStack[length-1], name);
195	} else if (typeRef instanceof JavadocQualifiedTypeReference) {	196	} else if (typeRef instanceof JavadocQualifiedTypeReference) {
196	char[][] tokens = ((JavadocQualifiedTypeReference)typeRef).tokens;	197	char[][] tokens = ((JavadocQualifiedTypeReference)typeRef).tokens;
197	name = tokens[tokens.length-1];	198	int last = tokens.length-1;
		199	isConstructor = CharOperation.equals(this.identifierStack[length-1], tokens[last]);
		200	if (isConstructor) {
		201	boolean valid = true;
		202	if (valid) {
		203	for (int i=0; i<length-1 && valid; i++) {
		204	valid = CharOperation.equals(this.identifierStack[i], tokens[i]);
		205	}
		206	}
		207	if (!valid) {
		208	if (this.reportProblems) {
		209	this.sourceParser.problemReporter().javadocInvalidConstructorQualification(this.memberStart+1, this.scanner.getCurrentTokenEndPosition());
		210	}
		211	return null;
		212	}
		213	}
198	} else {	214	} else {
199	throw new InvalidInputException();	215	throw new InvalidInputException();
200	}	216	}
201	isConstructor = CharOperation.equals(this.identifierStack[0], name);
202	}	217	}
203	// Create node	218	// Create node
204	if (arguments == null) {	219	if (arguments == null) {
205	if (isConstructor) {	220	if (isConstructor) {
206	JavadocAllocationExpression alloc = new JavadocAllocationExpression(this.identifierPositionStack[0]);	221	JavadocAllocationExpression allocation = new JavadocAllocationExpression(this.identifierPositionStack[length-1]);
207	alloc.type = typeRef;	222	allocation.type = typeRef;
208	alloc.tagValue = this.tagValue;	223	allocation.tagValue = this.tagValue;
209	alloc.sourceEnd = this.scanner.getCurrentTokenEndPosition();	224	allocation.sourceEnd = this.scanner.getCurrentTokenEndPosition();
210	return alloc;	225	if (length == 1) {
		226	allocation.qualification = new char[][] { this.identifierStack[0] };
		227	} else {
		228	System.arraycopy(this.identifierStack, 0, allocation.qualification = new char[length][], 0, length);
		229	}
		230	allocation.memberStart = this.memberStart;
		231	return allocation;
211	} else {	232	} else {
212	JavadocMessageSend msg = new JavadocMessageSend(this.identifierStack[0], this.identifierPositionStack[0]);	233	JavadocMessageSend msg = new JavadocMessageSend(this.identifierStack[length-1], this.identifierPositionStack[length-1]);
213	msg.receiver = typeRef;	234	msg.receiver = typeRef;
214	msg.tagValue = this.tagValue;	235	msg.tagValue = this.tagValue;
215	msg.sourceEnd = this.scanner.getCurrentTokenEndPosition();	236	msg.sourceEnd = this.scanner.getCurrentTokenEndPosition();
Lines 219-232 Link Here
219	JavadocArgumentExpression[] expressions = new JavadocArgumentExpression[arguments.size()];	240	JavadocArgumentExpression[] expressions = new JavadocArgumentExpression[arguments.size()];
220	arguments.toArray(expressions);	241	arguments.toArray(expressions);
221	if (isConstructor) {	242	if (isConstructor) {
222	JavadocAllocationExpression alloc = new JavadocAllocationExpression(this.identifierPositionStack[0]);	243	JavadocAllocationExpression allocation = new JavadocAllocationExpression(this.identifierPositionStack[length-1]);
223	alloc.arguments = expressions;	244	allocation.arguments = expressions;
224	alloc.type = typeRef;	245	allocation.type = typeRef;
225	alloc.tagValue = this.tagValue;	246	allocation.tagValue = this.tagValue;
226	alloc.sourceEnd = this.scanner.getCurrentTokenEndPosition();	247	allocation.sourceEnd = this.scanner.getCurrentTokenEndPosition();
227	return alloc;	248	if (length == 1) {
		249	allocation.qualification = new char[][] { this.identifierStack[0] };
		250	} else {
		251	System.arraycopy(this.identifierStack, 0, allocation.qualification = new char[length][], 0, length);
		252	}
		253	allocation.memberStart = this.memberStart;
		254	return allocation;
228	} else {	255	} else {
229	JavadocMessageSend msg = new JavadocMessageSend(this.identifierStack[0], this.identifierPositionStack[0], expressions);	256	JavadocMessageSend msg = new JavadocMessageSend(this.identifierStack[length-1], this.identifierPositionStack[length-1], expressions);
230	msg.receiver = typeRef;	257	msg.receiver = typeRef;
231	msg.tagValue = this.tagValue;	258	msg.tagValue = this.tagValue;
232	msg.sourceEnd = this.scanner.getCurrentTokenEndPosition();	259	msg.sourceEnd = this.scanner.getCurrentTokenEndPosition();
Lines 258-264 Link Here
258	*/	285	*/
259	protected Object createTypeReference(int primitiveToken) {	286	protected Object createTypeReference(int primitiveToken) {
260	TypeReference typeRef = null;	287	TypeReference typeRef = null;
261	int size = this.identifierLengthStack[this.identifierLengthPtr--];	288	int size = this.identifierLengthStack[this.identifierLengthPtr];
262	if (size == 1) { // Single Type ref	289	if (size == 1) { // Single Type ref
263	typeRef = new JavadocSingleTypeReference(	290	typeRef = new JavadocSingleTypeReference(
264	this.identifierStack[this.identifierPtr],	291	this.identifierStack[this.identifierPtr],
Lines 272-278 Link Here
272	System.arraycopy(this.identifierPositionStack, this.identifierPtr - size + 1, positions, 0, size);	299	System.arraycopy(this.identifierPositionStack, this.identifierPtr - size + 1, positions, 0, size);
273	typeRef = new JavadocQualifiedTypeReference(tokens, positions, this.tagSourceStart, this.tagSourceEnd);	300	typeRef = new JavadocQualifiedTypeReference(tokens, positions, this.tagSourceStart, this.tagSourceEnd);
274	}	301	}
275	this.identifierPtr -= size;
276	return typeRef;	302	return typeRef;
277	}	303	}
278		304
Lines 691-697 Link Here
691	* Fill associated comment fields with ast nodes information stored in stack.	717	* Fill associated comment fields with ast nodes information stored in stack.
692	*/	718	*/
693	protected void updateDocComment() {	719	protected void updateDocComment() {
694		720
695	// Set inherited positions	721	// Set inherited positions
696	this.docComment.inheritedPositions = this.inheritedPositions;	722	this.docComment.inheritedPositions = this.inheritedPositions;
697		723

Lines 86-92 Link Here

(-)dom/org/eclipse/jdt/core/dom/DocCommentParser.java (-17 / +14 lines)
86	buffer.append(super.toString());	86	buffer.append(super.toString());
87	return buffer.toString();	87	return buffer.toString();
88	}	88	}
89		89
90	/* (non-Javadoc)	90	/* (non-Javadoc)
91	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#createArgumentReference(char[], java.lang.Object, int)	91	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#createArgumentReference(char[], java.lang.Object, int)
92	*/	92	*/
Lines 167-179 Link Here
167	// Create method ref	167	// Create method ref
168	MethodRef methodRef = this.ast.newMethodRef();	168	MethodRef methodRef = this.ast.newMethodRef();
169	SimpleName methodName = new SimpleName(this.ast);	169	SimpleName methodName = new SimpleName(this.ast);
170	methodName.internalSetIdentifier(new String(this.identifierStack[0]));	170	int memberPtr = this.identifierLengthStack[0] - 1;
		171	methodName.internalSetIdentifier(new String(this.identifierStack[memberPtr]));
171	methodRef.setName(methodName);	172	methodRef.setName(methodName);
172	int start = (int) (this.identifierPositionStack[0] >>> 32);	173	int start = (int) (this.identifierPositionStack[memberPtr] >>> 32);
173	int end = (int) this.identifierPositionStack[0];	174	int end = (int) this.identifierPositionStack[memberPtr];
174	methodName.setSourceRange(start, end - start + 1);	175	methodName.setSourceRange(start, end - start + 1);
175	// Set qualifier	176	// Set qualifier
176	// int end = methodName.getStartPosition()+methodName.getLength()-1;
177	if (receiver == null) {	177	if (receiver == null) {
178	start = this.memberStart;	178	start = this.memberStart;
179	methodRef.setSourceRange(start, end - start + 1);	179	methodRef.setSourceRange(start, end - start + 1);
Lines 188-197 Link Here
188	while (parameters.hasNext()) {	188	while (parameters.hasNext()) {
189	MethodRefParameter param = (MethodRefParameter) parameters.next();	189	MethodRefParameter param = (MethodRefParameter) parameters.next();
190	methodRef.parameters().add(param);	190	methodRef.parameters().add(param);
191	// end = param.getStartPosition()+param.getLength()-1;
192	}	191	}
193	}	192	}
194	// methodRef.setSourceRange(start, end-start+1);
195	methodRef.setSourceRange(start, this.scanner.getCurrentTokenEndPosition()-start+1);	193	methodRef.setSourceRange(start, this.scanner.getCurrentTokenEndPosition()-start+1);
196	return methodRef;	194	return methodRef;
197	}	195	}
Lines 239-245 Link Here
239	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#createTypeReference()	237	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#createTypeReference()
240	*/	238	*/
241	protected Object createTypeReference(int primitiveToken) {	239	protected Object createTypeReference(int primitiveToken) {
242	int size = this.identifierLengthStack[this.identifierLengthPtr--];	240	int size = this.identifierLengthStack[this.identifierLengthPtr];
243	String[] identifiers = new String[size];	241	String[] identifiers = new String[size];
244	int pos = this.identifierPtr - size + 1;	242	int pos = this.identifierPtr - size + 1;
245	for (int i = 0; i < size; i++) {	243	for (int i = 0; i < size; i++) {
Lines 307-324 Link Here
307	int end = (int) this.identifierPositionStack[pos];	305	int end = (int) this.identifierPositionStack[pos];
308	typeRef.setSourceRange(start, end-start+1);	306	typeRef.setSourceRange(start, end-start+1);
309	}	307	}
310	this.identifierPtr -= size;
311	return typeRef;	308	return typeRef;
312	}	309	}
313		310
314	/*
315	* Parse @return tag declaration
316	*/
317	protected boolean parseReturn() {
318	createTag();
319	return true;
320	}
321
322	/* (non-Javadoc)	311	/* (non-Javadoc)
323	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#parseIdentifierTag(boolean)	312	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#parseIdentifierTag(boolean)
324	*/	313	*/
Lines 332-337 Link Here
332	return false;	321	return false;
333	}	322	}
334		323
		324	/*
		325	* Parse @return tag declaration
		326	*/
		327	protected boolean parseReturn() {
		328	createTag();
		329	return true;
		330	}
		331
335	/* (non-Javadoc)	332	/* (non-Javadoc)
336	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#parseTag(int)	333	* @see org.eclipse.jdt.internal.compiler.parser.AbstractCommentParser#parseTag(int)
337	*/	334	*/