Bug 159143 - subtype pattern not working for declare annotation on method
Summary: subtype pattern not working for declare annotation on method
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: DEVELOPMENT   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-09-28 12:47 EDT by Wes Isberg CLA
Modified: 2007-10-11 06:11 EDT (History)
1 user (show)

See Also:

Attachments
test case, includes test specification (3.13 KB, text/plain)
2006-09-28 12:50 EDT, Wes Isberg CLA 		no flags Details
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 Wes Isberg CLA 2006-09-28 12:47:33 EDT 
Using AspectJ 1.5.3.200609271036, when declaring an annotation on a method, get unexpected error when using subtype wildcard for the declaring type.  E.g.,

  declare @method : void Foo+.foo() : @MethodAnnotation;

Error text: "The method 'void Foo+.foo()' does not exist"
Comment 1 Wes Isberg CLA 2006-09-28 12:50:28 EDT 
Created attachment 51100 [details]
test case, includes test specification
Comment 2 Andrew Clement CLA 2006-10-03 11:12:02 EDT 
Hmmmm - that test program works fine for me.  So I tried creating a minimal test program:

@interface MethodAnnotation {}

aspect X {
  declare @method : void Foo+.foo() : @MethodAnnotation;
}

interface Foo {}

class Blah implements Foo {
  public void foo() {}
}

If I comment out 'foo()' then I get the same error Wes describes.  If I have that method in, then it works (and I can see the annotation in the class using javap).  Are you doing anything else that I'm not Wes?  I am on the command line, not in AJDT - and I'm using HEAD rather than 20060927.
Comment 3 Wes Isberg CLA 2006-10-03 15:52:46 EDT 
I get the same results as Andy in head with both test and comment program.  Fixed or ?
Comment 4 Andrew Clement CLA 2007-10-11 06:10:41 EDT 
assumed fixed - no progress for a year.
Comment 5 Andrew Clement CLA 2007-10-11 06:11:50 EDT 
see previous comment, silly bugzilla