On Wed, 12 May 2021 09:14:53 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

>> This fixes two javadoc tag references and several typos. References are 
>> fixed by removing whitespace before the opening `(`. That whitespace caused 
>> the opening `(` and the rest of the reference to be parsed as a link label.
>> 
>> Since we are here, I think this class could also benefit from using modern 
>> APIs. This should be done in a separate PR though. For example:
>> 
>> 1. Numerous instances of `Boolean.valueOf( <var> ).booleanValue()` could be 
>> changed to `Boolean.parseBoolean( <var> )`
>> 2. `throw (IllegalArgumentException) new 
>> IllegalArgumentException(msg).initCause(e);` could be changed to `throw new 
>> IllegalArgumentException(msg, e);`
>
> Pavel Rappo has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Added missing full stops

Marked as reviewed by dfuchs (Reviewer).

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

PR: https://git.openjdk.java.net/jdk/pull/3885

Reply via email to