From 390b387fd0f3695820a6fab69110ae46496f8473 Mon Sep 17 00:00:00 2001 From: aclement Date: Wed, 25 Feb 2009 22:52:23 +0000 Subject: [PATCH] 266220: generic aspect subtypes in aop.xml --- .../loadtime/ConcreteAspectCodeGen.java | 71 ++++++++++++++----- 1 file changed, 52 insertions(+), 19 deletions(-) diff --git a/loadtime/src/org/aspectj/weaver/loadtime/ConcreteAspectCodeGen.java b/loadtime/src/org/aspectj/weaver/loadtime/ConcreteAspectCodeGen.java index ac8730747..e58ecae6f 100644 --- a/loadtime/src/org/aspectj/weaver/loadtime/ConcreteAspectCodeGen.java +++ b/loadtime/src/org/aspectj/weaver/loadtime/ConcreteAspectCodeGen.java @@ -51,9 +51,10 @@ import org.aspectj.weaver.patterns.PerSingleton; /** * Generates bytecode for concrete-aspect. *

- * The concrete aspect is @AspectJ code generated. As it is build during aop.xml definitions registration we perform the type - * munging for perclause, ie. aspectOf() artifact directly, instead of waiting for it to go thru the weaver (that we are in the - * middle of configuring). + * The concrete aspect is @AspectJ code generated. As it is build during aop.xml + * definitions registration we perform the type munging for perclause, ie. + * aspectOf() artifact directly, instead of waiting for it to go thru the weaver + * (that we are in the middle of configuring). * * @author Alexandre Vasseur * @author Andy Clement @@ -133,7 +134,27 @@ public class ConcreteAspectCodeGen { } } - parent = world.resolve(concreteAspect.extend, true); + String parentAspectName = concreteAspect.extend; + if (parentAspectName.indexOf("<") != -1) { + // yikes, generic parent + parent = world.resolve(UnresolvedType.forName(parentAspectName), true); + if (parent.isMissing()) { + reportError("Unable to resolve type reference: " + stringify()); + return false; + } + if (parent.isParameterizedType()) { + UnresolvedType[] typeParameters = parent.getTypeParameters(); + for (int i = 0; i < typeParameters.length; i++) { + UnresolvedType typeParameter = typeParameters[i]; + if (typeParameter instanceof ResolvedType && ((ResolvedType) typeParameter).isMissing()) { + reportError("Unablet to resolve type parameter '" + typeParameter.getName() + "' from " + stringify()); + return false; + } + } + } + } else { + parent = world.resolve(concreteAspect.extend, true); + } // handle inner classes if (parent.isMissing()) { // fallback on inner class lookup mechanism @@ -176,7 +197,8 @@ public class ConcreteAspectCodeGen { ResolvedMember method = (ResolvedMember) iter.next(); if ("()V".equals(method.getSignature())) { String n = method.getName(); - // Allow for the abstract pointcut being from a code style aspect compiled with -1.5 (see test for 128744) + // Allow for the abstract pointcut being from a code style + // aspect compiled with -1.5 (see test for 128744) if (n.startsWith("ajc$pointcut")) { n = n.substring(14); n = n.substring(0, n.indexOf("$")); @@ -184,20 +206,23 @@ public class ConcreteAspectCodeGen { } else if (hasPointcutAnnotation(method)) { elligibleAbstractions.add(method.getName()); } else { - // error, an outstanding abstract method that can't be concretized in XML + // error, an outstanding abstract method that can't be + // concretized in XML reportError("Abstract method '" + method.toString() + "' cannot be concretized in XML: " + stringify()); return false; } } else { if (method.getName().startsWith("ajc$pointcut") || hasPointcutAnnotation(method)) { - // it may be a pointcut but it doesn't meet the requirements for XML concretization + // it may be a pointcut but it doesn't meet the requirements + // for XML concretization reportError("Abstract method '" + method.toString() + "' cannot be concretized as a pointcut (illegal signature, must have no arguments, must return void): " + stringify()); return false; } else { - // error, an outstanding abstract method that can't be concretized in XML + // error, an outstanding abstract method that can't be + // concretized in XML reportError("Abstract method '" + method.toString() + "' cannot be concretized in XML: " + stringify()); return false; } @@ -268,7 +293,8 @@ public class ConcreteAspectCodeGen { } /** - * Rebuild the XML snip that defines this concrete aspect, for log error purpose + * Rebuild the XML snip that defines this concrete aspect, for log error + * purpose * * @return string repr. */ @@ -339,14 +365,22 @@ public class ConcreteAspectCodeGen { // [@Pointcut(xxxExpression-n) // public void xxxName-n() {}] // } - + String parentName = "java/lang/Object"; + if (parent != null) { + if (parent.isParameterizedType()) { + parentName = parent.getGenericType().getName().replace('.', '/'); + } else { + parentName = parent.getName().replace('.', '/'); + } + } // @Aspect public class ... - LazyClassGen cg = new LazyClassGen(concreteAspect.name.replace('.', '/'), (parent == null) ? "java/lang/Object" : parent - .getName().replace('.', '/'), null,// TODO AV - we could point - // to the aop.xml that - // defines it and use - // JSR-45 - Modifier.PUBLIC + Constants.ACC_SUPER, EMPTY_STRINGS, world); + // TODO AV - we could point to the aop.xml that defines it and use + // JSR-45 + LazyClassGen cg = new LazyClassGen(concreteAspect.name.replace('.', '/'), parentName, null, Modifier.PUBLIC + + Constants.ACC_SUPER, EMPTY_STRINGS, world); + if (parent != null && parent.isParameterizedType()) { + cg.setSuperClass(parent); + } if (perclauseString == null) { AnnotationGen ag = new AnnotationGen(new ObjectType("org/aspectj/lang/annotation/Aspect"), Collections.EMPTY_LIST, true, cg.getConstantPool()); @@ -374,8 +408,8 @@ public class ConcreteAspectCodeGen { LazyMethodGen init = new LazyMethodGen(Modifier.PUBLIC, Type.VOID, "", EMPTY_TYPES, EMPTY_STRINGS, cg); InstructionList cbody = init.getBody(); cbody.append(InstructionConstants.ALOAD_0); - cbody.append(cg.getFactory().createInvoke((parent == null) ? "java/lang/Object" : parent.getName().replace('.', '/'), - "", Type.VOID, EMPTY_TYPES, Constants.INVOKESPECIAL)); + + cbody.append(cg.getFactory().createInvoke(parentName, "", Type.VOID, EMPTY_TYPES, Constants.INVOKESPECIAL)); cbody.append(InstructionConstants.RETURN); cg.addMethodGen(init); @@ -394,7 +428,6 @@ public class ConcreteAspectCodeGen { InstructionList body = mg.getBody(); body.append(InstructionConstants.RETURN); - cg.addMethodGen(mg); } -- 2.39.5