[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Re: [Rev 02] RFR: 8176270: Adding ChangeListener to TextField.selectedTextProperty causes StringOutO
From:       Robert Lichtenberger <rlichten () openjdk ! java ! net>
Date:       2020-05-27 10:20:57
Message-ID: Cf2-y3eBintKMyyWPOvV8s60lRhVVHH6CifK-lNcltg=.3be68332-fc78-4446-afc5-b8ad41f089f9 () github ! com
[Download RAW message or body]

> This PR fixes JDK-8176270 by clamping the end index of the selected text to the \
> length of the text.

Robert Lichtenberger has updated the pull request incrementally with one additional \
commit since the last revision:

  8176270: Adding ChangeListener to TextField.selectedTextProperty causes \
StringOutOfBoundsException  
  This fix will execute the selection change before the insertion of new text.
  Only if adjustmentAmount is not 0 a second selection change will be made.

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

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/138/files
  - new: https://git.openjdk.java.net/jfx/pull/138/files/f5209e05..54eacb41

Webrevs:
 - full: https://webrevs.openjdk.java.net/jfx/138/webrev.02
 - incr: https://webrevs.openjdk.java.net/jfx/138/webrev.01-02

  Stats: 20 lines in 3 files changed: 10 ins; 2 del; 8 mod
  Patch: https://git.openjdk.java.net/jfx/pull/138.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/138/head:pull/138

PR: https://git.openjdk.java.net/jfx/pull/138


[prev in list] [next in list] [prev in thread] [next in thread]