Bug 83645 - pertypewithin({interface}) illegal field modifier
Summary: pertypewithin({interface}) illegal field modifier
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: 1.5.0 M2   Edit
Assignee: Adrian Colyer CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-25 14:37 EST by Wes Isberg CLA
Modified: 2005-03-24 11:36 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wes Isberg CLA 2005-01-25 14:37:13 EST
AJDT 1.2.0.20050124144759, Java 1.5.0-b64 on XP SP2

runtime error "Illegal field modifiers in class lang15/IP: 0x8A"
when using pertype() with an interface type.

(I'll add to tests if I have time.)

------------lang15/PerType.java
package lang15;

interface IP {}
/**
 * PerType
 */
public class PerType implements IP {
	public static void main(String[] args) {
		System.out.println("you got " + PT.aspectOf(PerType.class));
		System.out.println("you got " + PT.aspectOf(IP.class));
		System.out.println("you got " + PT.aspectOf(C.class));
	}
	static class C extends PerType {}
}
aspect PT pertypewithin(IP+) {
	static int INDEX;
	final int index = INDEX++;
	public PT() {
		System.out.println("creating " + this);
	}
	public String toString() {
		return "me " + index;
	}
}
Comment 1 Wes Isberg CLA 2005-01-25 16:15:15 EST
See tests/bugs150/PR83645.java and tests/ajcTestsFailing.xml.  Not tied into
tests/src/../systemtest/...  

Test assumes, perhaps incorrectly, that a NoAspectBoundException is the correct
result.
Comment 2 Adrian Colyer CLA 2005-03-23 09:20:57 EST
we should look at this before closing out m2...
Comment 3 Adrian Colyer CLA 2005-03-24 05:27:34 EST
test passing with latest builds, presume fixed by Andy when sorting out other
ptw related bugs. I tried a few variants bug couldn't get it to fail. Will close
once published build is available.
Comment 4 Andrew Clement CLA 2005-03-24 11:36:39 EST
Build available, see download page:

aspectj-DEVELOPMENT-20050324155000.jar