RFR: 8353439: Shell grouping of -XX:OnError= commands is surprising [v2]

Kevin Walls kevinw at openjdk.org
Fri Apr 4 14:33:04 UTC 2025


> We should be consistent, and run all OnError items in a new shell.  Currently the ; separator causes a new shell, but multiple -XX:OnError= options are grouped into the same shell.
> 
> next_OnError_command() decides on where a new command starts.  It should recognise newlines, and all commands will get their own shell.

Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:

  Test udpate - multiple -XX:OnError=

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24354/files
  - new: https://git.openjdk.org/jdk/pull/24354/files/d9646e3a..880261a9

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

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

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


More information about the hotspot-dev mailing list