> Please review a medium simple update to prefer the use of comma (`,`) instead 
> of colon (`:`) in certain options taking a list of values.    Options taking 
> a search path (like `--class-path`, `--source-path`, etc) are **not** 
> affected.

Jonathan Gibbons has updated the pull request with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase. The pull request contains three additional 
commits since the last revision:

 - Address review feedback
 - Merge remote-tracking branch 'upstream/master' into 8290845.option-separator 
# Please enter a commit message to explain why this merge is necessary, # 
especially if it
   merges an updated upstream into a topic branch. # # Lines starting with '#' 
will be ignored, and an empty message aborts # the commit.
 - JDK-8290845: Consider an alternative item separator for multi-item option 
values

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11178/files
  - new: https://git.openjdk.org/jdk/pull/11178/files/aeca049f..84666156

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

  Stats: 41485 lines in 1055 files changed: 16778 ins; 20133 del; 4574 mod
  Patch: https://git.openjdk.org/jdk/pull/11178.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11178/head:pull/11178

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

Reply via email to