On Mon, 12 Oct 2020 22:36:51 GMT, Jonathan Gibbons <j...@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 <j...@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
