[jboss-jira] [JBoss JIRA] Commented: (JBAOP-731) problem with javac->aopc->javac sequence

[ https://jira.jboss.org/browse/JBAOP-731?page=com.atlassian.jira.plugin.sy... ] Andrew Dinn commented on JBAOP-731: ----------------------------------- I think you are right to be confused that javac complains that POJO2 is not abstract and does not override abstract method _getAdvisor() in org.jboss.aop.Advised when javap shows that POJO1 implements Advisable and indeed contains a non-abstract _getAdvisor() method. This seems to be very wrong. But it's not an AOP problem. This is the compiler failign to eat what looks liek legitimate bytecode. The reason javac is discounting this method is because it is tagged with the SYNTHETIC attribute -- all AOP -generated methods are so tagged. I checked the javac source code in OpenJDK and for some reason it explicitly discounts SYNTHETIC implementation methods when attempting to verify that a non-abstract class implements all inherited abstract methods (that includes interface methods which are implicitly taken to be abstract). Now I don't know of anything in the JVM spec which states that a SYNTHETIC implementation method is not a valid implementation of an abstract method. I believe this is a bug in javac. What it looks like it is doing (and doing wrongly) is attempting to discount so-called 'Miranda methods', i.e. pseudo-methods installed during compilation as markers for an unimplemented abstract method. These are tagged with attribute flags SYNTHETIC, ABSTRACT and IPROXY, a special flag used only by javac. I believe the check which discounts an implementation method should look for flag IPROXY, not flag SYNTHETIC. Thsi may be a hangover form earlier code as IPROXY appears to be a recent introduction. I will raise this as an issue with the OpenJDK team and see if we can agree that it is a bug and push a fix through into OpenJDK and, from there, the Sun JVM. Until this is fixed we cannot fix AOP. It is not possilble to remove the SYNTHETIC tag from AOP generated methods as this will cause other problems (see https://jira.jboss.org/browse/JBAOP-642 and https://jira.jboss.org/browse/JBAOP-782). -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira