[jboss-cvs] JBossAS SVN: r78953 - projects/aop/trunk/aop/src/main/org/jboss/aop/instrument.

[jboss-cvs-commits at lists.jboss.org]

Author: kabir.khan at jboss.com
Date: 2008-09-30 09:38:00 -0400 (Tue, 30 Sep 2008)
New Revision: 78953

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/Instrumentor.java
Log:
Handle possible exception if javassist does not follow assumption of hack

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/Instrumentor.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/Instrumentor.java	2008-09-30 13:12:39 UTC (rev 78952)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/instrument/Instrumentor.java	2008-09-30 13:38:00 UTC (rev 78953)
@@ -262,14 +262,23 @@
       
       //TODO remove this hack once we have a version of javassist that makes the _added_m$ methods synthetic 
       //The following depends on javassist internals
-      ClassFile cf = clazz.getClassFile();
-      List<MethodInfo> infos = cf.getMethods();
-      MethodInfo info = infos.get(infos.size() - 2);
-      if (info.getName().startsWith("_added_m$"))
+      try
       {
-         addSyntheticAttribute(info);
+         ClassFile cf = clazz.getClassFile();
+         List<MethodInfo> infos = cf.getMethods();
+         MethodInfo info = infos.get(infos.size() - 2);
+         if (info.getName().startsWith("_added_m$"))
+         {
+            addSyntheticAttribute(info);
+         }
       }
-      
+      catch(Exception e)
+      {
+         if (AspectManager.verbose)
+         {
+            logger.warn("Could not find wrapper method for " + method.getName());
+         }
+      }
       return newMethod;
    }