RFR: 8302511: HitInfo.toString() throws IllegalArgumentException [v2]

Andy Goryachev angorya at openjdk.org
Thu Jun 15 18:38:14 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.

Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:

  review comments

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1154/files
  - new: https://git.openjdk.org/jfx/pull/1154/files/c2685ba7..8014ba33

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1154&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1154&range=00-01

  Stats: 163 lines in 3 files changed: 140 ins; 7 del; 16 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