Integrated: JDK-8261665: Clean up naming of StringContent and FixedStringContent

Jonathan Gibbons jjg at openjdk.java.net
Tue Feb 23 19:32:58 UTC 2021


On Fri, 12 Feb 2021 19:51:34 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> Relatively simple update to clean up some of the old naming in the `html.markup` world.
> 
> `FixedStringContent` is renamed to just `Text` and a new factory method `Text.of(CharSequence)` introduced.
> 
> `StringContent` is renamed to `TextBuilder`.
> 
> `StringContent` was the original type and is widely used; `FixedStringContent` was added much later, along with `Contents` to support shared reusable tree nodes.  But most uses of `StringContent` do not need mutability, and most can be replaced with the lighter-weight `Text` class.  This change is done, just leaving `textBuilder` to be used in `ContentBuilder` and `HtmlTree` which support the `add(CharSequence)` method.

This pull request has now been integrated.

Changeset: 3132b1c4
Author:    Jonathan Gibbons <jjg at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/3132b1c4
Stats:     574 lines in 41 files changed: 200 ins; 206 del; 168 mod

8261665: Clean up naming of StringContent and FixedStringContent

Reviewed-by: hannesw

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

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


More information about the javadoc-dev mailing list