Skip to content

Commit 3b672e7

Browse files
apply feedback
1 parent 9d5a62e commit 3b672e7

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

test/one/profiler/test/Runner.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,11 +104,9 @@ private static boolean applicable(Test test) {
104104
Arch[] arch = test.arch();
105105
Jvm[] jvm = test.jvm();
106106
int[] jvmVer = test.jvmVer();
107-
Jvm additionalJvm = currentJvm == Jvm.HOTSPOT_C2 ? Jvm.HOTSPOT : null;
108-
109107
return (os.length == 0 || Arrays.asList(os).contains(currentOs)) &&
110108
(arch.length == 0 || Arrays.asList(arch).contains(currentArch)) &&
111-
(jvm.length == 0 || Arrays.asList(jvm).contains(currentJvm) || Arrays.asList(jvm).contains(additionalJvm)) &&
109+
(jvm.length == 0 || Arrays.asList(jvm).contains(currentJvm) || (currentJvm == Jvm.HOTSPOT_C2 && Arrays.asList(jvm).contains(Jvm.HOTSPOT))) &&
112110
(jvmVer.length == 0 || (currentJvmVersion >= jvmVer[0] && currentJvmVersion <= jvmVer[jvmVer.length - 1]));
113111
}
114112

test/one/profiler/test/TestProcess.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,17 +61,17 @@ private static MethodHandle getPidHandle() {
6161
private final Test test;
6262
private final Os currentOs;
6363
private final String logDir;
64-
private final Jvm currentJVM;
64+
private final Jvm currentJvm;
6565
private final String[] inputs;
6666
private final Process p;
6767
private final Map<String, File> tmpFiles = new HashMap<>();
6868
private final int timeout = 30;
6969

70-
public TestProcess(Test test, Os currentOs, String logDir, Jvm currentJVM) throws Exception {
70+
public TestProcess(Test test, Os currentOs, String logDir, Jvm currentJvm) throws Exception {
7171
this.test = test;
7272
this.currentOs = currentOs;
7373
this.logDir = logDir;
74-
this.currentJVM = currentJVM;
74+
this.currentJvm = currentJvm;
7575
this.inputs = test.inputs();
7676

7777
List<String> cmd = buildCommandLine(test);
@@ -114,7 +114,7 @@ public Os currentOs() {
114114
}
115115

116116
public Jvm currentJvm() {
117-
return this.currentJVM;
117+
return this.currentJvm;
118118
}
119119

120120
public String profilerLibPath() {

0 commit comments

Comments
 (0)