Bug 106728 - [type hierarchy] Incorrect 'overriding method' in type hierarchy
Summary: [type hierarchy] Incorrect 'overriding method' in type hierarchy
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1.1   Edit
Assignee: Martin Aeschlimann CLA Friend
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-11 05:35 EDT by Christof Marti CLA Friend
Modified: 2005-09-26 05:25 EDT (History)
1 user (show)

See Also:


Attachments
proposed patch (1.58 KB, patch)
2005-09-20 05:07 EDT, Martin Aeschlimann CLA Friend
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christof Marti CLA Friend 2005-08-11 05:35:51 EDT
3.1, 1.4 compiler compliance

With the following two CUs the Type Hierarchy shows the incorrect overriding
method when enabling 'Lock View and Show Members' and the Quick Type Hierarchy
will navigate to the incorrect overriding method for Object bar(String, String).

interface IFoo {
	Object bar(String s2, String s);
	String bar(String s2, Object o);
}

class Foo implements IFoo {
	public Object bar(String s2, String s) {
		return null;
	}
	public String bar(String s2, Object o) {
		return null;
	}
}
Comment 1 Dirk Baeumer CLA Friend 2005-08-11 06:35:40 EDT
Martin, can you please have a look. This is still J2SE 1.4.
Comment 2 Martin Aeschlimann CLA Friend 2005-09-16 12:12:20 EDT
fixed in 3.2, but should be fixed for 3.1.1
Comment 3 Philipe Mulet CLA Friend 2005-09-20 02:59:06 EDT
+1 for 3.1.1
Comment 4 Martin Aeschlimann CLA Friend 2005-09-20 05:07:29 EDT
Created attachment 27281 [details]
proposed patch
Comment 5 Martin Aeschlimann CLA Friend 2005-09-20 05:09:02 EDT
Dani, please review the patch. The bug is embarassing, so I would like to have
this in 3.1.1.
Comment 6 Dani Megert CLA Friend 2005-09-21 09:49:23 EDT
+1 to fix it.
I've reviewed the patch and tested it in R3_1_maintenance.
Comment 7 Martin Aeschlimann CLA Friend 2005-09-21 12:05:53 EDT
fix released in 3.1.1 stream
Comment 8 Benno Baumgartner CLA Friend 2005-09-26 05:13:40 EDT
Verified using M20050923-1430