]> source.dussan.org Git - aspectj.git/commitdiff
BCEL: use MAX_CP_ENTRIES from internal class, not from JRE
authorAlexander Kriegisch <Alexander@Kriegisch.name>
Wed, 21 Dec 2022 14:56:34 +0000 (15:56 +0100)
committerAlexander Kriegisch <Alexander@Kriegisch.name>
Wed, 21 Dec 2022 14:56:34 +0000 (15:56 +0100)
Instead of importing com.sun.org.apache.bcel.internal.Const, use
use org.aspectj.apache.bcel.Constants. The former class is from the
internal JRE module 'java.xml' which is not exposed by default.
Actually, no existing test failed because of it, but javadoc generation
for the AspectJ weaver.

Relates to #192.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
bcel-builder/src/main/java/org/aspectj/apache/bcel/classfile/ConstantPool.java

index 5b434651e1cbf337fe1c3518b92c4b14a0d786f3..ba188b064ce2ba86ad1c5fdfec3d3cdf7880207e 100644 (file)
@@ -59,7 +59,6 @@ import java.io.IOException;
 import java.util.HashMap;
 import java.util.Map;
 
-import com.sun.org.apache.bcel.internal.Const;
 import org.aspectj.apache.bcel.Constants;
 import org.aspectj.apache.bcel.generic.ArrayType;
 import org.aspectj.apache.bcel.generic.ObjectType;
@@ -294,7 +293,7 @@ public class ConstantPool implements Node {
                 * This is a redundant measure as the ConstantPoolGen should have already
                 * reported an error back in the situation.
                 */
-               final int size = Math.min(poolSize, Const.MAX_CP_ENTRIES);
+               final int size = Math.min(poolSize, Constants.MAX_CP_ENTRIES);
                file.writeShort(size);
                for (int i = 1; i < size; i++)
                        if (pool[i] != null)
@@ -425,17 +424,17 @@ public class ConstantPool implements Node {
 
        private void adjustSize() {
     // 3 extra spaces are needed as some entries may take 3 slots
-    if (poolSize + 3 >= Const.MAX_CP_ENTRIES + 1) {
+    if (poolSize + 3 >= Constants.MAX_CP_ENTRIES + 1) {
       throw new IllegalStateException(
         "The number of constants " + (poolSize + 3) +
-        " is over the size of the constant pool: " + Const.MAX_CP_ENTRIES
+        " is over the size of the constant pool: " + Constants.MAX_CP_ENTRIES
       );
     }
     if (poolSize + 3 >= pool.length) {
                        Constant[] cs = pool;
                        int size = cs.length + 8;
                        // the constant array shall not exceed the size of the constant pool
-                       size = Math.min(size, Const.MAX_CP_ENTRIES + 1);
+                       size = Math.min(size, Constants.MAX_CP_ENTRIES + 1);
                        pool = new Constant[size];
                        System.arraycopy(cs, 0, pool, 0, cs.length);
                }