On Fri, 8 Nov 2024 08:51:38 GMT, Jaikiran Pai <j...@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 <j...@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
