RFR: 8356814: LineBreakMeasurer.nextLayout() slower than necessary when no break needed
Phil Race
prr at openjdk.org
Wed May 14 17:12:52 UTC 2025
On Wed, 14 May 2025 07:40:07 GMT, Daniel Gredler <dgredler at 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.
I don't think we need to do that. If it is done, a separate clean up PR would be better rather than adding it in here.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25193#issuecomment-2880960222
More information about the client-libs-dev
mailing list