Bug 87174 - Hover doesn't show fully resolved information [render]
Summary: Hover doesn't show fully resolved information [render]
Status: RESOLVED 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 RC1   Edit
Assignee: Martin Aeschlimann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 85811 87362 90186
Blocks:
  Show dependency tree
 
Reported: 2005-03-04 14:23 EST by Dirk Baeumer CLA
Modified: 2005-05-30 11:01 EDT (History)
2 users (show)

See Also:

Attachments
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 Dirk Baeumer CLA 2005-03-04 14:23:58 EST 
However over the call foo(as, 123). The Hover should show foo(List<String>,
Integer) not foo(List<E>, Integer)

public class MethodArguments {
	public <T> void foo(List<T> lt, Integer i) {
	}
	void use() {
		ArrayList<String> as= null;
		foo(as, 123);
	}
}

The java element has now resolved information for parameterized types via the
getKey and the BindingsKey class.
Comment 1 Dani Megert CLA 2005-03-08 04:33:28 EST 
JavaElementLabels.USE_RESOLVED is used to get the element label. This uses the
resolved information. I checked that isResolved() returns true for use(as, 123).

Related to bug 85811?
Comment 2 Martin Aeschlimann CLA 2005-03-08 04:59:51 EST 
yes, we're waiting for the fix in bug 85811, then this should work.

*** This bug has been marked as a duplicate of 85811 ***
Comment 3 Martin Aeschlimann CLA 2005-03-08 05:00:55 EST 
better mark as 'dependend on bug 85811' than duplicate
Comment 4 Martin Aeschlimann CLA 2005-03-08 07:37:27 EST 
also depends on bug 87362 to get the field types correct
Comment 5 Martin Aeschlimann CLA 2005-04-04 12:58:06 EDT 
Keys for methods are now there but still buggy (bug 90186).
Also waiting for bug 87362.
Comment 6 Martin Aeschlimann CLA 2005-05-24 05:03:14 EDT 
marking as fixed. Still missing the resolved field types, but this has been
deferred (bug 87362)
Comment 7 Tobias Widmer CLA 2005-05-30 10:17:54 EDT 
verifying...
Comment 8 Tobias Widmer CLA 2005-05-30 10:21:56 EDT 
I get <String> void Test.MethodArguments.foo(List<T> lt, Integer i)

Bug 87362 seems to be fixed and verified?
Comment 9 Martin Aeschlimann CLA 2005-05-30 11:01:27 EDT 
yes, Bug 87362 was fixed after all.

It seems that there are still bugs in jdt.core. I'm not sure but I think
<String> void Test.MethodArguments.foo(List<String> lt, Integer i) should be shown
I filed bug 97275 against jdt.core.