Bug 260748 - Add type parameters to program elements in the model
Summary: Add type parameters to program elements in the model
Status: RESOLVED WONTFIX
Alias: None
Product: AspectJ
Classification: Tools
Component: Build (show other bugs)
Version: 1.6.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.6.4   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 257437
  Show dependency tree
 
Reported: 2009-01-12 12:16 EST by Andrew Eisenberg CLA
Modified: 2009-03-13 13:01 EDT (History)
1 user (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 Andrew Eisenberg CLA 2009-01-12 12:16:09 EST 
In order to properly perform reconciling and content assist in Eclipse on ITDs that contain type parameters we need to know what type parameters they have on the introduced methods.  Currently, this does not exist in the AJ structure model.
Comment 1 Andrew Clement CLA 2009-01-12 16:10:45 EST 
do you want just the letters? "T"
or the letters with full bounds? "T extends a.b.Something"
if the latter, were you planning to parse the strings to determine the types involved?

if we go too far down this route, you might as well access it through the world as it has already taken everything apart and analysed bounds.
Comment 2 Andrew Clement CLA 2009-01-20 13:28:16 EST 
do we still need this? or are we going to go ahead with the world supplying the necessary information to AJDT?
Comment 3 Andrew Eisenberg CLA 2009-01-20 14:04:12 EST 
I'm leaning towards using world for this, but I haven't had time to really look into this for a while.  Can we hold off on making this decision for a little while?
Comment 4 Andrew Eisenberg CLA 2009-03-13 12:52:52 EDT 
I am using the World object for this.   This functionality has been available since AJDT 1.6.3.  It seems to be working fine.  I do not think there is any more work needed for this bug.
Comment 5 Andrew Clement CLA 2009-03-13 13:01:41 EDT 
Using the World is a reasonable solution, closing for now.