RFR: 8339918: Remove checks for outdated -t -tm -Xfuture -checksource -cs -noasyncgc options from the launcher [v7]
David Holmes
dholmes at openjdk.org
Mon Sep 16 05:04:04 UTC 2024
On Mon, 16 Sep 2024 04:44:39 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Can I please get a review for this change which cleans up the `java` launcher to remove checks/support for outdated options?
>>
>> As noted in https://bugs.openjdk.org/browse/JDK-8339918, these 6 options have been outdated and unsupported for several releases now. 2 of them even throw an error currently. The change in this PR removes the code which had specific checks for these options and will now consider these options just like any other unknown option to the launcher.
>>
>> tier1, tier2 and tier3 testing passed with these changes. Higher tier testing is in progress.
>>
>> Would this change require a CSR?
>
> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>
> remove usage of -ms -mx -ss and -oss from com.sun.tools.example.debug.tty.TTY
Ship it! :)
-------------
Marked as reviewed by dholmes (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/20945#pullrequestreview-2305746316
More information about the core-libs-dev
mailing list