RFR: 8302511: HitInfo.toString() throws IllegalArgumentException

Andy Goryachev angorya at openjdk.org
Wed Jun 14 16:13:26 UTC 2023


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.

-------------

Commit messages:
 - no change
 - Merge branch 'master' into 8302511.hit.info
 - 8302511: HitInfo.toString() throws IllegalArgumentException
 - 8194704: Text/TextFlow hitTest() javadoc
 - Merge master to resolve merge conflict
 - Stabilizing the system tests
 - Add new test. Optimizations to make the tests robust
 - Change insertion index initialization approach. Add additional test
 - Initialize insertion index in PrismTextLayout
 - Address code review
 - ... and 2 more: https://git.openjdk.org/jfx/compare/c20f6d0f...c2685ba7

Changes: https://git.openjdk.org/jfx/pull/1154/files
 Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1154&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8302511
  Stats: 48 lines in 3 files changed: 3 ins; 27 del; 18 mod
  Patch: https://git.openjdk.org/jfx/pull/1154.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1154/head:pull/1154

PR: https://git.openjdk.org/jfx/pull/1154


More information about the openjfx-dev mailing list