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

Reply via email to