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