Bug 204624 - [1.5][compiler] No error on abstract method implementation with missing parameter generics
Summary: [1.5][compiler] No error on abstract method implementation with missing param...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: 3.3.2   Edit
Assignee: Kent Johnson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-09-25 19:57 EDT by Nick Radov CLA
Modified: 2008-09-16 09:49 EDT (History)
3 users (show)

See Also:
philippe_mulet: pmc_approved+

Attachments
Test case (289 bytes, text/plain)
2007-09-25 19:57 EDT, Nick Radov CLA 		no flags Details
Proposed patch (3.22 KB, patch)
2007-09-28 15:47 EDT, Kent Johnson CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Radov CLA 2007-09-25 19:57:16 EDT 
Created attachment 79177 [details]
Test case

Build ID: I20070621-1340

Steps To Reproduce:
Try the code in the attachment. It compiles with no errors in Eclipse, although there are two warnings. But the Sun Java 1.5.0_12 command-line compiler fails on the same code with this error.

org\eclipse\bugs\AbstractTest.java:9: org.eclipse.bugs.TestImpl is not abstract and does not override abstract method <T>go(java.util.List<T>) in org.eclipse.bugs.AbstractTest
class TestImpl extends AbstractTest {
^

The Eclipse compiler should be consistent with the Sun compiler.

There are several other bugs which I think may be related: Bug 159851, Bug 166355, Bug 203905.
Comment 1 Kent Johnson CLA 2007-09-28 15:47:01 EDT 
Created attachment 79423 [details]
Proposed patch
Comment 2 Kent Johnson CLA 2007-10-01 10:53:59 EDT 
Released into HEAD for 3.4M3

Added MethodVerifyTest test148
Comment 3 Kent Johnson CLA 2007-10-01 14:02:17 EDT 
Philippe - need a +1 for 3.3.2
Comment 4 Philipe Mulet CLA 2007-10-08 05:51:25 EDT 
+1 for 3.3.2
Comment 5 Kent Johnson CLA 2007-10-09 10:11:28 EDT 
Released for 3.3.2
Comment 6 Frederic Fusier CLA 2007-10-29 06:26:46 EDT 
Verified for 3.4M3 using I20071029-0010 build.
Comment 7 Eric Jodet CLA 2008-01-24 06:17:11 EST 
Verified for 3.3.2 using M20080123-0800 build