> Option --disable-@files is not recognised as valid JVM option, however it is 
> already implemented in the launcher.
> Proposed patch skips --disable-@files option in 
> src/hotspot/share/runtime/arguments.cpp processing, so it does not fall into 
> the category of unknown options.
> The patch also corrects typo in the launcher.properties and adds simple 
> DisableFilesOptionTest.
> 
> Please review.
> 
> Thank you,
> Adam

Adam Sotona has updated the pull request incrementally with one additional 
commit since the last revision:

  handling of --disable-@files moved from runtime/arguments.cpp to 
libjli/java.c ParseArguments
  DisableFilesOptionTest merged to ArgsFileTest::killSwitch
  help typo moved to a new bug

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11183/files
  - new: https://git.openjdk.org/jdk/pull/11183/files/ab754b8d..acddbffb

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=11183&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11183&range=00-01

  Stats: 58 lines in 15 files changed: 6 ins; 40 del; 12 mod
  Patch: https://git.openjdk.org/jdk/pull/11183.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11183/head:pull/11183

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

Reply via email to