> 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

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/20945/files
  - new: https://git.openjdk.org/jdk/pull/20945/files/4822c831..8cdf5627

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=20945&range=06
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20945&range=05-06

  Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/20945.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20945/head:pull/20945

PR: https://git.openjdk.org/jdk/pull/20945

Reply via email to