Skip to content

Commit

Permalink
Merge pull request #21 from LeMyst/fix-add-processmanager-jdk9
Browse files Browse the repository at this point in the history
Forgot the main part
  • Loading branch information
LeMyst authored Jun 19, 2024
2 parents b7a910c + 834a625 commit d9faa3a
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/main/java/org/cyclopsgroup/jmxterm/cc/JPMFactory.java
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import org.cyclopsgroup.jmxterm.JavaProcessManager;
import org.cyclopsgroup.jmxterm.jdk5.Jdk5JavaProcessManager;
import org.cyclopsgroup.jmxterm.jdk6.Jdk6JavaProcessManager;
import org.cyclopsgroup.jmxterm.jdk9.Jdk9JavaProcessManager;
import org.cyclopsgroup.jmxterm.pm.JConsoleClassLoaderFactory;
import org.cyclopsgroup.jmxterm.pm.UnsupportedJavaProcessManager;

Expand All @@ -27,16 +28,18 @@ public JPMFactory() {
JavaProcessManager j;
try {
ClassLoader cl = JConsoleClassLoaderFactory.getClassLoader();
if (SystemUtils.IS_JAVA_1_5) {
j = new Jdk5JavaProcessManager(cl);
} else {
if (SystemUtils.isJavaVersionAtLeast(JavaVersion.JAVA_9)) {
j = new Jdk9JavaProcessManager(cl);
} else if (SystemUtils.isJavaVersionAtLeast(JavaVersion.JAVA_1_6)) {
j = new Jdk6JavaProcessManager(cl);
} else {
j = new Jdk5JavaProcessManager(cl);
}
} catch (ClassNotFoundException e) {
j =
new UnsupportedJavaProcessManager(
e.getMessage()
+ ", operation on this JDK("
+ ", operation on this JDK ("
+ SystemUtils.JAVA_RUNTIME_VERSION
+ ") isn't fully supported",
e);
Expand Down

0 comments on commit d9faa3a

Please sign in to comment.