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