Skip to content
This repository was archived by the owner on Mar 27, 2026. It is now read-only.

Remove code for JDK 5 and 6#156

Merged
LeMyst merged 7 commits into
LeMyst:mainfrom
nyg:clean_up_old_jdk_code
Sep 28, 2025
Merged

Remove code for JDK 5 and 6#156
LeMyst merged 7 commits into
LeMyst:mainfrom
nyg:clean_up_old_jdk_code

Conversation

@nyg
Copy link
Copy Markdown

@nyg nyg commented Sep 26, 2025

As this fork of jmxterm is compiled with JDK 17, it make no sense to keep code that is specific to the (very) old JDK 5 and 6.

Let me know if you want more details on the changes.

@LeMyst LeMyst requested a review from Copilot September 27, 2025 10:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes legacy code for JDK 5 and 6 compatibility from the jmxterm fork, since it's now compiled with JDK 17. The changes modernize the codebase by eliminating obsolete version-specific implementations and simplifying the process manager factory.

  • Removes all JDK 5 and 6 specific classes and their reflection-based implementations
  • Eliminates dynamic JDK version detection and weak casting utilities
  • Simplifies the process manager factory to only support JDK 9+ functionality

Reviewed Changes

Copilot reviewed 29 out of 29 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
WeakCastUtils.java Removes reflection utility class used for legacy JDK compatibility
UnsupportedJavaProcessManager.java Removes fallback process manager implementation
JConsoleClassLoaderFactory.java Removes class loader factory for tools.jar/jconsole.jar handling
jdk5/ and jdk6/ packages Removes entire packages with JDK-specific implementations
jdk9/ package Modernizes to use direct JDK API instead of reflection
JPMFactory.java Simplifies to only create JDK 9+ process managers
Test files Updates test classes to use modernized implementations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/main/java/org/cyclopsgroup/jmxterm/jdk9/Jdk9JavaProcess.java Outdated
Comment thread src/main/java/org/cyclopsgroup/jmxterm/jdk9/Jdk9JavaProcess.java Outdated
Comment thread src/main/java/org/cyclopsgroup/jmxterm/cc/CommandCenter.java Outdated
@LeMyst LeMyst requested a review from Copilot September 28, 2025 01:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 30 out of 30 changed files in this pull request and generated 3 comments.

Comment thread src/main/java/org/cyclopsgroup/jmxterm/jdk9/Jdk9JavaProcessManager.java Outdated
Comment thread src/main/java/org/cyclopsgroup/jmxterm/jdk9/Jdk9JavaProcess.java Outdated
Comment thread src/main/java/org/cyclopsgroup/jmxterm/cc/JPMFactory.java
@LeMyst LeMyst merged commit 902b6fc into LeMyst:main Sep 28, 2025
5 checks passed
@nyg nyg deleted the clean_up_old_jdk_code branch October 15, 2025 16:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants