RFR: JDK-8290845: Consider an alternative item separator for multi-item option values

ExE Boss duke at openjdk.org
Wed Nov 16 21:12:08 UTC 2022


On Tue, 15 Nov 2022 23:06:27 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> 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.

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/BaseConfiguration.java line 478:

> 476:                  * String.split(regex);  instead, we tokenize the string, allowing
> 477:                  * special characters to be escaped with '\'. */
> 478:                 List<String> tokens = tokenize(args.get(1),3);

This should have a space:
Suggestion:

                List<String> tokens = tokenize(args.get(1), 3);

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

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


More information about the javadoc-dev mailing list