You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2015/08/13 11:22:05 UTC

svn commit: r1695664 - /commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/Type.java

Author: sebb
Date: Thu Aug 13 09:22:04 2015
New Revision: 1695664

URL: http://svn.apache.org/r1695664
Log:
Don't expose the setter unnecessarily

Modified:
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/Type.java

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/Type.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/Type.java?rev=1695664&r1=1695663&r2=1695664&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/Type.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/Type.java Thu Aug 13 09:22:04 2015
@@ -393,10 +393,12 @@ public abstract class Type implements ja
     }
 
 
-    /**
-     * @param signature the signature to set
+    /*
+     * Currently only used by the ArrayType constructor.
+     * The signature has a complicated dependency on other parameter
+     * so it's tricky to do it in a call to the super ctor.
      */
-    public void setSignature(String signature) {
+    void setSignature(String signature) {
         this.signature = signature;
     }
 }