Integrated: JDK-8313349: Introduce `abstract void HtmlDocletWriter.buildPage()`

Jonathan Gibbons jjg at openjdk.org
Wed Aug 2 22:02:50 UTC 2023


On Tue, 1 Aug 2023 23:52:15 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> Please review a medium size update that is the last in the series of recent updates to refactor and simplify  builders and writers.
> 
> Having cleared away some of the earlier obfuscation, it becomes clear that all subtypes of `HtmlDocletWriter` generate pages for the overall documentation, and so a new abstract method `buildPage` is introduced to make that more obvious. This resulted in renaming some existing `build` methods and/or removing some static `generate` methods, though some of the more complex ones still remain.
> 
> In addition, all writers are now generally created by the `WriterFactory` class, providing an extension point should it ever be needed to subtype these classes. The methods on `WriterFactory` return the appropriate supertype (`HtmlDocletWriter` or `AbstractMemberWriter`) thus enforcing the use of the standard abstract methods that are now available.
> 
> There is also some cleanup to some of the member writers, including introducing more abstract methods to define common behavior, simplifying some method signatures, removing unused code, and providing documentation comments.

This pull request has now been integrated.

Changeset: 6d180d5f
Author:    Jonathan Gibbons <jjg at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6d180d5fbfb7ba2a6cebebe637e791dd540a80d7
Stats:     1249 lines in 46 files changed: 420 ins; 499 del; 330 mod

8313349: Introduce `abstract void HtmlDocletWriter.buildPage()`

Reviewed-by: prappo

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

PR: https://git.openjdk.org/jdk/pull/15114


More information about the javadoc-dev mailing list