Skip to content

Commit 57ba5b7

Browse files
committed
Rename list containing active jvmti envs. Pass cause to shouldNotReachHere in NativeMemberOffsetGetter.
1 parent 334f349 commit 57ba5b7

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

espresso/src/com.oracle.truffle.espresso/src/com/oracle/truffle/espresso/jvmti/JVMTI.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public final class JVMTI {
5757
@CompilationFinal //
5858
private volatile Structs structs;
5959

60-
private final ArrayList<JVMTIEnv> created = new ArrayList<>();
60+
private final ArrayList<JVMTIEnv> activeEnvironments = new ArrayList<>();
6161

6262
private JvmtiPhase phase;
6363

@@ -128,23 +128,23 @@ public synchronized TruffleObject create(int version) {
128128
return null;
129129
}
130130
JVMTIEnv jvmtiEnv = new JVMTIEnv(context, initializeJvmtiContext, version);
131-
created.add(jvmtiEnv);
131+
activeEnvironments.add(jvmtiEnv);
132132
return jvmtiEnv.getEnv();
133133
}
134134

135135
public synchronized void dispose() {
136-
for (JVMTIEnv jvmtiEnv : created) {
136+
for (JVMTIEnv jvmtiEnv : activeEnvironments) {
137137
jvmtiEnv.dispose(disposeJvmtiContext);
138138
}
139-
created.clear();
139+
activeEnvironments.clear();
140140
}
141141

142142
@TruffleBoundary
143143
synchronized void dispose(JVMTIEnv env) {
144144
CompilerAsserts.neverPartOfCompilation();
145-
if (created.contains(env)) {
145+
if (activeEnvironments.contains(env)) {
146146
env.dispose(disposeJvmtiContext);
147-
created.remove(env);
147+
activeEnvironments.remove(env);
148148
}
149149
}
150150

espresso/src/com.oracle.truffle.espresso/src/com/oracle/truffle/espresso/jvmti/structs/NativeMemberOffsetGetter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ private long lookupInfo(String str) {
5555
try (RawBuffer memberBuffer = RawBuffer.getNativeString(str)) {
5656
return (long) library.execute(lookupMemberOffset, memberInfoPtr, memberBuffer.pointer());
5757
} catch (UnsupportedTypeException | ArityException | UnsupportedMessageException e) {
58-
throw EspressoError.shouldNotReachHere();
58+
throw EspressoError.shouldNotReachHere(e);
5959
}
6060
}
6161
}

0 commit comments

Comments
 (0)