On Mon, 16 Sep 2024 04:15:28 GMT, Jaikiran Pai <j...@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
