On Fri, 31 Oct 2025 17:17:06 GMT, Daniel Gredler <[email protected]> wrote:
> The class `TextSourceLabel` is unused, as is the associated factory method > `TextLabelFactory.createSimple(...)`. > > The abstractions provided by the abstract classes `TextLabel` and > `ExtendedTextLabel` only make sense when there are two types of text labels > (`TextSourceLabel` and `ExtendedTextSourceLabel`). With the deletion of > `TextSourceLabel`, both `TextLabel` and `ExtendedTextLabel` can also be > removed. > > The JavaDoc for the abstract methods in `TextLabel` and `ExtendedTextLabel`, > which are implemented in `ExtendedTextSourceLabel`, was moved to > `ExtendedTextSourceLabel` prior to the deletion of these two classes. > > The few small convenience methods in `TextLabel` and `ExtendedTextLabel` were > also moved to `ExtendedTextSourceLabel` prior to the deletion of these two > classes. > > I would have liked to give the combined `TextLabel` + `ExtendedTextLabel` + > `ExtendedTextSourceLabel` class the name `TextLabel`, but I've used > `ExtendedTextSourceLabel` instead because it makes the diff much smaller and > maximizes code history traceability. > > See initial mailing list discussion: > https://mail.openjdk.org/pipermail/client-libs-dev/2025-September/032302.html This pull request has now been integrated. Changeset: 4042e821 Author: Daniel Gredler <[email protected]> URL: https://git.openjdk.org/jdk/commit/4042e821c6f582bf31201acb9f2d98d940383f1c Stats: 649 lines in 7 files changed: 137 ins; 480 del; 32 mod 8371066: Remove unused class TextSourceLabel and associated class hierarchy Reviewed-by: prr, psadhukhan ------------- PR: https://git.openjdk.org/jdk/pull/28090
