Bug 98473 - AST binding: type variable reports as 'isTopLevel'
Summary: AST binding: type variable reports as 'isTopLevel'
Status: CLOSED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1 RC2   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-06 03:48 EDT by Martin Aeschlimann CLA
Modified: 2005-06-10 10:17 EDT (History)
1 user (show)

See Also:

Attachments
Apply on HEAD (695 bytes, patch)
2005-06-06 14:02 EDT, Olivier Thomann CLA 		no flags Details | Diff
Apply on HEAD of org.eclipse.jdt.core.tests.model (1.80 KB, patch)
2005-06-06 14:02 EDT, Olivier Thomann 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 Martin Aeschlimann CLA 2005-06-06 03:48:24 EDT 
20050606

Example from bug 97442:
import java.util.List;

public class A {
	class Counter<T, L extends List<T>> {
		private L _attribute;
	}
}

Open the AST view on 'L' and look at the binding. The binding reports
'isTopLevel'. No other type variable does that and I think it should only apply
to classes, enums and interfaces/annotations
Comment 1 Olivier Thomann CLA 2005-06-06 14:02:10 EDT 
Created attachment 22437 [details]
Apply on HEAD

Possible fix
Comment 2 Olivier Thomann CLA 2005-06-06 14:02:47 EDT 
Created attachment 22438 [details]
Apply on HEAD of org.eclipse.jdt.core.tests.model

Corresponding regression test
Comment 3 Olivier Thomann CLA 2005-06-06 14:03:43 EDT 
Philippe, this is a low risk fix.
I don't know how important this is for Martin, but it could be released for RC2.
All our DOM tests passed with this fix.
Comment 4 Philipe Mulet CLA 2005-06-06 14:48:15 EDT 
+1 for RC2
Comment 5 Olivier Thomann CLA 2005-06-06 14:50:38 EDT 
Fixed and released in HEAD.
Regression test added in org.eclipse.jdt.core.tests.dom.ASTConverter15Test.test0188
Comment 6 Frederic Fusier CLA 2005-06-07 11:58:23 EDT 
Verified for 3.1 RC2 using build N20050607-0010 + JDT/Core HEAD
Comment 7 David Audel CLA 2005-06-10 10:17:41 EDT 
Verified for 3.1 RC2 using build I20050610-0010