Skip to content

Commit

Permalink
Rename isRefOrValSignature to isClassSignature
Browse files Browse the repository at this point in the history
Signed-off-by: Theresa Mammarella <[email protected]>
  • Loading branch information
theresa-m committed Sep 12, 2024
1 parent b979879 commit c6a93e4
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/j9/ArgBits.java
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ public class ArgBits
/* Parse the signature inside the ()'s */
char thisChar;
while ((thisChar = signature.charAt(++stringPtr)) != ')') {
if ((thisChar == '[') || J9ClassHelper.isRefOrValSignature(thisChar)) {
if ((thisChar == '[') || J9ClassHelper.isClassSignature(thisChar)) {
/* Mark a bit for objects or arrays */
resultArray[writePtr] |= argBit;

Expand All @@ -59,7 +59,7 @@ public class ArgBits
stringPtr++;
}

if (J9ClassHelper.isRefOrValSignature(thisChar)) {
if (J9ClassHelper.isClassSignature(thisChar)) {
/* Walk past the name of the object class */
while ((thisChar = signature.charAt(stringPtr)) != ';') {
stringPtr++;
Expand Down
2 changes: 1 addition & 1 deletion debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/j9/SendSlot.java
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ public static UDATA getSendSlotsFromSignature(J9UTF8Pointer signature) throws Co
/* skip all '['s */
for (i++; J9UTF8Helper.stringValue(signature).charAt(i) == '['; i++);
char charAti = J9UTF8Helper.stringValue(signature).charAt(i);
if (J9ClassHelper.isRefOrValSignature(charAti)) {
if (J9ClassHelper.isClassSignature(charAti)) {
/* FALL THRU */
} else {
sendArgs = sendArgs.add(1);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -502,7 +502,7 @@ private int mapStack(U16 totalStack, byte[] map,
utf8Signature = J9ROMFieldRefPointer.cast(pool.add(index)).nameAndSignature().signature();

signature = J9UTF8Helper.stringValue(utf8Signature).charAt(0);
if (J9ClassHelper.isRefOrValSignature(signature) || (signature == '[')) {
if (J9ClassHelper.isClassSignature(signature) || (signature == '[')) {
PUSH(OBJ);
} else {
PUSH(INT);
Expand All @@ -515,7 +515,7 @@ private int mapStack(U16 totalStack, byte[] map,
utf8Signature = J9ROMFieldRefPointer.cast(pool.add(index)).nameAndSignature().signature();

signature = J9UTF8Helper.stringValue(utf8Signature).charAt(0);
if (J9ClassHelper.isRefOrValSignature(signature) || (signature == '[')) {
if (J9ClassHelper.isClassSignature(signature) || (signature == '[')) {
PUSH(OBJ);
} else {
PUSH(INT);
Expand Down Expand Up @@ -574,7 +574,7 @@ private int mapStack(U16 totalStack, byte[] map,
while (args[i] == '[') {
i += 1;
}
if (J9ClassHelper.isRefOrValSignature(args[i])) {
if (J9ClassHelper.isClassSignature(args[i])) {
do {
i += 1;
} while (args[i] != ';');
Expand All @@ -583,7 +583,7 @@ private int mapStack(U16 totalStack, byte[] map,

signature = args[i + 1];
if (signature != 'V') {
if (J9ClassHelper.isRefOrValSignature(signature) || (signature == '[')) {
if (J9ClassHelper.isClassSignature(signature) || (signature == '[')) {
PUSH(OBJ);
} else {
PUSH(INT);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -533,7 +533,7 @@ public static boolean isAnonymousClass(J9ClassPointer clazz) throws CorruptDataE
* @param firstChar the first character of the signature being checked
* @return true if the character indicates the beginning of a reference or value signature, false otherwise
*/
public static boolean isRefOrValSignature(char firstChar) {
public static boolean isClassSignature(char firstChar) {
return firstChar == 'L';
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ public static J9ClassPointer findClass(J9ClassLoaderPointer classLoader, String

int arity = calculateClassArity(signature);

if ((arity > 0) && !J9ClassHelper.isRefOrValSignature(signature.charAt(arity))) {
if ((arity > 0) && !J9ClassHelper.isClassSignature(signature.charAt(arity))) {
return PRIMITIVE_TO_CLASS.get(signature.charAt(arity));
} else {
while (classIterator.hasNext()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,7 @@ boolean isObjectArray() throws CorruptDataException, com.ibm.j9ddr.CorruptDataEx
J9ArrayClassPointer arrayClass = J9ArrayClassPointer.cast(j9class);

// Any multi-dimensional array is an object array, as is any single dimensional array with an object type (i.e. [Lwhatever; or [Qwhatever;).
if (arrayClass.arity().gt(1) || J9ClassHelper.isRefOrValSignature(getName().charAt(1))) {
if (arrayClass.arity().gt(1) || J9ClassHelper.isClassSignature(getName().charAt(1))) {
isObjectArray = true;
} else {
isObjectArray = false;
Expand Down

0 comments on commit c6a93e4

Please sign in to comment.