]> source.dussan.org Git - aspectj.git/commitdiff
fix for problem LTW in multi-threaded configuration - removed static from bcel classes
authoraclement <aclement>
Tue, 22 May 2007 09:11:40 +0000 (09:11 +0000)
committeraclement <aclement>
Tue, 22 May 2007 09:11:40 +0000 (09:11 +0000)
bcel-builder/src/org/aspectj/apache/bcel/generic/BranchHandle.java
bcel-builder/src/org/aspectj/apache/bcel/generic/InstructionHandle.java

index eed275112dc682d155ae3af7d392ba3cd7e2090b..74788f5ea6ecfe7c8b23264c917788b520830284 100644 (file)
@@ -63,7 +63,7 @@ package org.aspectj.apache.bcel.generic;
  * @see InstructionHandle
  * @see Instruction
  * @see InstructionList
- * @version $Id: BranchHandle.java,v 1.2 2004/11/19 16:45:19 aclement Exp $
+ * @version $Id: BranchHandle.java,v 1.3 2007/05/22 09:11:40 aclement Exp $
  * @author  <A HREF="mailto:markus.dahm@berlin.de">M. Dahm</A>
  */
 public final class BranchHandle extends InstructionHandle {
@@ -76,26 +76,9 @@ public final class BranchHandle extends InstructionHandle {
 
   /** Factory methods.
    */
-  private static BranchHandle bh_list = null; // List of reusable handles
 
   static final BranchHandle getBranchHandle(BranchInstruction i) {
-    if(bh_list == null)
       return new BranchHandle(i);
-    else {
-      BranchHandle bh = bh_list;
-      bh_list = (BranchHandle)bh.next;
-
-      bh.setInstruction(i);
-
-      return bh;
-    }
-  }
-  
-  /** Handle adds itself to the list of resuable handles.
-   */
-  protected void addHandle() {
-    next    = bh_list;
-    bh_list = this;
   }
 
   /* Override InstructionHandle methods: delegate to branch instruction.
index df4b4b547ae29a5bb26f4fc4de43ee903719ab17..b4a2b4c8ef659ae43de77d483bb8766df6c2dc25 100644 (file)
@@ -71,7 +71,7 @@ import java.util.HashMap;
  * can traverse the list via an Enumeration returned by
  * InstructionList.elements().
  *
- * @version $Id: InstructionHandle.java,v 1.2 2004/11/19 16:45:19 aclement Exp $
+ * @version $Id: InstructionHandle.java,v 1.3 2007/05/22 09:11:40 aclement Exp $
  * @author  <A HREF="mailto:markus.dahm@berlin.de">M. Dahm</A>
  * @see Instruction
  * @see BranchHandle
@@ -120,21 +120,11 @@ public class InstructionHandle implements java.io.Serializable {
     setInstruction(i);
   }
 
-  private static InstructionHandle ih_list = null; // List of reusable handles
 
   /** Factory method.
    */
   static final InstructionHandle getInstructionHandle(Instruction i) {
-    if(ih_list == null)
       return new InstructionHandle(i);
-    else {
-      InstructionHandle ih = ih_list;
-      ih_list = ih.next;
-
-      ih.setInstruction(i);
-
-      return ih;
-    }
   }
 
   /**
@@ -163,12 +153,6 @@ public class InstructionHandle implements java.io.Serializable {
    */
   void setPosition(int pos) { i_position = pos; }
 
-  /** Overridden in BranchHandle
-   */
-  protected void addHandle() {
-    next    = ih_list;
-    ih_list = this;
-  }
 
   /**
    * Delete contents, i.e., remove user access and make handle reusable.
@@ -180,7 +164,6 @@ public class InstructionHandle implements java.io.Serializable {
     i_position = -1;
     attributes = null;
     removeAllTargeters();
-    addHandle();
   }
 
   /** Remove all targeters, if any.