View | Details | Raw Unified | Return to bug 141931 | Differences between
and this patch

Collapse All | Expand All

(-)src/org/eclipse/jdt/core/tests/compiler/regression/MethodVerifyTest.java (-1 / +2 lines)
Lines 348-354 Link Here
348
			null
348
			null
349
		);
349
		);
350
	}
350
	}
351
351
	//https://bugs.eclipse.org/bugs/show_bug.cgi?id=141931 may change this
352
	// if we decide to mandate the @Override upon K<T>#foo
352
	public void test004() { // all together
353
	public void test004() { // all together
353
		this.runNegativeTest(
354
		this.runNegativeTest(
354
			new String[] {
355
			new String[] {
(-)src/org/eclipse/jdt/core/tests/compiler/regression/AnnotationTest.java (+28 lines)
Lines 2436-2441 Link Here
2436
			"----------\n");
2436
			"----------\n");
2437
	}	
2437
	}	
2438
	// check @Override annotation - strictly for superclasses (overrides) and not interfaces (implements)
2438
	// check @Override annotation - strictly for superclasses (overrides) and not interfaces (implements)
2439
	//https://bugs.eclipse.org/bugs/show_bug.cgi?id=141931 will change this by
2440
	// suppressing the last message (note also that the message label should change).
2439
	public void test077() {
2441
	public void test077() {
2440
		this.runNegativeTest(
2442
		this.runNegativeTest(
2441
			new String[] {
2443
			new String[] {
Lines 3971-3976 Link Here
3971
			"");
3973
			"");
3972
    }		
3974
    }		
3973
    //https://bugs.eclipse.org/bugs/show_bug.cgi?id=90484 - check no missing @Override warning
3975
    //https://bugs.eclipse.org/bugs/show_bug.cgi?id=90484 - check no missing @Override warning
3976
    //https://bugs.eclipse.org/bugs/show_bug.cgi?id=141931 will change this
3977
    // since X#clone() is override-compatible with Object#clone.
3974
    public void test126() {
3978
    public void test126() {
3975
        this.runNegativeTest(
3979
        this.runNegativeTest(
3976
            new String[] {
3980
            new String[] {
Lines 5605-5610 Link Here
5605
		"----------\n");
5609
		"----------\n");
5606
    }
5610
    }
5607
    //https://bugs.eclipse.org/bugs/show_bug.cgi?id=94759
5611
    //https://bugs.eclipse.org/bugs/show_bug.cgi?id=94759
5612
    //https://bugs.eclipse.org/bugs/show_bug.cgi?id=141931
5613
    // will probably change this (the new ref impl should
5614
    // add @Override capability to interfaces - so no error here)
5608
    public void test168() {
5615
    public void test168() {
5609
        this.runNegativeTest(
5616
        this.runNegativeTest(
5610
            new String[] {
5617
            new String[] {
Lines 6565-6568 Link Here
6565
		},
6572
		},
6566
		"");
6573
		"");
6567
}
6574
}
6575
//https://bugs.eclipse.org/bugs/show_bug.cgi?id=141931
6576
// still have to decide if we mandate the @Override when a class
6577
// implements an interface method; however, the reference impl
6578
// may (as of 20060516) consider tolerating it
6579
public void _test200() {
6580
	this.runConformTest(
6581
		new String[] {
6582
			"I.java",
6583
			"public interface I {\n" + 
6584
			"	void foo();\n" + 
6585
			"}\n",
6586
			"X.java",
6587
			"class X implements I {\n" + 
6588
			"  @Override\n" +
6589
			"  public void foo() {}\n" +
6590
			"}\n",
6591
		},
6592
		"");
6593
}
6594
6595
6568
}
6596
}
(-)src/org/eclipse/jdt/core/tests/compiler/regression/Compliance_1_5.java (+5 lines)
Lines 1683-1688 Link Here
1683
 * http://bugs.eclipse.org/bugs/show_bug.cgi?id=39467
1683
 * http://bugs.eclipse.org/bugs/show_bug.cgi?id=39467
1684
 * should diagnose missing abstract method implementation
1684
 * should diagnose missing abstract method implementation
1685
 */
1685
 */
1686
//https://bugs.eclipse.org/bugs/show_bug.cgi?id=141931 will
1687
// change this
1688
// we should be tolerant to T#foo should bearing an @Override 
1689
// annotation since it implements I#foo; however, mandating it
1690
// is probably too much...
1686
public void test046() {
1691
public void test046() {
1687
	this.runNegativeTest(
1692
	this.runNegativeTest(
1688
		new String[] {
1693
		new String[] {

Return to bug 141931