On Wed, 30 Sep 2020 10:02:11 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

>> This is a general cleanup of various classes related to the handling of tags 
>> and taglets in the standard doclet.
>> 
>> The initial motivation was to clean up static methods on `TagletWriter` that 
>> took a `TagletWriter` as a parameter.
>> These methods are converted into instance methods, and the call sites 
>> simplified as well.
>> Another area of cleanup is to make clear which methods are used for inline 
>> tags and which are used for block tags.
>> 
>> Some dead code is removed, and the doc comments are generally cleaned up as 
>> well.  More details are given in the JBS
>> issue.
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/TagletWriter.java
>  line 282:
> 
>> 280:             } catch (UnsupportedTagletOperationException e) {
>> 281:                 // malformed taglet:
>> 282:                 // claims to support block tags but does not provide 
>> the appropriate method.
> 
> This deserves a more detailed explanation: what does this claim look like? 
> (It's not obvious from reading this excerpt.)

Fixed; for reference here, "claims to support" is a reference to 
`Taglet.isBlockTag` / `Taglet.isInlineTag`

> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/TagletWriter.java
>  line 372:
> 
>> 370:
>> 371:     /**
>> 372:      * Returns  an instance of the configuration used for this doclet.
> 
> Remove extra whitespace in "Returns  an".

👍

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

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

Reply via email to