On Tue, 13 May 2025 20:49:59 GMT, Phil Race <p...@openjdk.org> wrote:
> Looks OK. Testing passes. My only concern would have been if these are > mutable, but they aren't Unless something casts to a known sub-class that is > mutable, but I don't know think that is the case. It doesn't look like it, but let me know if you'd like me to make `ExtendedTextSourceLabel` final and make all the instance variables private, just to tighten things up. ------------- PR Comment: https://git.openjdk.org/jdk/pull/25193#issuecomment-2879071641