On Wed, 14 Jun 2023 15:40:41 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
> A recent change in JDK-8304831 ensured that insertionIndex is always computed > prior to HitInfo construction, making it possible to remove buggy logic in > HitInfo. This change also allows for removal of the 'text' field. > > There should be no impact since the constructor is package protected. The fix looks good to me. Can we add a test to verify the changes? Another minor comment: copyright year can be updated in Text.java as well. ------------- PR Review: https://git.openjdk.org/jfx/pull/1154#pullrequestreview-1481756712