Integrated: 8211033: Clean up the processing -classpath argument not to set LM_CLASS
Jaikiran Pai
jpai at openjdk.org
Mon Nov 11 04:35:31 UTC 2024
On Fri, 8 Nov 2024 08:51:38 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Can I please get a review of this change which addresses https://bugs.openjdk.org/browse/JDK-8211033?
>
> As noted in that issue, this is a clean up of the code which determines the "mode" through with the `java` application is being launched. In its current form the presence of `--classpath` (or its equivalent arguments) was unnecessary updating the mode to `LM_CLASS`. The commit in this PR removes that part to allow for the mode to be detected based merely on the presence (or absence) of `-m`, `-jar`, `--source` options. If neither is specified, the file extension is checked to determine the launch mode.
>
> Given the nature of this clean up, no new tests have been introduced. Existing tests in tier1, tier2, tier3 continue to pass with this change.
This pull request has now been integrated.
Changeset: ca69a53b
Author: Jaikiran Pai <jpai at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/ca69a53b76bf0106443985a1207fbd3d9036e1ab
Stats: 8 lines in 1 file changed: 3 ins; 5 del; 0 mod
8211033: Clean up the processing -classpath argument not to set LM_CLASS
Reviewed-by: alanb
-------------
PR: https://git.openjdk.org/jdk/pull/21971
More information about the core-libs-dev
mailing list