Integrated: JDK-8254627: Cleanup {Abstract, Single, Split}IndexWriter classes

Jonathan Gibbons jjg at openjdk.java.net
Tue Oct 13 16:26:13 UTC 2020


On Mon, 12 Oct 2020 22:36:51 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> Inspired by other recent work to cleanup index generation, this change is primarily about cleanup for
> `AbstractIndexWriter`, `SingleIndexWriter`, `SplitIndexWriter` to reduce code duplication and to reduce the 3 class to
> just 1 that can handle either single index files or split index files.  As part of this cleanup, some anomalies were
> uncovered in the naming of annotation types, enum constants and records, which have also been addressed.
> The changes involved moving/merging code from `SingleIndexWriter` and `SplitIndexWriter` in to `AbstractIndexWriter`,
> which is now no longer abstract and is thus renamed to just `IndexWriter`. Regrettably, the rename is not properly
> tracked.

This pull request has now been integrated.

Changeset: 3fb2e822
Author:    Jonathan Gibbons <jjg at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/3fb2e822
Stats:     1065 lines in 15 files changed: 417 ins; 607 del; 41 mod

8254627: Cleanup {Abstract,Single,Split}IndexWriter classes

Reviewed-by: hannesw

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

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


More information about the javadoc-dev mailing list