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

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

Author: kabir.khan at jboss.com
Date: 2007-04-16 11:50:13 -0400 (Mon, 16 Apr 2007)
New Revision: 62369

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java
Log:
[JBAOP-378] Use a unique sequence number instead of GUID for instance domains

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java	2007-04-16 15:43:43 UTC (rev 62368)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java	2007-04-16 15:50:13 UTC (rev 62369)
@@ -46,7 +46,6 @@
 import org.jboss.aop.pointcut.PointcutStats;
 import org.jboss.aop.pointcut.Typedef;
 import org.jboss.aop.pointcut.ast.ClassExpression;
-import org.jboss.util.id.GUID;
 
 /**
  * Comment
@@ -72,6 +71,7 @@
    protected boolean hasOwnTypedefs;
    protected boolean hasOwnPrecedenceDefs;
    protected boolean hasOwnClassMetaData;
+   private static int sequenceNumber;
 
    public Domain(AspectManager manager, String name, boolean parentFirst)
    {
@@ -104,9 +104,8 @@
 
             if (forInstance)
             {
-               GUID guid = new GUID();//Are these guys expensive to create?
                sb.append("_");
-               sb.append(guid.toString());
+               sb.append(getNextSequenceNumber());
             }
             return sb.toString();
          }
@@ -114,6 +113,10 @@
       return name;
    }
 
+   private synchronized static int getNextSequenceNumber()
+   {
+      return sequenceNumber++;
+   }
    /**
     * Inherits interceptor, aspect, advice stack definitions
     *