On Fri, 18 Nov 2022 21:21:20 GMT, Jonathan Gibbons <[email protected]> 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.
>
> Jonathan Gibbons has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fix man page header

Marked as reviewed by prappo (Reviewer).

Looks good to me.

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

> 518:      * @return an array of tokens
> 519:      */
> 520:     private List<String> tokenize(String s, int maxTokens) {

Not saying that we should do anything about it, but I would be surprised if 
there were no such tokenization functionality in the JDK API.

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

PR: https://git.openjdk.org/jdk/pull/11178Marked as reviewed by prappo 
(Reviewer).

Reply via email to