RFR: 8339918: Remove checks for outdated -t -tm -Xfuture -checksource -cs -noasyncgc options from the launcher [v5]
David Holmes
dholmes at openjdk.org
Mon Sep 16 04:31:05 UTC 2024
On Mon, 16 Sep 2024 04:15:28 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/TTY.java line 991:
>>
>>> 989: // Old-style options (These should remain in place as long as
>>> 990: // the standard VM accepts them)
>>> 991: token.equals("-prof") ||
>>
>> The `-prof` flag doesn't exist either.
>
> Done. There also was `-v` and `-v:...` in this example which aren't supported either (not even in JDK 8). So I've removed those as well.
What about:
token.startsWith("-ms") || token.startsWith("-mx") ||
token.startsWith("-ss") || token.startsWith("-oss") ) {
? `-oss` is gone as of 9. The others are not need as the -X counterpart can be used.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20945#discussion_r1760517430
More information about the core-libs-dev
mailing list