This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git


The following commit(s) were added to refs/heads/master by this push:
     new 7529e74d Javadoc
7529e74d is described below

commit 7529e74def488c2e949fb5c3fe2d877f82f645b6
Author: Gary David Gregory (Code signing key) <ggreg...@apache.org>
AuthorDate: Wed Nov 23 10:21:41 2022 -0500

    Javadoc
---
 .../org/apache/bcel/generic/ConstantPoolGen.java   | 42 ++++++++++++----------
 1 file changed, 23 insertions(+), 19 deletions(-)

diff --git a/src/main/java/org/apache/bcel/generic/ConstantPoolGen.java 
b/src/main/java/org/apache/bcel/generic/ConstantPoolGen.java
index 835fda3b..68feb64f 100644
--- a/src/main/java/org/apache/bcel/generic/ConstantPoolGen.java
+++ b/src/main/java/org/apache/bcel/generic/ConstantPoolGen.java
@@ -89,7 +89,7 @@ public class ConstantPoolGen {
     private final Map<String, Integer> cpTable = new HashMap<>();
 
     /**
-     * Create empty constant pool.
+     * Constructs a new empty constant pool.
      */
     public ConstantPoolGen() {
         size = DEFAULT_BUFFER_SIZE;
@@ -97,7 +97,7 @@ public class ConstantPoolGen {
     }
 
     /**
-     * Initialize with given array of constants.
+     * Constructs a new instance with the given array of constants.
      *
      * @param cs array of given constants, new ones will be appended
      */
@@ -215,7 +215,7 @@ public class ConstantPoolGen {
     }
 
     /**
-     * Initialize with given constant pool.
+     * Constructs a new instance with the given constant pool.
      *
      * @param cp the constant pool.
      */
@@ -267,41 +267,45 @@ public class ConstantPoolGen {
     }
 
     /**
-     * Import constant from another ConstantPool and return new index.
+     * Adds a constant from another ConstantPool and returns the new index.
+     *
+     * @param constant The constant to add.
+     * @param cpGen Source pool.
+     * @return index of entry
      */
-    public int addConstant(final Constant c, final ConstantPoolGen cp) {
-        final Constant[] constants = cp.getConstantPool().getConstantPool();
-        switch (c.getTag()) {
+    public int addConstant(final Constant constant, final ConstantPoolGen 
cpGen) {
+        final Constant[] constants = cpGen.getConstantPool().getConstantPool();
+        switch (constant.getTag()) {
         case Const.CONSTANT_String: {
-            final ConstantString s = (ConstantString) c;
+            final ConstantString s = (ConstantString) constant;
             final ConstantUtf8 u8 = (ConstantUtf8) 
constants[s.getStringIndex()];
             return addString(u8.getBytes());
         }
         case Const.CONSTANT_Class: {
-            final ConstantClass s = (ConstantClass) c;
+            final ConstantClass s = (ConstantClass) constant;
             final ConstantUtf8 u8 = (ConstantUtf8) constants[s.getNameIndex()];
             return addClass(u8.getBytes());
         }
         case Const.CONSTANT_NameAndType: {
-            final ConstantNameAndType n = (ConstantNameAndType) c;
+            final ConstantNameAndType n = (ConstantNameAndType) constant;
             final ConstantUtf8 u8 = (ConstantUtf8) constants[n.getNameIndex()];
             final ConstantUtf8 u8_2 = (ConstantUtf8) 
constants[n.getSignatureIndex()];
             return addNameAndType(u8.getBytes(), u8_2.getBytes());
         }
         case Const.CONSTANT_Utf8:
-            return addUtf8(((ConstantUtf8) c).getBytes());
+            return addUtf8(((ConstantUtf8) constant).getBytes());
         case Const.CONSTANT_Double:
-            return addDouble(((ConstantDouble) c).getBytes());
+            return addDouble(((ConstantDouble) constant).getBytes());
         case Const.CONSTANT_Float:
-            return addFloat(((ConstantFloat) c).getBytes());
+            return addFloat(((ConstantFloat) constant).getBytes());
         case Const.CONSTANT_Long:
-            return addLong(((ConstantLong) c).getBytes());
+            return addLong(((ConstantLong) constant).getBytes());
         case Const.CONSTANT_Integer:
-            return addInteger(((ConstantInteger) c).getBytes());
+            return addInteger(((ConstantInteger) constant).getBytes());
         case Const.CONSTANT_InterfaceMethodref:
         case Const.CONSTANT_Methodref:
         case Const.CONSTANT_Fieldref: {
-            final ConstantCP m = (ConstantCP) c;
+            final ConstantCP m = (ConstantCP) constant;
             final ConstantClass clazz = (ConstantClass) 
constants[m.getClassIndex()];
             final ConstantNameAndType n = (ConstantNameAndType) 
constants[m.getNameAndTypeIndex()];
             ConstantUtf8 u8 = (ConstantUtf8) constants[clazz.getNameIndex()];
@@ -310,7 +314,7 @@ public class ConstantPoolGen {
             final String name = u8.getBytes();
             u8 = (ConstantUtf8) constants[n.getSignatureIndex()];
             final String signature = u8.getBytes();
-            switch (c.getTag()) {
+            switch (constant.getTag()) {
             case Const.CONSTANT_InterfaceMethodref:
                 return addInterfaceMethodref(className, name, signature);
             case Const.CONSTANT_Methodref:
@@ -318,11 +322,11 @@ public class ConstantPoolGen {
             case Const.CONSTANT_Fieldref:
                 return addFieldref(className, name, signature);
             default: // Never reached
-                throw new IllegalArgumentException("Unknown constant type " + 
c);
+                throw new IllegalArgumentException("Unknown constant type " + 
constant);
             }
         }
         default: // Never reached
-            throw new IllegalArgumentException("Unknown constant type " + c);
+            throw new IllegalArgumentException("Unknown constant type " + 
constant);
         }
     }
 

Reply via email to