On Wed, 15 Apr 2026 16:05:01 GMT, Hannes Wallnöfer <[email protected]> wrote:

>> Please review an enhancement that adds a new default `toString` method to 
>> the `jdk.javadoc.doclet.Taglet` interface which receives the path to the 
>> documentation root directory as additional argument of type `java.net.URI`. 
>> (The initial version of this PR used `java.nio.file.Path` which was of 
>> course a blunder as that class uses a platform-dependent separator.)
>> 
>> This makes it possible to create correct relative URLs when a tag is 
>> rendered outside of its element's resource (such as in a summary page), 
>> which was previously impossible using the public Taglet API. It also makes 
>> it easier to build relative URLs in the common case where a tag is rendered 
>> in its element's normal location.
>> 
>> I also added the changes for 
>> [JDK-8379520](https://bugs.openjdk.org/browse/JDK-8379520) to make use of 
>> the new API in JDK build Taglets to this PR. The changes to the `JSpec` and 
>> `ToolGuide` taglets consist in undoing the changes from 
>> [JDK-8373909](https://bugs.openjdk.org/browse/JDK-8373909) and replacing it 
>> with the new functionality. 
>> 
>> For `SealedGraph` it's a bit more involved, because in order to be able to 
>> use the `pathToRoot` URI I had to change the naming of .svg files for nested 
>> classes to not use subdirectories for enclosing classes so that these files 
>> use the same directory as all the other files from that package.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Hannes Wallnöfer has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains nine commits:
> 
>  - Merge branch 'master' into JDK-8373922
>  - More doc comment changes
>  - Refine doc comments
>  - Document standard doclet behavior and thrown exceptions
>  - Update copyright headers
>  - JDK-8379520: Make use of new Taglet.toString method in JDK build taglets
>  - Switch to java.net.URI, improve spec and test
>  - Add test
>  - JDK-8373922: Enhance Taglet API to support relative URLs

Marked as reviewed by liach (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/30120#pullrequestreview-4121639633

Reply via email to