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

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

Author: flavia.rainone at jboss.com
Date: 2008-01-10 18:05:51 -0500 (Thu, 10 Jan 2008)
New Revision: 68854

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/standalone/Compiler.java
Log:
[JBAOP-256] The previous solution was not working correctly on all possible scenarios.

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/standalone/Compiler.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/standalone/Compiler.java	2008-01-10 22:38:12 UTC (rev 68853)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/standalone/Compiler.java	2008-01-10 23:05:51 UTC (rev 68854)
@@ -263,7 +263,12 @@
       {
          String fileName = file.getName().substring(0, index) + ".class";
          File superClassFile = new File(fileName);
-         if (!loadFile(superClassFile))
+         // checking last modified date is not enough
+         // because these values can differ in some miliseconds sometimes
+         if (superClassFile.lastModified() > file.lastModified()
+         // so, check this class has been recompiled and, hence,
+         // is not Advised anymore
+               && !loadFile(superClassFile))
          {
             file.delete();
             return;