Skip to content

Commit

Permalink
BCEL: use MAX_CP_ENTRIES from internal class, not from JRE
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
kriegaex committed Dec 21, 2022
1 parent ffacda7 commit 9161b23
Showing 1 changed file with 4 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@
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;
Expand Down Expand Up @@ -294,7 +293,7 @@ public void dump(DataOutputStream file) throws IOException {
* 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)
Expand Down Expand Up @@ -425,17 +424,17 @@ public int addClass(String classname) {

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);
}
Expand Down

0 comments on commit 9161b23

Please sign in to comment.