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

Reply via email to