Author: sebb
Date: Mon Aug 10 15:59:45 2015
New Revision: 1695121

URL: http://svn.apache.org/r1695121
Log:
Don't call overrideable methods from a constructor

Modified:
    
commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/classfile/ConstantPool.java

Modified: 
commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/classfile/ConstantPool.java
URL: 
http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/classfile/ConstantPool.java?rev=1695121&r1=1695120&r2=1695121&view=diff
==============================================================================
--- 
commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/classfile/ConstantPool.java
 (original)
+++ 
commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/classfile/ConstantPool.java
 Mon Aug 10 15:59:45 2015
@@ -47,7 +47,7 @@ public class ConstantPool implements Clo
      * @param constant_pool Array of constants
      */
     public ConstantPool(Constant[] constant_pool) {
-        setConstantPool(constant_pool);
+        this.constant_pool = constant_pool;
     }
 
 
@@ -335,7 +335,7 @@ public class ConstantPool implements Clo
     /**
      * @param constant_pool
      */
-    public void setConstantPool( Constant[] constant_pool ) {
+    public void setConstantPool( Constant[] constant_pool ) { // TODO unused
         this.constant_pool = constant_pool;
     }
 


Reply via email to