Description Andrew Clement 2012-04-03 21:44:00 EDT

From the mailing list:
Hi Andy,

Thank you so much!

Unfortunately we have a little ltw vs. source weaving problem. Consider I have the following setup:

Source Weaving:
##############

package ch.annotation;
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;

@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.FIELD)
public @interface Anno {

}

package ch.aspects;

import ch.annotation.Anno;

public aspect TriggerAll {
	declare @field : * *.myInt : @Anno;
	
	before(Anno anno) : @annotation(anno) && set(* *.myInt) {
	       System.out.println("Triggered");
	     }
}

package ch.tests;

public class MyObject {
	public int myInt;

	public int getMyInt() {
		return myInt;
	}

	public void setMyInt(int myInt) {
		this.myInt = myInt;
	}
}

package ch.tests;

public class Main {

	public static void main(String[] args) {
		MyObject mo = new MyObject();
		mo.myInt = 13;
		mo.setMyInt(13);
	}
}

This outputs:
Triggered
Triggered

// two times "Triggered". As expected.

Now LTW:
########

Infos: 
- -javaagent:lib/aspectjweaver.jar is set
- the compiled aspect "TriggerAll.aj" is in the class searchpath (classpath) packed within a jar file.

public class MyObject2 {
	public int myInt;

	public int getMyInt() {
		return myInt;
	}

	public void setMyInt(int myInt) {
		this.myInt = myInt;
	}
}

public class Main {

	public static void main(String[] args) {
		MyObject2 mo = new MyObject2();
		mo.myInt = 13;
		mo.setMyInt(13);
	}
}

This outputs:
Triggered
// just one "Triggered" message. And this comes from mo.setMyInt(13). 

So mo.myInt = 13 does not trigger the pointcut!

Note: It has nothing to do with your recent changes. This happens on AspectJ 1.6.12.

Output form the weaving "process":

[AppClassLoader@5acac268] info AspectJ Weaver Version 1.6.12 built on Tuesday Oct 18, 2011 at 17:52:06 GMT
[AppClassLoader@5acac268] info register classloader sun.misc.Launcher$AppClassLoader@5acac268
[AppClassLoader@5acac268] info using configuration /path/META-INF/aop.xml
[AppClassLoader@5acac268] info using configuration /other-path/lib/TriggerAll.jar!/META-INF/aop-ajc.xml
[AppClassLoader@5acac268] info register aspect ch.aspects.TriggerAll
[AppClassLoader@5acac268] info weaver operating in reweavable mode.  Need to verify any required types exist.
[AppClassLoader@5acac268] debug weaving 'Main'
[AppClassLoader@5acac268] debug weaving 'MyObject2'
[AppClassLoader@5acac268] weaveinfo 'public int myInt' of type 'MyObject2' (MyObject2.java) is annotated with @Anno field annotation from 'ch.aspects.TriggerAll' (TriggerAll.aj:6)
[AppClassLoader@5acac268] weaveinfo Join point 'field-set(int MyObject2.myInt)' in Type 'MyObject2' (MyObject2.java:11) advised by before advice from 'ch.aspects.TriggerAll' (TriggerAll.aj:8)
[AppClassLoader@5acac268] debug weaving 'ch.aspects.TriggerAll'
[AppClassLoader@5acac268] info processing reweavable type ch.aspects.TriggerAll: ch\aspects\TriggerAll.aj
[AppClassLoader@5acac268] debug cannot weave 'org.aspectj.lang.NoAspectBoundException'
[AppClassLoader@5acac268] debug weaving 'ch.annotation.Anno'
[AppClassLoader@5acac268] debug weaving '$Proxy3'
Triggered

But again: Thanks very much for you effort