On Tue, 19 Aug 2025 14:01:48 GMT, Marc Philipp <[email protected]> wrote:

> While copying `resources/stylesheet.css` to the target location, the
> line containing the CSS import statement for `fonts/dejavu.css` is now
> removed from the output in case the `--no-fonts` option is enabled.

Thanks for this cleanup of javadoc output. The changes look good, but there are 
two very long lines that should be wrapped.

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/DocFile.java
 line 220:

> 218:      * @throws ResourceIOException if there is a problem while reading 
> the resource
> 219:      */
> 220:     public void copyResource(DocPath resource, URL url, 
> UnaryOperator<String> lineTransformer) throws DocFileIOException, 
> ResourceIOException {

Please wrap the throws clauses on this method and the method below.

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

Changes requested by hannesw (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26843#pullrequestreview-3143813462
PR Review Comment: https://git.openjdk.org/jdk/pull/26843#discussion_r2293225084

Reply via email to