RFR: 8304831: TextFlow.hitTest.insertionIndex incorrect with surrogate pairs [v6]

Karthik P K kpk at openjdk.org
Mon May 22 15:34:07 UTC 2023


On Fri, 19 May 2023 15:41:50 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> For the purposes of this test, we could remove this sequence from the test, I think.

Removed red heart from the text sequence.

> Another variant is to add something like that to Util and use that in place of Thread.sleep(). This method will trigger and wait for an arbitrary number of pulses (currently 10, but we can pick any reasonable number):

Added above suggested method in Util and tests looks to be stable now. Currently waiting for 10 pulses as suggested. Added parameterized method to specify the number of pulses to be waited as well.

Also removed the unnecessary latches as I was already using `Util.runAndWait(() -> { });`.

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

PR Comment: https://git.openjdk.org/jfx/pull/1091#issuecomment-1557312984


More information about the openjfx-dev mailing list